-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[devices]: DellEMC new platform support for DellEMC s5296f- 96x25G #3960
[devices]: DellEMC new platform support for DellEMC s5296f- 96x25G #3960
Conversation
* Switch Vendor: DellEMC * Switch SKU: s5296F * ASIC Vendor: Broadcom * Swich ASIC: Trident3 * Port Configuration: 96x25G * SONiC Image: sonic-broadcom.bin * NPU device profile support Added 25G(96x25G,8x100G) and 10G(96x10G,8x100G) configruation ( 25G is default). * LED support for s5296f, * platform changes include syseeprom,ipmitool implementation for platform_sensors script is inclued in pmon startup.
Test logs root@sonic:~# show platform summary root@sonic:~# show platform syseeprom Product Name 0x21 9 S5296F-ON (checksum valid) root@sonic:~# show environment coretemp-isa-0000 Onboard Temperature Sensors: Fan Trays: PSUs: PSU 2 Not present
root@sonic:~# root@sonic:~# show interfaces status Ethernet0 5 25G 9100 twentyfiveGigE1/1/1 routed down up SFP/SFP+/SFP28 N/A root@sonic:~# show interfaces transceiver eeprom --dom Ethernet0 root@sonic:~# |
Retest vsimage please |
Retest mellanox please |
Retest vsimage please |
2 similar comments
Retest vsimage please |
Retest vsimage please |
Retest vsimage please |
platform/broadcom/sonic-platform-modules-dell/debian/platform-modules-s5296f.init
Outdated
Show resolved
Hide resolved
platform/broadcom/sonic-platform-modules-dell/s5296f/scripts/s5296f_platform.sh
Outdated
Show resolved
Hide resolved
platform/broadcom/sonic-platform-modules-dell/s5296f/scripts/s5296f_platform.sh
Outdated
Show resolved
Hide resolved
platform/broadcom/sonic-platform-modules-dell/s5296f/scripts/s5296f_platform.sh
Outdated
Show resolved
Hide resolved
platform/broadcom/sonic-platform-modules-dell/s5296f/scripts/s5296f_platform.sh
Show resolved
Hide resolved
platform/broadcom/sonic-platform-modules-dell/s5296f/scripts/s5296f_platform.sh
Show resolved
Hide resolved
platform/broadcom/sonic-platform-modules-dell/s5296f/scripts/s5296f_platform.sh
Show resolved
Hide resolved
platform/broadcom/sonic-platform-modules-dell/s5296f/scripts/sensors
Outdated
Show resolved
Hide resolved
- Indentation, formatting - Removing dead code Signed-off-by: Howard Persh <[email protected]>
Retest vsimage please |
Retest broadcom please |
* For Dell EMC S5296F, obey timeout argument in xcvrd get_transceiver_change_event() Signed-off-by: Howard Persh <[email protected]>
retest default please |
retest mellanox please |
retest default please |
This pull request introduces 26 alerts when merging d0af265 into 30bbbbf - view on LGTM.com new alerts:
|
Retest vsimage please |
Signed-off-by: Howard Persh <[email protected]>
eb3ee6a
to
90a82a7
Compare
Retest vsimage please |
Retest vsimage please |
device/dell/x86_64-dellemc_s5296f_c3538-r0/pmon_daemon_control.json
Outdated
Show resolved
Hide resolved
Signed-off-by: Howard Persh <[email protected]>
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.
LGTM. @lguohan to review as well, please.
retest broadcom please |
I'm not really sure why the "broadcom" test is failing. From the Jenkins log, sonic-broadcom.bin seems to build OK, but another installer fails. It seems unrelated to these changes. |
retest broadcom please |
Hi, can this be approved/merged? Been sitting for some time now. We would like to open a follow-up PR to migrate this platform to API 2.0 after this is merged. |
…onic-net#3960) Added files, driver, npu configs for the DellEMC S5296f platform
- What I did
Added platform support for DellEMC S5296f
- How I did it
Added files,driver,npu configs for the platforms
- How to verify it
Verified build and tested on DellEMC S5296 platform.
logs attached below.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)