Commit 0b50d753 authored by Ishizaki Kou's avatar Ishizaki Kou Committed by Jeff Garzik

spidernet: revise link status logging

This patch revises the logging for link informations of spidernet.

  - The link down message is too verbose because auto-negotiation timeout
    occurs periodically while an ethernet cable is not connected.
  - We want to see the link result, and we think it should be displayed.
Signed-off-by: default avatarKou Ishizaki <kou.ishizaki@toshiba.co.jp>
Acked-by: default avatarJens Osterkamp <jens@de.ibm.com>
Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
parent 9a11fcb5
...@@ -2045,7 +2045,8 @@ static void spider_net_link_phy(unsigned long data) ...@@ -2045,7 +2045,8 @@ static void spider_net_link_phy(unsigned long data)
/* if link didn't come up after SPIDER_NET_ANEG_TIMEOUT tries, setup phy again */ /* if link didn't come up after SPIDER_NET_ANEG_TIMEOUT tries, setup phy again */
if (card->aneg_count > SPIDER_NET_ANEG_TIMEOUT) { if (card->aneg_count > SPIDER_NET_ANEG_TIMEOUT) {
pr_info("%s: link is down trying to bring it up\n", card->netdev->name); pr_debug("%s: link is down trying to bring it up\n",
card->netdev->name);
switch (card->medium) { switch (card->medium) {
case BCM54XX_COPPER: case BCM54XX_COPPER:
...@@ -2096,9 +2097,10 @@ static void spider_net_link_phy(unsigned long data) ...@@ -2096,9 +2097,10 @@ static void spider_net_link_phy(unsigned long data)
card->aneg_count = 0; card->aneg_count = 0;
pr_debug("Found %s with %i Mbps, %s-duplex %sautoneg.\n", pr_info("%s: link up, %i Mbps, %s-duplex %sautoneg.\n",
phy->def->name, phy->speed, phy->duplex==1 ? "Full" : "Half", card->netdev->name, phy->speed,
phy->autoneg==1 ? "" : "no "); phy->duplex == 1 ? "Full" : "Half",
phy->autoneg == 1 ? "" : "no ");
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment