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 extra transceiver status flags for non-CMIS #2861

Open
longhuan-cisco opened this issue Jun 2, 2023 · 0 comments
Open

Add extra transceiver status flags for non-CMIS #2861

longhuan-cisco opened this issue Jun 2, 2023 · 0 comments

Comments

@longhuan-cisco
Copy link
Contributor

          > @longhuan-cisco can u check why Page 0h Byte 3 to 14 flags are missing For eg RX LOL, TX CDR LOL etc.

For CMIS/C-CMIS,
They are there in the CMIS output:
TX CDR LOL as Tx clock and data recovery loss of lock
RX CDR LOL as Rx clock and data recovery loss of lock

For 100G,
previously, there was no get_transceiver_status API implementation for non-CMIS at all. I was told to add get_transceiver_status API implementation for non-CMIS (including 100G) with only these four fields tx_disable/tx_disabled_channel/rx_los/tx_fault via sonic-net/sonic-platform-common#359. If we want to add more fields for 100G (in terms of, support in eeprom parsing and get_transceiver_status API/etc), we may better to track it as separate git issue/PR in another repo https://github.com/sonic-net/sonic-platform-common

Originally posted by @longhuan-cisco in #2772 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant