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

CLONE: saiport: add support for link training ability query #1474

Closed
wants to merge 1 commit into from

Conversation

ds952811
Copy link
Contributor

Cherry-pick PR#1434

  • Description
    Add support for link-training ability query to suppress redundant link-training requests those are doomed to fail

  • Motivation and Context
    link-training could either be unavailable on certain ports of a switch silicon or totally unsupported on an entire switch silicon
    In order to prevent swss and syncd error out from redundant link-training requests, it's better to introduce this port attribute

Signed-off-by: Dante Su dante.su@broadcom.com

Related HLD:

Repo PR title State
SONiC [link-training] initial support

Cherry-pick PR#1434

- Description
Add support for link-training ability query to suppress redundant link-training requests those are doomed to fail

- Motivation and Context
link-training could either be unavailable on certain ports of a switch silicon or totally unsupported on an entire switch silicon
In order to prevent swss and syncd error out from redundant link-training requests, it's better to introduce this port attribute

Signed-off-by: Dante Su <dante.su@broadcom.com>
@adyeung
Copy link

adyeung commented May 5, 2022

@rlhui pls help signoff and merge this CLONE PR for v1.9

@ds952811 ds952811 closed this May 9, 2022
@ds952811
Copy link
Contributor Author

ds952811 commented May 9, 2022

Replaced by PR #1475, retargeted to v1.10

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