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

[action] [PR:490] [CMIS] Add lane_mask parameter to set_loopback_mode() to enable setti… (#490) #505

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

mssonicbld
Copy link
Collaborator

  • [CMIS] Add lane_mask parameter to set_loopback_mode() to enable setting loopback mode for individual lanes rather than the entire physical port

Signed-off-by: xinyu [email protected]

  • [CMIS] Add options to individually clear host/media, input/output loopback modes and log errors for all rejection cases

Signed-off-by: xinyu [email protected]

  • [CMIS] Reject the loopback mode request if simultaneous host and media loopback is not supported or per-lane loopback is not supported

Signed-off-by: xinyu [email protected]

  • [CMIS] Log errors for all set loopback mode rejection cases

Signed-off-by: xinyu [email protected]

  • [CMIS] Improve readability by using individual functions for setting host, media, input, and output loopback modes

Signed-off-by: xinyu [email protected]

  • [CMIS] Add an enable parameter to eliminate the need for loopback modes with the -none suffix

Signed-off-by: xinyu [email protected]


Signed-off-by: xinyu [email protected]

sonic-net#490)

* [CMIS] Add lane_mask parameter to set_loopback_mode() to enable setting loopback mode for individual lanes rather than the entire physical port

Signed-off-by: xinyu <[email protected]>

* [CMIS] Add options to individually clear host/media, input/output loopback modes and log errors for all rejection cases

Signed-off-by: xinyu <[email protected]>

* [CMIS] Reject the loopback mode request if simultaneous host and media loopback is not supported or per-lane loopback is not supported

Signed-off-by: xinyu <[email protected]>

* [CMIS] Log errors for all set loopback mode rejection cases

Signed-off-by: xinyu <[email protected]>

* [CMIS] Improve readability by using individual functions for setting host, media, input, and output loopback modes

Signed-off-by: xinyu <[email protected]>

* [CMIS] Add an enable parameter to eliminate the need for loopback modes with the -none suffix

Signed-off-by: xinyu <[email protected]>

---------

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

Original PR: #490

@mssonicbld mssonicbld merged commit 1abd824 into sonic-net:202405 Oct 17, 2024
5 checks passed
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.

2 participants