Skip to content

DellEMC: DellEMC platform bug fixes#4071

Merged
lguohan merged 1 commit intosonic-net:masterfrom
srideepDell:s5248_bugfix
Jan 28, 2020
Merged

DellEMC: DellEMC platform bug fixes#4071
lguohan merged 1 commit intosonic-net:masterfrom
srideepDell:s5248_bugfix

Conversation

@srideepDell
Copy link
Contributor

      * Dynamic bus detection and loading driver for eeprom on s5248,s5232
      * sfputil fix for S5248

- What I did
* Dynamic bus detection and loading driver for eeprom on s5248,s5232
* sfputil fix for S5248
- How I did it
Modified Dell EMC platform specific files
- How to verify it
Verified on S5232 and S5248 platfoms.
logs below.
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

          * Dynamic bus detection and loading driver for eeprom on s5248,s5232
          * sfputil fix for S5248
@srideepDell
Copy link
Contributor Author

Logs from commands for the changes

root@sonic:/# show platform syseeprom
TlvInfo Header:
Id String: TlvInfo
Version: 1
Total Length: 180
TLV Name Code Len Value


Product Name 0x21 9 S5248F-ON
Part Number 0x22 6 01XR4W
Serial Number 0x23 20 CN046MRJCES0085E0016
Base MAC Address 0x24 6 54:BF:64:BA:3E:C0
Manufacture Date 0x25 19 01/15/2017 11:03:36
Device Version 0x26 1 1
Label Revision 0x27 3 X01
Platform Name 0x28 30 x86_64-dellemc_s5248f_c2538-r0
ONIE Version 0x29 10 3.40.1.1-6
MAC Addresses 0x2A 2 256
Manufacturer 0x2B 5 77931
Manufacture Country 0x2C 2 CN
Vendor Name 0x2D 8 DELL EMC
Diag Version 0x2E 10 3.40.4.1-2
Service Tag 0x2F 7 GCPQG02
Vendor Extension 0xFD 4
CRC-32 0xFE 4 0xD5CB85D8

(checksum valid)

root@sonic:/# show interfaces status
Interface Lanes Speed MTU Alias Vlan Oper Admin Type Asym PFC


