diff options
| author | Gal Pressman <galp@mellanox.com> | 2017-03-19 14:10:41 +0200 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2017-03-24 15:06:02 -0400 |
| commit | e33c8841f26090031d571fddd71dab06f56ab1bf (patch) | |
| tree | a51c8a54c7f568616d39c322a2a76f946a503b7a /sfpid.c | |
| parent | adbaa18b9bc108da442e773f5c10941d609e5688 (diff) | |
| download | ethtool-e33c8841f26090031d571fddd71dab06f56ab1bf.tar.gz | |
ethtool: Fix SFF 8079 cable technology bit parsing
According to the transceiver compliance code definition in the spec, bits
2 & 3 in the 8th byte are indication of active/passive cable, and not
specifically related to FC/copper.
Fixes: 2edf56749abe ("ethtool: Addition of -m option to dump module eeprom")
Signed-off-by: Gal Pressman <galp@mellanox.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'sfpid.c')
| -rw-r--r-- | sfpid.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -137,9 +137,9 @@ static void sff8079_show_transceiver(const __u8 *id) if (id[8] & (1 << 4)) printf("%s FC: Longwave laser (LL)\n", pfx); if (id[8] & (1 << 3)) - printf("%s FC: Copper Active\n", pfx); + printf("%s Active Cable\n", pfx); if (id[8] & (1 << 2)) - printf("%s FC: Copper Passive\n", pfx); + printf("%s Passive Cable\n", pfx); if (id[8] & (1 << 1)) printf("%s FC: Copper FC-BaseT\n", pfx); /* Fibre Channel transmission media */ |