Skip to content
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

[Cables] Add support for 'Extended Specification Compliance' for QSFP cables #108

Merged
merged 2 commits into from
Aug 4, 2020
Merged

[Cables] Add support for 'Extended Specification Compliance' for QSFP cables #108

merged 2 commits into from
Aug 4, 2020

Conversation

shlomibitton
Copy link
Contributor

Signed-off-by: Shlomi Bitton shlomibi@mellanox.com

- Why I did it
To have this field in DB along with other cable information.

- How I did it
Parse 'Extended Specification Compliance' field from EEPROM and add it to DB, if exist.

- How to verify it
Run 'show interface transceiver eeprom Ethernet#'.

Shlomi Bitton added 2 commits July 29, 2020 14:06
@keboliu keboliu self-requested a review August 4, 2020 10:46
@jleveque jleveque merged commit 3ff137c into sonic-net:master Aug 4, 2020
abdosi pushed a commit that referenced this pull request Aug 9, 2020
… cables (#108)

Parse 'Extended Specification Compliance' field from EEPROM and add it to DB, if exist

Signed-off-by: Shlomi Bitton <shlomibi@mellanox.com>
@shlomibitton shlomibitton deleted the shlomi_ext_spec_com_qsfp branch March 25, 2021 08:12
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
…onfig (sonic-net#108)

Add check to make sure that the initializeGlobalConfig is invoked only in multi-asic platforms.

Additionaly remove the initializeGlobalConfig() call in the DomUpdate thread and SFPUpdate process. This is because initializeGlobalConfig() is already invoked and initialized in the parent Xcvrd daemon which is available to the child thread/process.
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.

4 participants