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

APIs to help in finding NPU SI settings #410

Merged
merged 2 commits into from
Dec 5, 2023

Conversation

mihirpat1
Copy link
Contributor

@mihirpat1 mihirpat1 commented Nov 2, 2023

Description

This PR has definition of APIs to retrieve transceiver specific information.
The change set is part of implementation of the HLD OA crash handling to reinitialize port through xcvrd

Motivation and Context

How Has This Been Tested?

Please refer to sonic-net/sonic-platform-daemons#403 for the test results.

Additional Information (Optional)

@mihirpat1
Copy link
Contributor Author

@prgeor @shyam77git @jaganbal-a - It will be great if you can help in reviewing this.

@mihirpat1 mihirpat1 marked this pull request as ready for review November 3, 2023 00:46
@mihirpat1 mihirpat1 requested a review from keboliu November 3, 2023 18:36
prgeor
prgeor previously approved these changes Nov 23, 2023
@prgeor prgeor force-pushed the npu_si_settings_key branch from af47627 to ecc3db4 Compare December 5, 2023 02:33
@prgeor prgeor merged commit d09e009 into sonic-net:master Dec 5, 2023
5 checks passed
@prgeor
Copy link
Collaborator

prgeor commented Dec 5, 2023

@mihirpat1 please track this for 202311 with MSFT ADO for cherry pick

@mihirpat1
Copy link
Contributor Author

@mihirpat1 please track this for 202311 with MSFT ADO for cherry pick

@StormLiangMS - Can you please help to cherry pick this to 202311?
ADO - 25709769

mssonicbld pushed a commit to mssonicbld/sonic-platform-common that referenced this pull request Feb 2, 2024
* APIs to help in finding NPU SI settings

Signed-off-by: Mihir Patel <[email protected]>

* Renamed get_cable_type to get_cable_length_type

---------

Signed-off-by: Mihir Patel <[email protected]>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #435

mssonicbld pushed a commit that referenced this pull request Feb 2, 2024
* APIs to help in finding NPU SI settings

Signed-off-by: Mihir Patel <[email protected]>

* Renamed get_cable_type to get_cable_length_type

---------

Signed-off-by: Mihir Patel <[email protected]>
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
* 400ZR not linking up with latest SONiC master image

Signed-off-by: Mihir Patel <[email protected]>

* 400ZR not linking up with latest SONiC master image

Signed-off-by: Mihir Patel <[email protected]>

* Added testcase to improve code coverage

---------

Signed-off-by: Mihir Patel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants