diff options
author | Ishizaki Kou <kou.ishizaki@toshiba.co.jp> | 2008-04-10 23:33:53 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-04-16 20:06:50 -0400 |
commit | 0b50d753874ad4843d305bf841ba5e28fc0f0ce7 (patch) | |
tree | 54b398fe4a2751f35467c7dbd9e928947e623b17 /drivers | |
parent | 9a11fcb5215d6ecade9aca1f1fba272746a3882d (diff) |
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: Kou Ishizaki <kou.ishizaki@toshiba.co.jp>
Acked-by: Jens Osterkamp <jens@de.ibm.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/spider_net.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/net/spider_net.c b/drivers/net/spider_net.c index e8b3b0802e73..477671606273 100644 --- a/drivers/net/spider_net.c +++ b/drivers/net/spider_net.c | |||
@@ -2045,7 +2045,8 @@ static void spider_net_link_phy(unsigned long data) | |||
2045 | /* if link didn't come up after SPIDER_NET_ANEG_TIMEOUT tries, setup phy again */ | 2045 | /* if link didn't come up after SPIDER_NET_ANEG_TIMEOUT tries, setup phy again */ |
2046 | if (card->aneg_count > SPIDER_NET_ANEG_TIMEOUT) { | 2046 | if (card->aneg_count > SPIDER_NET_ANEG_TIMEOUT) { |
2047 | 2047 | ||
2048 | pr_info("%s: link is down trying to bring it up\n", card->netdev->name); | 2048 | pr_debug("%s: link is down trying to bring it up\n", |
2049 | card->netdev->name); | ||
2049 | 2050 | ||
2050 | switch (card->medium) { | 2051 | switch (card->medium) { |
2051 | case BCM54XX_COPPER: | 2052 | case BCM54XX_COPPER: |
@@ -2096,9 +2097,10 @@ static void spider_net_link_phy(unsigned long data) | |||
2096 | 2097 | ||
2097 | card->aneg_count = 0; | 2098 | card->aneg_count = 0; |
2098 | 2099 | ||
2099 | pr_debug("Found %s with %i Mbps, %s-duplex %sautoneg.\n", | 2100 | pr_info("%s: link up, %i Mbps, %s-duplex %sautoneg.\n", |
2100 | phy->def->name, phy->speed, phy->duplex==1 ? "Full" : "Half", | 2101 | card->netdev->name, phy->speed, |
2101 | phy->autoneg==1 ? "" : "no "); | 2102 | phy->duplex == 1 ? "Full" : "Half", |
2103 | phy->autoneg == 1 ? "" : "no "); | ||
2102 | 2104 | ||
2103 | return; | 2105 | return; |
2104 | } | 2106 | } |