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

Add support for tx power threshold warning/alarm and FlatMem #66

Merged
merged 2 commits into from
Nov 19, 2019

Conversation

stephenxs
Copy link
Collaborator

[sonic_platform_base/sonic_sfp/sff8436.py]
Add support for tx power threshold. Since to support tx power threshold requires more bytes to be read from eeprom and the file is shared by new platform api and plugin, the QSFP_CHANNL_THRESHOLD_WIDTH defined in sfputil.py has been updated as well.
Add support for flatmem which indicates whether the pages other than the first page is supported.

@qiluo-msft qiluo-msft requested a review from jleveque November 19, 2019 17:47
@jleveque jleveque merged commit d22f0a0 into sonic-net:master Nov 19, 2019
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
- Add support for QSFP-DD cable consist from 8 lanes.
- Add a DB entry for 'Application advertisement" for CMIS cable type.

Signed-off-by: Shlomi Bitton <[email protected]>
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
…ic-net#78)

Merge of QSFP-DD support via PR# sonic-net#66
is making xcvrd to bail out on platforms that do not support
new key/attribute.

Fix is to ignore new key/attribute if not present/supported yet.

Fix takes changes done as part of PR#sonic-net#76 which can be closed
once this gets merged

Verified manually xcvrd running fine after above changes.
also command show interface transceiver is showing correct o/p

Signed-off-by: Abhishek Dosi <[email protected]>
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