Ethernet0 49 25G 9100 twentyfiveGigE1/1/1 routed up up SFP/SFP+/SFP28 N/A
Ethernet1 50 25G 9100 twentyfiveGigE1/1/2 routed up up SFP/SFP+/SFP28 N/A
Ethernet2 51 25G 9100 twentyfiveGigE1/1/3 routed down up N/A N/A
Ethernet3 52 25G 9100 twentyfiveGigE1/1/4 routed down up N/A N/A
Ethernet4 57 25G 9100 twentyfiveGigE1/2/1 routed down up N/A N/A
Ethernet5 58 25G 9100 twentyfiveGigE1/2/2 routed down up N/A N/A
Ethernet6 59 25G 9100 twentyfiveGigE1/2/3 routed down up N/A N/A
Ethernet7 60 25G 9100 twentyfiveGigE1/2/4 routed down up N/A N/A
Ethernet8 61 25G 9100 twentyfiveGigE1/3/1 routed down up N/A N/A
Ethernet9 62 25G 9100 twentyfiveGigE1/3/2 routed down up N/A N/A
Ethernet10 63 25G 9100 twentyfiveGigE1/3/3 routed down up N/A N/A
Ethernet11 64 25G 9100 twentyfiveGigE1/3/4 routed down up N/A N/A
Ethernet12 77 25G 9100 twentyfiveGigE1/4/1 routed down up N/A N/A
Ethernet13 78 25G 9100 twentyfiveGigE1/4/2 routed down up N/A N/A
Ethernet14 79 25G 9100 twentyfiveGigE1/4/3 routed down up N/A N/A
Ethernet15 80 25G 9100 twentyfiveGigE1/4/4 routed down up N/A N/A
Ethernet16 85 25G 9100 twentyfiveGigE1/5/1 routed up up SFP/SFP+/SFP28 N/A
Ethernet17 86 25G 9100 twentyfiveGigE1/5/2 routed up up SFP/SFP+/SFP28 N/A
Ethernet18 87 25G 9100 twentyfiveGigE1/5/3 routed down up N/A N/A
Ethernet19 88 25G 9100 twentyfiveGigE1/5/4 routed down up N/A N/A
Ethernet20 93 25G 9100 twentyfiveGigE1/6/1 routed down up N/A N/A
Ethernet21 94 25G 9100 twentyfiveGigE1/6/2 routed down up N/A N/A
Ethernet22 95 25G 9100 twentyfiveGigE1/6/3 routed down up N/A N/A
Ethernet23 96 25G 9100 twentyfiveGigE1/6/4 routed down up N/A N/A
Ethernet24 13 25G 9100 twentyfiveGigE1/7/1 routed down up N/A N/A
Ethernet25 14 25G 9100 twentyfiveGigE1/7/2 routed down up N/A N/A
Ethernet26 15 25G 9100 twentyfiveGigE1/7/3 routed down up N/A N/A
Ethernet27 16 25G 9100 twentyfiveGigE1/7/4 routed down up N/A N/A
Ethernet28 21 25G 9100 twentyfiveGigE1/8/1 routed down up N/A N/A
Ethernet29 22 25G 9100 twentyfiveGigE1/8/2 routed down up N/A N/A
Ethernet30 23 25G 9100 twentyfiveGigE1/8/3 routed down up N/A N/A
Ethernet31 24 25G 9100 twentyfiveGigE1/8/4 routed down up N/A N/A
Ethernet32 29 25G 9100 twentyfiveGigE1/9/1 routed down up N/A N/A
Ethernet33 30 25G 9100 twentyfiveGigE1/9/2 routed down up N/A N/A
Ethernet34 31 25G 9100 twentyfiveGigE1/9/3 routed down up N/A N/A
Ethernet35 32 25G 9100 twentyfiveGigE1/9/4 routed down up N/A N/A
Ethernet36 97 25G 9100 twentyfiveGigE1/10/1 routed down up N/A N/A
Ethernet37 98 25G 9100 twentyfiveGigE1/10/2 routed down up N/A N/A
Ethernet38 99 25G 9100 twentyfiveGigE1/10/3 routed down up N/A N/A
Ethernet39 100 25G 9100 twentyfiveGigE1/10/4 routed down up N/A N/A
Ethernet40 105 25G 9100 twentyfiveGigE1/11/1 routed down up N/A N/A
Ethernet41 106 25G 9100 twentyfiveGigE1/11/2 routed down up N/A N/A
Ethernet42 107 25G 9100 twentyfiveGigE1/11/3 routed down up N/A N/A
Ethernet43 108 25G 9100 twentyfiveGigE1/11/4 routed down up N/A N/A
Ethernet44 113 25G 9100 twentyfiveGigE1/12/1 routed down up N/A N/A
Ethernet45 114 25G 9100 twentyfiveGigE1/12/2 routed down up N/A N/A
Ethernet46 115 25G 9100 twentyfiveGigE1/12/3 routed down up N/A N/A
Ethernet47 116 25G 9100 twentyfiveGigE1/12/4 routed down up N/A N/A
Ethernet48 121,122,123,124 100G 9100 hundredGigE1/49 routed up up QSFP28 or later N/A
Ethernet49 125,126,127,128 100G 9100 hundredGigE1/50 routed down up N/A N/A
Ethernet50 69,70,71,72 100G 9100 hundredGigE1/51 routed up up QSFP28 or later N/A
Ethernet51 65,66,67,68 100G 9100 hundredGigE1/52 routed down up N/A N/A
Ethernet52 1,2,3,4 100G 9100 hundredGigE1/53 routed down up Unknown or unspecified N/A
Ethernet53 33,34,35,36 100G 9100 hundredGigE1/54 routed down up Unknown N/A
Ethernet54 5,6,7,8 100G 9100 hundredGigE1/55 routed down up N/A N/A
Ethernet55 41,42,43,44 100G 9100 hundredGigE1/56 routed down up N/A N/A
root@sonic:/#

@srideepDell
Copy link
Contributor Author

restest vsimage please

@srideepDell
Copy link
Contributor Author

Retest vsimage please

