Inverted logic Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx> --- Usually the inverted occurs see: s="[[:space:]]*" em='!' git grep -E -n -A1 "if$s\($s$em?${s}netif_running$s\(" | grep -B1 "[-]EBUSY" | less Is my patch correct? please review. diff --git a/drivers/net/wireless/arlan-main.c b/drivers/net/wireless/arlan-main.c index d84caf1..4ac519f 100644 --- a/drivers/net/wireless/arlan-main.c +++ b/drivers/net/wireless/arlan-main.c @@ -1022,7 +1022,7 @@ static int arlan_mac_addr(struct net_device *dev, void *p) ARLAN_DEBUG_ENTRY("arlan_mac_addr"); return -EINVAL; - if (!netif_running(dev)) + if (netif_running(dev)) return -EBUSY; memcpy(dev->dev_addr, addr->sa_data, dev->addr_len); -- 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