ethernet: amd: use IS_ENABLED() instead of checking for built-in or module
authorJavier Martinez Canillas <javier@osg.samsung.com>
Mon, 12 Sep 2016 14:03:34 +0000 (10:03 -0400)
committerDavid S. Miller <davem@davemloft.net>
Tue, 13 Sep 2016 03:27:58 +0000 (20:27 -0700)
The IS_ENABLED() macro checks if a Kconfig symbol has been enabled either
built-in or as a module, use that macro instead of open coding the same.

Using the macro makes the code more readable by helping abstract away some
of the Kconfig built-in and module enable details.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/amd/7990.c
drivers/net/ethernet/amd/amd8111e.c

index dcf2a1f3643d79aa3bf7971c5dda37c22561a27a..dc57f2759f44c551a2fb9c4b52b36eb59335de50 100644 (file)
 #define WRITERDP(lp, x)        out_be16(lp->base + LANCE_RDP, (x))
 #define READRDP(lp)    in_be16(lp->base + LANCE_RDP)
 
-#if defined(CONFIG_HPLANCE) || defined(CONFIG_HPLANCE_MODULE)
+#if IS_ENABLED(CONFIG_HPLANCE)
 #include "hplance.h"
 
 #undef WRITERAP
 #undef WRITERDP
 #undef READRDP
 
-#if defined(CONFIG_MVME147_NET) || defined(CONFIG_MVME147_NET_MODULE)
+#if IS_ENABLED(CONFIG_MVME147_NET)
 
 /* Lossage Factor Nine, Mr Sulu. */
 #define WRITERAP(lp, x)        (lp->writerap(lp, x))
@@ -86,7 +86,7 @@ static inline __u16 READRDP(struct lance_private *lp)
 }
 
 #endif
-#endif /* CONFIG_HPLANCE || CONFIG_HPLANCE_MODULE */
+#endif /* IS_ENABLED(CONFIG_HPLANCE) */
 
 /* debugging output macros, various flavours */
 /* #define TEST_HITS */
index 94960055fa1f802fcee1debd7584b4b1e7ed0cd3..f92cc97151ecf6180a53cfd6f5da6a1fe6f6fe39 100644 (file)
@@ -89,7 +89,7 @@ Revision History:
 #include <asm/byteorder.h>
 #include <asm/uaccess.h>
 
-#if defined(CONFIG_VLAN_8021Q) || defined(CONFIG_VLAN_8021Q_MODULE)
+#if IS_ENABLED(CONFIG_VLAN_8021Q)
 #define AMD8111E_VLAN_TAG_USED 1
 #else
 #define AMD8111E_VLAN_TAG_USED 0