Skip to content

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

Merged
mudsut4ke merged 6 commits intoSONIC-DEV:master_dx010_fix_failed_test_cases_of_apifrom
mudsut4ke:master_dx010_fix_failed_test_cases_of_api_fix
Mar 16, 2021
Merged

[device/celestica]: Fix error on Chassis and SFP APIs#240
mudsut4ke merged 6 commits intoSONIC-DEV:master_dx010_fix_failed_test_cases_of_apifrom
mudsut4ke:master_dx010_fix_failed_test_cases_of_api_fix

Conversation

@mudsut4ke
Copy link
Copy Markdown

@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
@mudsut4ke mudsut4ke requested a review from celestica-gl March 12, 2021 11:19
Copy link
Copy Markdown

@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