qlge: fix an "&&" vs "||" bug
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 12 Jul 2012 04:47:03 +0000 (04:47 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 Jul 2012 16:40:53 +0000 (09:40 -0700)
The condition is always true so WOL will never work.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Jitendra Kalsaria <jitendra.kalsaria@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/qlogic/qlge/qlge_ethtool.c

index 3d4462bd9de7c5922155bfc3f69ac78661067338..6f316ab23257dc7343776d92db05cc471561d681 100644 (file)
@@ -440,7 +440,7 @@ static int ql_set_wol(struct net_device *ndev, struct ethtool_wolinfo *wol)
        unsigned short ssys_dev = qdev->pdev->subsystem_device;
 
        /* WOL is only supported for mezz card. */
-       if (ssys_dev != QLGE_MEZZ_SSYS_ID_068 ||
+       if (ssys_dev != QLGE_MEZZ_SSYS_ID_068 &&
                        ssys_dev != QLGE_MEZZ_SSYS_ID_180) {
                netif_info(qdev, drv, qdev->ndev,
                                "WOL is only supported for mezz card\n");