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

[device/celestica]: Implement Silverstone platform API [THERMAL/COMPONENT] #3938

Merged
merged 2 commits into from
Jan 16, 2020
Merged

[device/celestica]: Implement Silverstone platform API [THERMAL/COMPONENT] #3938

merged 2 commits into from
Jan 16, 2020

Conversation

mudsut4ke
Copy link
Contributor

@mudsut4ke mudsut4ke commented Dec 23, 2019 •
edited
Loading

- What I did

  • Add Thermal APIs
  • Add Component APIs

- How I did it

  1. Implement Sfp APIs.
  2. Implement Component APIs.
  3. Init component object and thermal object list on chassis API.

- How to verify it

Note: Require #3607
Signed-off-by: Wirut Getbamrung [email protected]

@mudsut4ke mudsut4ke marked this pull request as ready for review January 6, 2020 04:14
@yxieca yxieca requested review from jleveque and sujinmkang January 16, 2020 02:19
@lguohan lguohan merged commit 4c526d9 into sonic-net:master Jan 16, 2020
@mudsut4ke mudsut4ke deleted the master-silverstone-thermal-component branch April 16, 2020 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan lguohan approved these changes

@jleveque jleveque Awaiting requested review from jleveque

@sujinmkang sujinmkang Awaiting requested review from sujinmkang

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mudsut4ke @lguohan