netdev
[Top] [All Lists]

[e1000] print bus type/speed/width on probe

To: ganesh.venkatesan@xxxxxxxxx, netdev@xxxxxxxxxxx
Subject: [e1000] print bus type/speed/width on probe
From: Lennert Buytenhek <buytenh@xxxxxxxxxxxxxx>
Date: Wed, 24 Nov 2004 16:15:17 +0100
Cc: Robert Olsson <Robert.Olsson@xxxxxxxxxxx>, hadi@xxxxxxxxxx
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.1i
Hi,

The attached patch (based on a patch by Robert Olsson) makes e1000
display bus type/speed/width on probe.  All this info is already
determined by e1000_hw.  Output is as such (a la tg3):

eth1: e1000 (PCI:33MHz:32-bit) 00:0e:0c:64:cc:a1
eth2: e1000 (PCI:66MHz:32-bit) 00:04:23:46:e2:50

I've found it useful -- please consider applying.


cheers,
Lennert


Signed-off-by: Lennert Buytenhek <buytenh@xxxxxxxxxxxxxx>

diff -urN e1000.orig/e1000_main.c e1000/e1000_main.c
--- e1000.orig/e1000_main.c     2004-11-24 15:35:23.000000000 +0100
+++ e1000/e1000_main.c  2004-11-24 16:04:30.000000000 +0100
@@ -585,6 +585,21 @@
        if(eeprom_data & E1000_EEPROM_APME)
                adapter->wol |= E1000_WUFC_MAG;
 
+       /* print bus type/speed/width info */
+       printk(KERN_INFO "%s: e1000 (PCI%s:%s:%s) ", netdev->name,
+               ((adapter->hw.bus_type == e1000_bus_type_pcix) ? "X" : ""),
+               ((adapter->hw.bus_speed == e1000_bus_speed_133) ? "133MHz" :
+                (adapter->hw.bus_speed == e1000_bus_speed_120) ? "120MHz" :
+                (adapter->hw.bus_speed == e1000_bus_speed_100) ? "100MHz" :
+                (adapter->hw.bus_speed == e1000_bus_speed_66) ? "66MHz" :
+                 "33MHz"),
+               ((adapter->hw.bus_width == e1000_bus_width_64) ? "64-bit" :
+                 "32-bit"));
+
+       for (i = 0; i < 6; i++)
+               printk("%2.2x%c", netdev->dev_addr[i],
+                       i == 5 ? '\n' : ':');
+
        /* reset the hardware with the new settings */
        e1000_reset(adapter);
 

<Prev in Thread] Current Thread [Next in Thread>
  • [e1000] print bus type/speed/width on probe, Lennert Buytenhek <=