Skip to content

[DELL][Z9100,S6100,S6000] Platform 2.0 SFP Changes#3229

Merged
lguohan merged 3 commits intosonic-net:masterfrom
sridhar-ravindran:Sfp_Platform_2_0
Aug 13, 2019
Merged

[DELL][Z9100,S6100,S6000] Platform 2.0 SFP Changes#3229
lguohan merged 3 commits intosonic-net:masterfrom
sridhar-ravindran:Sfp_Platform_2_0

Conversation

@sridhar-ravindran
Copy link
Copy Markdown
Contributor

- What I did
Added support for SFP APIs for platform 2.0
- How I did it
modified: platform/broadcom/sonic-platform-modules-dell/s6000/sonic_platform/init.py
modified: platform/broadcom/sonic-platform-modules-dell/s6000/sonic_platform/chassis.py
new file: platform/broadcom/sonic-platform-modules-dell/s6000/sonic_platform/sfp.py
modified: platform/broadcom/sonic-platform-modules-dell/s6100/sonic_platform/init.py
modified: platform/broadcom/sonic-platform-modules-dell/s6100/sonic_platform/chassis.py
new file: platform/broadcom/sonic-platform-modules-dell/s6100/sonic_platform/sfp.py
modified: platform/broadcom/sonic-platform-modules-dell/z9100/sonic_platform/init.py
modified: platform/broadcom/sonic-platform-modules-dell/z9100/sonic_platform/chassis.py
new file: platform/broadcom/sonic-platform-modules-dell/z9100/sonic_platform/sfp.py

Added support in sfp.py file which will be generic. Send the eeprom path and sfp_control path from chassis.py

- How to verify it
Added test show commands to invoke the 2.0 api and check the results.

- Description for the changelog

[DELL][Z9100,S6100,S6000] Platform 2.0 SFP Changes
- A picture of a cute animal (not mandatory but encouraged)

@sridhar-ravindran
Copy link
Copy Markdown
Contributor Author

@jleveque
Copy link
Copy Markdown
Contributor

Retest this please

@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Aug 7, 2019

retest broadcom please

@jleveque
Copy link
Copy Markdown
Contributor

jleveque commented Aug 7, 2019

@sridhar-ravindran: Please fix new conflicts

@sridhar-ravindran
Copy link
Copy Markdown
Contributor Author

@sridhar-ravindran: Please fix new conflicts

Hi @jleveque
I have synced to latest and resolved all conflicts.
-Sridhar.R

@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Aug 8, 2019

retest this please

@lguohan lguohan merged commit 88dc7ad into sonic-net:master Aug 13, 2019
@sridhar-ravindran sridhar-ravindran deleted the Sfp_Platform_2_0 branch August 14, 2019 05:02
wangshengjun pushed a commit to wangshengjun/sonic-buildimage that referenced this pull request Nov 16, 2020
* [DELL][Z9100,S6100,S6000] Platform 2.0 SFP Changes

Added support in sfp.py file which will be generic. Send the eeprom path and sfp_control path from chassis.py
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