Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Inventec][D6356] Update Inventec 6356 #3839

Merged
merged 2 commits into from
Feb 10, 2020
Merged

[Inventec][D6356] Update Inventec 6356 #3839

merged 2 commits into from
Feb 10, 2020

Conversation

david-xk
Copy link
Contributor

@david-xk david-xk commented Dec 3, 2019

- What I did

  • [Platform API] Add get_change_event implementation for Chassis class
  • [Platforn monitor] skip ledd to avoid pmon init failure

- How I did it

  • Add pmon_daemon_control.json to device/inventec/x86_64-inventec_d6356-r0/
  • Add transceiver_event.py to platform/broadcom/sonic-platform-modules-inventec/d6356/sonic_platform
  • Modify chassis.py to platform/broadcom/sonic-platform-modules-inventec/d6356/sonic_platform

- How to verify it

  • check platform eeprom: show platform syseeprom
  • check platform PSUs: show platform psustatus
  • check interface status: show interfaces status
  • check transceiver presence: show interfaces transceiver presence

- Description for the changelog

  • 20191102
    • [Platform API] Add get_change_event implementation for Chassis class
    • [Platforn monitor] skip ledd to avoid pmon init failure

- A picture of a cute animal (not mandatory but encouraged)

David Xiao added 2 commits December 3, 2019 09:48
[Inventec][D6356] Update Inventec 6356
6471203 
   1.[Platform API] Add get_change_event implementation for Chassis class
   2.[Platforn monitor] skip ledd to avoid pmon init failure

Signed-off-by: David Xiao <xiao.david@inventec.com>
remove commented lines in led_proc_init.soc
eaa9c37 
replace tab with 4 spaces in platform-modules-d6356.init

Signed-off-by: David Xiao <xiao.david@inventec.com>
@david-xk david-xk mentioned this pull request Jan 19, 2020
Closed
@jleveque
Copy link
Contributor

Retest vsimage please

3 similar comments
@david-xk
Copy link
Contributor Author

Retest vsimage please

@david-xk
Copy link
Contributor Author

Retest vsimage please

@jleveque
Copy link
Contributor

jleveque commented Feb 7, 2020

Retest vsimage please

@david-xk
Copy link
Contributor Author

david-xk commented Feb 10, 2020 •
edited
Loading

Hi @jleveque, Looks like it past the vsiamge test, would you please merge this PR at your convenience?
Thanks,
David

@jleveque jleveque merged commit 1f4325e into sonic-net:master Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jleveque jleveque jleveque approved these changes

Assignees
No one assigned
Labels
Enhancement ➕ Platform: Inventec
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@david-xk @jleveque