-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[Ufispace] Update BCM port and PDDF platform configurations due to SDK and firmware updates. #21483
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
Conversation
/azp run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
@cytsai0409 , please help review the change. Thanks! |
@nonodark Looks good to me. Thanks. |
/azp run Azure.sonic-buildimage |
Commenter does not have sufficient privileges for PR 21483 in repo sonic-net/sonic-buildimage |
…C 11.8 firmware no longer supports it. Remove the high threshold of the PSU on the following platforms, as the BMC 11.8 firmware no longer supports it. * s7801-54xs * s8901-54xc * s9110-32x
b9b10d3
to
b6b5831
Compare
/azp run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
@yxieca Could you please assist me in merging it? |
…K and firmware updates. (sonic-net#21483) * [ufispace][platforms] Remove the high threshold of the PSU, as the BMC 11.8 firmware no longer supports it. Remove the high threshold of the PSU on the following platforms, as the BMC 11.8 firmware no longer supports it. * s7801-54xs * s8901-54xc * s9110-32x * [ufispace][s9110-32x] Update bcm port configuration file
@yxieca Is it also possible to cherry-pick to the 202411 branch? |
@nonodark please state the justification for this change to be included 202411 to let @kperumalbfn to assess. Is this a must have as to match new SAI/SDK? Or new feature, feature enhancement, bug fix? |
[ufispace][s9110-32x] Update bcm port configuration file [ufispace][platforms] Remove the high threshold of the PSU, as the BMC 11.8 firmware no longer supports it.
Yes, I have validated with the latest BCM SAI/SDK (12.3) in master, and it worked fine.
@kperumalbfn and @prgeor, could you please approve cherry-picking this to the 202411 branch? |
@kperumalbfn and @prgeor, could you please provide me with some feedback? Is there any additional information I need to provide? |
@kperumalbfn and @prgeor, could you please provide me with some feedback? Do I need to provide any additional information to cherry-pick this to the 202411 branch? |
1 similar comment
@kperumalbfn and @prgeor, could you please provide me with some feedback? Do I need to provide any additional information to cherry-pick this to the 202411 branch? |
@prgeor Can you confirm this for 202411 |
…K and firmware updates. (sonic-net#21483) * [ufispace][platforms] Remove the high threshold of the PSU, as the BMC 11.8 firmware no longer supports it. Remove the high threshold of the PSU on the following platforms, as the BMC 11.8 firmware no longer supports it. * s7801-54xs * s8901-54xc * s9110-32x * [ufispace][s9110-32x] Update bcm port configuration file
…K and firmware updates. (sonic-net#21483) * [ufispace][platforms] Remove the high threshold of the PSU, as the BMC 11.8 firmware no longer supports it. Remove the high threshold of the PSU on the following platforms, as the BMC 11.8 firmware no longer supports it. * s7801-54xs * s8901-54xc * s9110-32x * [ufispace][s9110-32x] Update bcm port configuration file
Why I did it
Update BCM port and PDDF platform configurations due to SDK and firmware updates.
Work item tracking
How I did it
Update BCM port and PDDF platform configuration files.
How to verify it
Run the PDDF and show commands to check the platform status
S9110-32X without the BCM port configuration fix.
S9110-32X with the BCM port configuration fix.
Without the fix for the high threshold removal of the PSU, the temperature display for TEMP_PSU0_TEMP1 and TEMP_PSU1_TEMP1 will appear abnormal.
s9110-32x
s8901-54xc
s7801-54xs
With the fix for the high threshold removal of the PSU, the temperature display for TEMP_PSU0_TEMP1 and TEMP_PSU1_TEMP1 will appear normal.
s9110-32x
s8901-54xc
s7801-54xs
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)