-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[Nokia][Device] Add BCM DNX soc properties to set STAN_ALN mode correctly #21864
base: master
Are you sure you want to change the base?
[Nokia][Device] Add BCM DNX soc properties to set STAN_ALN mode correctly #21864
Conversation
@vmittal-msft @arlakshm @rlhui , Please review it |
/azp run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@saksarav-nokia we may need to change high/low to 50%/25% of total fabric links as per BRCM recommendation. please update/re-test.
/azp run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
@vmittal-msft do we need this in 202411, could you pls check it. |
…ctly Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
255ca81
to
8160f63
Compare
/azp run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
No. Let me remove tag. |
Yes |
Thanks @vmittal-msft @saksarav-nokia Please create a separate PR for 202411 after merge as we have conflicts. |
Why I did it
The STAN_ALN mode was set incorrectly in BCM DNX asics and was advised by BCM to set these soc variables to correct it (One FAP with STAN_ALN=1 and other FAPs with STAN_ALN=0).
Work item tracking
How I did it
Added the soc variables in BCM config
How to verify it
Verified the STAN_ALN mode and also passing traffic without any issues.
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)