tg3: ethtool phys_id default
authorStephen Hemminger <shemminger@vyatta.com>
Sun, 24 Feb 2008 03:51:59 +0000 (19:51 -0800)
committerDavid S. Miller <davem@davemloft.net>
Sun, 24 Feb 2008 03:51:59 +0000 (19:51 -0800)
When asked to blink LEDs the tg3 driver behaves when using:
ethtool -p ethX
The default value for data is zero, and other drivers interpret this
as blink forever (or at least a really long time).  The tg3 driver
interprets this as blink once.  All drivers should have the same
behaviour.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Michael Chan <mchan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/tg3.c

index db606b603884530aa0f433c56a4d080f0e0c6714..26ffb67f1da29ad7feec464a2a80c04f2d82f7bb 100644 (file)
@@ -8781,7 +8781,7 @@ static int tg3_phys_id(struct net_device *dev, u32 data)
                return -EAGAIN;
 
        if (data == 0)
                return -EAGAIN;
 
        if (data == 0)
-               data = 2;
+               data = UINT_MAX / 2;
 
        for (i = 0; i < (data * 2); i++) {
                if ((i % 2) == 0)
 
        for (i = 0; i < (data * 2); i++) {
                if ((i % 2) == 0)