cxgb4: Report correct link speed for unsupported ones
authorHariprasad Shenai <hariprasad@chelsio.com>
Thu, 1 Oct 2015 08:18:48 +0000 (13:48 +0530)
committerDavid S. Miller <davem@davemloft.net>
Mon, 5 Oct 2015 10:48:41 +0000 (03:48 -0700)
When we get garbage from the firmware with weird Port Speeds,
etc. we should emit a warning regarding unsupported speeds rather than
use the bogus default of "10Mbps" which isn't even an option in the
firmware Port Information message

Signed-off-by: Hariprasad Shenai <hariprasad@chelsio.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c

index 0b7570ddbe1a7c52d78ec62b74f3ad21df5b6638..9f1f5b2aacf7a767310ca9c3a09441e8b7ff6cc9 100644 (file)
@@ -275,7 +275,7 @@ static void link_report(struct net_device *dev)
        else {
                static const char *fc[] = { "no", "Rx", "Tx", "Tx/Rx" };
 
-               const char *s = "10Mbps";
+               const char *s;
                const struct port_info *p = netdev_priv(dev);
 
                switch (p->link_cfg.speed) {
@@ -291,6 +291,10 @@ static void link_report(struct net_device *dev)
                case 40000:
                        s = "40Gbps";
                        break;
+               default:
+                       pr_info("%s: unsupported speed: %d\n",
+                               dev->name, p->link_cfg.speed);
+                       return;
                }
 
                netdev_info(dev, "link up, %s, full-duplex, %s PAUSE\n", s,