@jleveque
Copy link
Contributor

Retest vs please

@srideepDell
Copy link
Contributor Author

Retest vsimage please

@srideepDell
Copy link
Contributor Author

Retest vs please

@lguohan lguohan merged commit dc994b8 into sonic-net:master Jan 28, 2020
@srideepDell srideepDell deleted the s5248_bugfix branch December 2, 2020 07:44
mssonicbld added a commit that referenced this pull request Oct 26, 2025
…atically (#24272)

#### Why I did it
src/sonic-utilities
```
* 8d2bc08 - (HEAD -> master, origin/master, origin/HEAD) Add pfc_stat_history support (#4102) (6 hours ago) [Xincun Li]
* 7a046d6 - [trim]: Fix GCU trimming eligibility modification (#4087) (28 hours ago) [Nazarii Hnydyn]
* f4e5de3 - [GCU] Handle duplicate array entries and auto-create empty tables during patch application (#4095) (2 days ago) [Xincun Li]
* a131061 - [fast/warm-reboot] Fix timers query (#4022) (2 days ago) [Stepan Blyshchak]
* 3bf5c27 - [Mellanox] Update generate_dump to include SDK sysfs files (#4071) (2 days ago) [Noa Or]
* d4eb8ec - [portstat] Add FEC FLR statistics support to port counters (#4054) (3 days ago) [Apoorv Sachan]
* 55b665b - Secureboot: Image signing verification enhancements (#3989) (7 days ago) [Brad House - NextHop]
* e59bbfc - Fixing state_db not having delete_field attribute causing a crash when DPUs in bad state (#4064) (12 days ago) [rameshraghupathy]
* 9386963 - Improve set/get cmdline to support uefi (#4062) (2 weeks ago) [Hua Liu]
* 3a7d0b4 - [dhcp_relay] Update show cli sample for dhcp_relay (#4070) (2 weeks ago) [Balakrishna-goshika]
* 89c9aef - FEC histogram with ability to clear stat (#4075) (3 weeks ago) [Prince George]
* b247e93 - Skip speed validation for chassis. (#4076) (3 weeks ago) [Xincun Li]
* 70926dd - [FRR]Adding additional FRR dumps (#4073) (3 weeks ago) [Sudharsan Dhamal Gopalarathnam]
* d7c16c3 - Fix incorrect output format for pre-fec ber in sfpshow pm (#4066) (4 weeks ago) [Changrong Wu]
* 80a20e7 - [doc][dhcp_server] Update cli doc for dhcp_server #4069 (4 weeks ago) [Balakrishna-goshika]
* c1843fa - Issue #22759: Prevent CLI from adding invalid routed interfaces (#3901) (4 weeks ago) [Anders Linn]
* 7c5378e - Issue 23798: Wrap getpass.getpass in a signal handler to avoid SIGTTOU (#4061) (5 weeks ago) [Anders Linn]
* 28dfb29 - Fix issue that dynamic/static threshold 0 can not be configured using mmuconfig (#4049) (5 weeks ago) [Stephen Sun]
* e276765 - Support multi-asic in gcu.py (#4057) (6 weeks ago) [ganglv]
* d2c697f - Add sonic-error-report tool for structured error reporting (#4037) (6 weeks ago) [Dawei Huang]
* ed5afd8 - Add python wheels for GCU (#4042) (6 weeks ago) [ganglv]
* 98e4916 - Add Arista-7060X6-64PE-B-O128S2, Arista-7060X6-16PE-384C-B-O128S2 to GCU (#4055) (6 weeks ago) [rick-arista]
* 9e9a65b - Issue #22420: Modify 'config route add' command not to include empty elements (#12) (#3862) (7 weeks ago) [Anders Linn]
* 0edb592 - Mux cable show config command Added prober_type and fixed one format (#4013) (7 weeks ago) [harjotsinghpawra]
* 2657ee3 - Fixed cli command for ECN config on voq switch to set the WRED_PROFILE for all Voqs (#4029) (7 weeks ago) [saksarav-nokia]
* d1c9d1a - [show][config] Add CLI support for configurable drop monitor feature (#3756) (7 weeks ago) [HP]
* 7baa75b - [spm] Rename entry tag variable to docker_image_reference (#4019) (8 weeks ago) [DavidZagury]
* 63364a3 - Add BlockingMode for Reboot script (#3958) (8 weeks ago) [Litao Yu]
* ee8113f - Support for platforms based on Clounix net device (#3970) (8 weeks ago) [LongWuuu]
* f53a5c1 - [config show]BGP Suppress fib pending config and display for multi-asic (#3948) (8 weeks ago) [vganesan-nokia]
* b3de0af - Add Arista 7800 platforms to GCU validator (#4038) (9 weeks ago) [Xincun Li]
* 13a0cb2 - Add check_pfc_storm_active() to fast-reboot script (#3969) (9 weeks ago) [Dawei Huang]
* f45d896 - [smartswitch] Update get_gnmi_port() based on smartswitch config updates (#4041) (9 weeks ago) [Vasundhara Volam]
* ea33ef3 - [nvidia-bluefield] Add CLI for packet-drop and config-record (#4002) (9 weeks ago) [Vivek]
* ffc891d - [dhcp_server] Add CLI sample for dhcp_server (#4033) (10 weeks ago) [Yaqiang Zhu]
* 1e9d04c - Update doc to including dhcp_server ipv4 counter related CLI (#4028) (2 months ago) [Yaqiang Zhu]
* 19594b9 - Fix show int transceiver EEPROM crash for for Backplane cartridge + enhance EEPROM CLI output (#4020) (2 months ago) [mihirpat1]
* 0f8ac9b - Added MAX pre-fec_ber for FEC counter (#4027) (2 months ago) [Prince George]
* 732dc09 - Added json support for show platform temperature (#3874) (2 months ago) [Vinod Kumar]
* bacff45 - Add Arista 7800 platforms to GCU validator (#4030) (2 months ago) [Xincun Li]
* c63e9ea - [trim]: Add Packet Trimming Drop Counters CLI (#3993) (3 months ago) [Nazarii Hnydyn]
* 50df9ea - Adapt 'show muxcable tunnel-route' for prefix route based mux neighbors (#4007) (3 months ago) [manamand2020]
* 868189c - Pr json support queue and priority-group watermark and persistent-watermark (#3875) (3 months ago) [Vinod Kumar]
* 5347757 - Revert "[SPM] Rename the variable tag to docker-image-reference (#3998)" (#4024) (3 months ago) [Jianquan Ye]
* 1418f21 - Added json support intfutil (#3906) (3 months ago) [Vinod Kumar]
* ec01962 - sfputil and sfpshow eeprom and DOM CLI enhancement to display data for all CMIS transceivers (#4010) (3 months ago) [mihirpat1]
* c0838d7 - CLI for Configuring PFC Historical Statistics (#3779) (3 months ago) [Peter Bailey]
* d623c25 - [Mellanox][Smartswitch]Added dpu status output to dump (#3959) (3 months ago) [Gagan Punathil Ellath]
* a3101ea - Fix for #23205 [Smartswitch] Issues caused due to introduction of the chassisd/sonic-utiltiies changes for consecutive admin state changes (#3984) (3 months ago) [rameshraghupathy]
* d3bc688 - CLI addition for PFC counters --history (#3778) (3 months ago) [Peter Bailey]
* 3282ab3 - DOM for flat memory transceiver modules (#3950) (3 months ago) [Ariz Zubair]
* 6f1a794 - Add queuestat changes for aggregate VOQ counters (#3617) (3 months ago) [Vivek Verma]
* d86b2b6 - g[sfputil debug] Fix issue: do not check output status when CMIS version is lower than 5.0 (#3938) (3 months ago) [Junchao-Mellanox]
* 252a643 - [SPM] Rename the variable tag to docker-image-reference (#3998) (3 months ago) [DavidZagury]
```
#### How I did it
#### How to verify it
#### Description for the changelog
FengPan-Frank pushed a commit to FengPan-Frank/sonic-buildimage that referenced this pull request Dec 4, 2025
…atically (sonic-net#24272)

#### Why I did it
src/sonic-utilities
```
* 8d2bc08 - (HEAD -> master, origin/master, origin/HEAD) Add pfc_stat_history support (sonic-net#4102) (6 hours ago) [Xincun Li]
* 7a046d6 - [trim]: Fix GCU trimming eligibility modification (sonic-net#4087) (28 hours ago) [Nazarii Hnydyn]
* f4e5de3 - [GCU] Handle duplicate array entries and auto-create empty tables during patch application (sonic-net#4095) (2 days ago) [Xincun Li]
* a131061 - [fast/warm-reboot] Fix timers query (sonic-net#4022) (2 days ago) [Stepan Blyshchak]
* 3bf5c27 - [Mellanox] Update generate_dump to include SDK sysfs files (sonic-net#4071) (2 days ago) [Noa Or]
* d4eb8ec - [portstat] Add FEC FLR statistics support to port counters (sonic-net#4054) (3 days ago) [Apoorv Sachan]
* 55b665b - Secureboot: Image signing verification enhancements (sonic-net#3989) (7 days ago) [Brad House - NextHop]
* e59bbfc - Fixing state_db not having delete_field attribute causing a crash when DPUs in bad state (sonic-net#4064) (12 days ago) [rameshraghupathy]
* 9386963 - Improve set/get cmdline to support uefi (sonic-net#4062) (2 weeks ago) [Hua Liu]
* 3a7d0b4 - [dhcp_relay] Update show cli sample for dhcp_relay (sonic-net#4070) (2 weeks ago) [Balakrishna-goshika]
* 89c9aef - FEC histogram with ability to clear stat (sonic-net#4075) (3 weeks ago) [Prince George]
* b247e93 - Skip speed validation for chassis. (sonic-net#4076) (3 weeks ago) [Xincun Li]
* 70926dd - [FRR]Adding additional FRR dumps (sonic-net#4073) (3 weeks ago) [Sudharsan Dhamal Gopalarathnam]
* d7c16c3 - Fix incorrect output format for pre-fec ber in sfpshow pm (sonic-net#4066) (4 weeks ago) [Changrong Wu]
* 80a20e7 - [doc][dhcp_server] Update cli doc for dhcp_server sonic-net#4069 (4 weeks ago) [Balakrishna-goshika]
* c1843fa - Issue sonic-net#22759: Prevent CLI from adding invalid routed interfaces (sonic-net#3901) (4 weeks ago) [Anders Linn]
* 7c5378e - Issue 23798: Wrap getpass.getpass in a signal handler to avoid SIGTTOU (sonic-net#4061) (5 weeks ago) [Anders Linn]
* 28dfb29 - Fix issue that dynamic/static threshold 0 can not be configured using mmuconfig (sonic-net#4049) (5 weeks ago) [Stephen Sun]
* e276765 - Support multi-asic in gcu.py (sonic-net#4057) (6 weeks ago) [ganglv]
* d2c697f - Add sonic-error-report tool for structured error reporting (sonic-net#4037) (6 weeks ago) [Dawei Huang]
* ed5afd8 - Add python wheels for GCU (sonic-net#4042) (6 weeks ago) [ganglv]
* 98e4916 - Add Arista-7060X6-64PE-B-O128S2, Arista-7060X6-16PE-384C-B-O128S2 to GCU (sonic-net#4055) (6 weeks ago) [rick-arista]
* 9e9a65b - Issue sonic-net#22420: Modify 'config route add' command not to include empty elements (sonic-net#12) (sonic-net#3862) (7 weeks ago) [Anders Linn]
* 0edb592 - Mux cable show config command Added prober_type and fixed one format (sonic-net#4013) (7 weeks ago) [harjotsinghpawra]
* 2657ee3 - Fixed cli command for ECN config on voq switch to set the WRED_PROFILE for all Voqs (sonic-net#4029) (7 weeks ago) [saksarav-nokia]
* d1c9d1a - [show][config] Add CLI support for configurable drop monitor feature (sonic-net#3756) (7 weeks ago) [HP]
* 7baa75b - [spm] Rename entry tag variable to docker_image_reference (sonic-net#4019) (8 weeks ago) [DavidZagury]
* 63364a3 - Add BlockingMode for Reboot script (sonic-net#3958) (8 weeks ago) [Litao Yu]
* ee8113f - Support for platforms based on Clounix net device (sonic-net#3970) (8 weeks ago) [LongWuuu]
* f53a5c1 - [config show]BGP Suppress fib pending config and display for multi-asic (sonic-net#3948) (8 weeks ago) [vganesan-nokia]
* b3de0af - Add Arista 7800 platforms to GCU validator (sonic-net#4038) (9 weeks ago) [Xincun Li]
* 13a0cb2 - Add check_pfc_storm_active() to fast-reboot script (sonic-net#3969) (9 weeks ago) [Dawei Huang]
* f45d896 - [smartswitch] Update get_gnmi_port() based on smartswitch config updates (sonic-net#4041) (9 weeks ago) [Vasundhara Volam]
* ea33ef3 - [nvidia-bluefield] Add CLI for packet-drop and config-record (sonic-net#4002) (9 weeks ago) [Vivek]
* ffc891d - [dhcp_server] Add CLI sample for dhcp_server (sonic-net#4033) (10 weeks ago) [Yaqiang Zhu]
* 1e9d04c - Update doc to including dhcp_server ipv4 counter related CLI (sonic-net#4028) (2 months ago) [Yaqiang Zhu]
* 19594b9 - Fix show int transceiver EEPROM crash for for Backplane cartridge + enhance EEPROM CLI output (sonic-net#4020) (2 months ago) [mihirpat1]
* 0f8ac9b - Added MAX pre-fec_ber for FEC counter (sonic-net#4027) (2 months ago) [Prince George]
* 732dc09 - Added json support for show platform temperature (sonic-net#3874) (2 months ago) [Vinod Kumar]
* bacff45 - Add Arista 7800 platforms to GCU validator (sonic-net#4030) (2 months ago) [Xincun Li]
* c63e9ea - [trim]: Add Packet Trimming Drop Counters CLI (sonic-net#3993) (3 months ago) [Nazarii Hnydyn]
* 50df9ea - Adapt 'show muxcable tunnel-route' for prefix route based mux neighbors (sonic-net#4007) (3 months ago) [manamand2020]
* 868189c - Pr json support queue and priority-group watermark and persistent-watermark (sonic-net#3875) (3 months ago) [Vinod Kumar]
* 5347757 - Revert "[SPM] Rename the variable tag to docker-image-reference (sonic-net#3998)" (sonic-net#4024) (3 months ago) [Jianquan Ye]
* 1418f21 - Added json support intfutil (sonic-net#3906) (3 months ago) [Vinod Kumar]
* ec01962 - sfputil and sfpshow eeprom and DOM CLI enhancement to display data for all CMIS transceivers (sonic-net#4010) (3 months ago) [mihirpat1]
* c0838d7 - CLI for Configuring PFC Historical Statistics (sonic-net#3779) (3 months ago) [Peter Bailey]
* d623c25 - [Mellanox][Smartswitch]Added dpu status output to dump (sonic-net#3959) (3 months ago) [Gagan Punathil Ellath]
* a3101ea - Fix for sonic-net#23205 [Smartswitch] Issues caused due to introduction of the chassisd/sonic-utiltiies changes for consecutive admin state changes (sonic-net#3984) (3 months ago) [rameshraghupathy]
* d3bc688 - CLI addition for PFC counters --history (sonic-net#3778) (3 months ago) [Peter Bailey]
* 3282ab3 - DOM for flat memory transceiver modules (sonic-net#3950) (3 months ago) [Ariz Zubair]
* 6f1a794 - Add queuestat changes for aggregate VOQ counters (sonic-net#3617) (3 months ago) [Vivek Verma]
* d86b2b6 - g[sfputil debug] Fix issue: do not check output status when CMIS version is lower than 5.0 (sonic-net#3938) (3 months ago) [Junchao-Mellanox]
* 252a643 - [SPM] Rename the variable tag to docker-image-reference (sonic-net#3998) (3 months ago) [DavidZagury]
```
#### How I did it
#### How to verify it
#### Description for the changelog

Signed-off-by: Feng Pan <fenpan@microsoft.com>
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.

3 participants