Skip to contentSkip to conversation

Linux 2.5.72: ixgb_ethtool: strange SUPPORTED_10000baseT_Full

DISCUSSION Adrian Bunk 2 participants 3 messages Jun 17, 2003 Network drivers

lore
Share & tools
lore
#0 lore raw

<-- snip -->

...
CC drivers/net/ixgb/ixgb_ethtool.o
drivers/net/ixgb/ixgb_ethtool.c:53:1: warning: "SUPPORTED_10000baseT_Full" redefined In file included from drivers/net/ixgb/ixgb.h:60,
from drivers/net/ixgb/ixgb_ethtool.c:30:
include/linux/ethtool.h:302:1: warning: this is the location of the previous definition
...

<-- snip -->

ixgb_ethtool.c says:
#define SUPPORTED_10000baseT_Full (1 << 11)

ethtool.h says:
#define SUPPORTED_BNC (1 << 11)
#define SUPPORTED_10000baseT_Full (1 << 12)

The correct solution seems to be to remove the #define from ixgb_ethtool.c ?

cu
Adrian

Signature · 4 lines
       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed
#1 lore raw
Quoted 1 line · Adrian Bunk

The correct solution seems to be to remove the #define from ixgb_ethtool.c ?

correct.

wanna send a patch? ;-)

Jeff

#2 lore raw
Quoted 8 lines · Jeff Garzik
Quoted 2 lines · Adrian Bunk

The correct solution seems to be to remove the #define from ixgb_ethtool.c ?

correct.

wanna send a patch? ;-)

Yup. :-)

Compilation tested with 2.5.72.

drivers/net/ixgb/ixgb_ethtool.c renamed from drivers/net/ixgb/ixgb_ethtool.c.old +0 -1
@@ -50,7 +50,6 @@
 	return (IXGB_EEPROM_SIZE << 1);
 }
 
-#define SUPPORTED_10000baseT_Full (1 << 11)
 #define SPEED_10000 10000
 
 static void
Quoted 1 line

Jeff

cu
Adrian

Signature · 4 lines
       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed