Commit 0ed586d0 authored Jun 10, 2009 by Roel Kluin Committed by David S. Miller Jun 11, 2009

atl1c: WAKE_MCAST tested twice, not WAKE_UCAST

The WAKE_MCAST bit is tested twice, the first should be WAKE_UCAST.

Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Cc: Jie Yang <jie.yang@atheros.com>
Cc: Jay Cliburn <jcliburn@gmail.com>
Cc: Chris Snook <csnook@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent b77e5228

drivers/net/atl1c/atl1c_ethtool.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -271,7 +271,7 @@ static int atl1c_set_wol(struct net_device netdev, struct ethtool_wolinfo wol)
		struct atl1c_adapter *adapter = netdev_priv(netdev);

		if (wol->wolopts & (WAKE_ARP \| WAKE_MAGICSECURE \|
		WAKE_MCAST \| WAKE_BCAST \| WAKE_MCAST))
		WAKE_UCAST \| WAKE_BCAST \| WAKE_MCAST))
		return -EOPNOTSUPP;
		/* these settings will always override what we currently have */
		adapter->wol = 0;