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.

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

[device/celestica]: Fix error on Chassis and SFP APIs #240

Conversation

mudsut4ke
Copy link

@mudsut4ke mudsut4ke commented Mar 11, 2021

- What I did

  • Fix Chassis APIs error
  • Fix SFP APIs error

- How I did it

  • Add STATUS_LED_PATH
  • Update EEROM API
  • Update SFP APIs follow Seastone2

- How to verify it

  • Runing platform api test on testbed

- Description for the changelog

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

@mudsut4ke mudsut4ke marked this pull request as ready for review March 12, 2021 11:19
Copy link

@119064273 119064273 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good, thanks

@mudsut4ke mudsut4ke merged commit 2e572e4 into SONIC-DEV:master_dx010_fix_failed_test_cases_of_api Mar 16, 2021
mudsut4ke pushed a commit that referenced this pull request Jan 28, 2022
…t#9412)

c2aac75 [SFP-Refactor] Fix LP mode API issue (#247)
dba17c8 Firmware upgrade CLI support for QSFP-DD transceivers (#244)
cd69212 [SFP-Refactor] Implement CMIS Low Power mode (#237)
9cea07f Fix RegGroupField decode (#245)
6ae1909 Add CMIS QSFP support (#246)
c1f317d Gracefully handle CMIS APIs for passive modules (#238)
ec7335d fix for firmware functions  (#243)
cf2ebe9 Fix RegBitField decode/encode (#242)
ef4f2c6 Fix SFP_CABLE_TECH_FIELD (#240)
e118644 remove time counting message in functions because function running time could be difficult to predict in unit tests (#241)

Signed-off-by: Prince George <prgeor@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants