sparse says: drivers/net/wireless/atmel.c:1501:3: warning: Initializer entry defined twice drivers/net/wireless/atmel.c:1505:3: also defined here and it's correct; atmel has its own ndo_change_mtu and shouldn't use eth_change_mtu. Signed-off-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> Cc: Stephen Hemminger <shemminger@xxxxxxxxxx> --- It looks like you (davem) took the initial conversion, do you want this one as well? drivers/net/wireless/atmel.c | 1 - 1 file changed, 1 deletion(-) --- wireless-testing.orig/drivers/net/wireless/atmel.c 2009-04-21 11:01:20.000000000 +0200 +++ wireless-testing/drivers/net/wireless/atmel.c 2009-04-21 11:01:22.000000000 +0200 @@ -1500,11 +1500,10 @@ static const struct net_device_ops atmel .ndo_stop = atmel_close, .ndo_change_mtu = atmel_change_mtu, .ndo_set_mac_address = atmel_set_mac_address, .ndo_start_xmit = start_tx, .ndo_do_ioctl = atmel_ioctl, - .ndo_change_mtu = eth_change_mtu, .ndo_validate_addr = eth_validate_addr, }; struct net_device *init_atmel_card(unsigned short irq, unsigned long port, const AtmelFWType fw_type, -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html