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

[platform/cel]: add transceivershow tool #163

Open
wants to merge 1 commit into
base: 201811_cel_wb
Choose a base branch
from

Conversation

mudsut4ke
Copy link

@mudsut4ke mudsut4ke commented May 11, 2020

- What I did

  • Implement transceivershow tool

- How I did it

  • Implement function to display transceiver information
root@sonic:/home/admin# tranceivershow info
Port           Number of lanes  Vendor Name    Cable length[m]    Serial Number    Model Name        Nominal bit rate[100Mbs]    Rev
-----------  -----------------  -------------  -----------------  ---------------  ----------------  --------------------------  -----
Ethernet0                    4  ColorChip ltd  0                  18130506         C100QSFPCWDM400B  255                         00
Ethernet4                    4  INTEL CORP     0                  CRCL175003YTP    SPTSBP2CLCKS      255                         02
Ethernet8                    4  INNOLIGHT      0                  INIBI5602491     TR-FC13T-NFB      255                         1A
Ethernet12                   4  Amphenol       0                  APF164800178EC   07126189          255                         A
  • Implement function to display transceiver DOM data
root@sonic:/home/admin# tranceivershow dom
Port         Lane number    Temp[C]    Voltage[V]    Current[mA]    Tx Power[dBm]    Rx Power[dBm]
-----------  -------------  ---------  ------------  -------------  ---------------  ---------------
Ethernet0    Lane 1         39.0195    3.2764        44.83          -1.0791          -1.0791
             Lane 2         39.0195    3.2764        38.79          -1.4327          -1.4327
             Lane 3         39.0195    3.2764        35.134         -0.4287          -0.4287
             Lane 4         39.0195    3.2764        37.426         -1.5304          -1.5304
Ethernet4    Lane 1         46.8242    3.2694        60.95          0.3969           0.3969
             Lane 2         46.8242    3.2694        60.95          -0.2429          -0.2429
             Lane 3         46.8242    3.2694        60.95          -0.7099          -0.7099
             Lane 4         46.8242    3.2694        60.42          -0.2104          -0.2104
Ethernet8    Lane 1         42.7852    3.268         41.476         -2.9628          -2.9628
             Lane 2         42.7852    3.268         41.264         -3.6957          -3.6957
             Lane 3         42.7852    3.268         40.304         -2.6544          -2.6544
             Lane 4         42.7852    3.268         41.356         -3.9394          -3.9394

- How to verify it

Note: QSFP-DD driver on SONIC doesn't support DOM monitoring yet

@mudsut4ke mudsut4ke self-assigned this May 11, 2020
@mudsut4ke mudsut4ke marked this pull request as ready for review May 11, 2020 07:52
@tiantianlv
Copy link

Hi Wirut,

Can you add a common format to do it?, such as:
Usage:

show interfaces transceiver (eeprom [-d|--dom] | lpmode | presence) [<interface_name>]

@mudsut4ke
Copy link
Author

mudsut4ke commented May 19, 2020

Hi Wirut,

Can you add a common format to do it?, such as:
Usage:

show interfaces transceiver (eeprom [-d|--dom] | lpmode | presence) [<interface_name>]

@tiantianlv , Do you mean I should add this command to under show interfaces transceiver command ?

mudsut4ke pushed a commit that referenced this pull request Jan 25, 2021
pick up follow commits:

  > hwmon-lm75: backport support for PCT2075 thermal sensor (#165)
  > [arista] Reassign prefetch memory per platform (#163)
  > Add .gitignore file (#157)
  > Enable PCA9541 I2C mux module (#160)
mudsut4ke pushed a commit that referenced this pull request Mar 16, 2021
Submodule commits included:

* src/sonic-platform-common 6ad0004...bd4dc03 (1):
  > [sonic_sfp/qsfp_dd.py] Update DOM capability method name to align with other drivers (#163)

Also align all calling function names to match.
jerseyang pushed a commit that referenced this pull request Jun 4, 2021
Submodule commits included:

* src/sonic-platform-common 6ad0004...bd4dc03 (1):
  > [sonic_sfp/qsfp_dd.py] Update DOM capability method name to align with other drivers (#163)

Also align all calling function names to match.
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

Successfully merging this pull request may close these issues.

2 participants