Message ID | 1489925441-20098-1-git-send-email-galp@mellanox.com |
---|---|
State | Accepted, archived |
Delegated to: | John Linville |
Headers | show |
diff --git a/sfpid.c b/sfpid.c index fd6415c..1732e5e 100644 --- a/sfpid.c +++ b/sfpid.c @@ -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 */
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> --- sfpid.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)