net: fec: properly parenthesize macro args
authorLothar Waßmann <LW@KARO-electronics.de>
Mon, 17 Nov 2014 09:51:19 +0000 (10:51 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 18 Nov 2014 20:19:31 +0000 (15:19 -0500)
Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/fec.h

index e778b9e4f0a4622cf1e379d2d4497b37d004be97..27e75cfdc490aad908c41d37edf36298f778e2e2 100644 (file)
@@ -279,36 +279,37 @@ struct bufdesc_ex {
 #define FEC_ENET_MAX_TX_QS     3
 #define FEC_ENET_MAX_RX_QS     3
 
-#define FEC_R_DES_START(X)     ((X == 1) ? FEC_R_DES_START_1 : \
-                               ((X == 2) ? \
+#define FEC_R_DES_START(X)     (((X) == 1) ? FEC_R_DES_START_1 : \
+                               (((X) == 2) ? \
                                        FEC_R_DES_START_2 : FEC_R_DES_START_0))
-#define FEC_X_DES_START(X)     ((X == 1) ? FEC_X_DES_START_1 : \
-                               ((X == 2) ? \
+#define FEC_X_DES_START(X)     (((X) == 1) ? FEC_X_DES_START_1 : \
+                               (((X) == 2) ? \
                                        FEC_X_DES_START_2 : FEC_X_DES_START_0))
-#define FEC_R_DES_ACTIVE(X)    ((X == 1) ? FEC_R_DES_ACTIVE_1 : \
-                               ((X == 2) ? \
+#define FEC_R_DES_ACTIVE(X)    (((X) == 1) ? FEC_R_DES_ACTIVE_1 : \
+                               (((X) == 2) ? \
                                   FEC_R_DES_ACTIVE_2 : FEC_R_DES_ACTIVE_0))
-#define FEC_X_DES_ACTIVE(X)    ((X == 1) ? FEC_X_DES_ACTIVE_1 : \
-                               ((X == 2) ? \
+#define FEC_X_DES_ACTIVE(X)    (((X) == 1) ? FEC_X_DES_ACTIVE_1 : \
+                               (((X) == 2) ? \
                                   FEC_X_DES_ACTIVE_2 : FEC_X_DES_ACTIVE_0))
 
-#define FEC_DMA_CFG(X)         ((X == 2) ? FEC_DMA_CFG_2 : FEC_DMA_CFG_1)
+#define FEC_DMA_CFG(X)         (((X) == 2) ? FEC_DMA_CFG_2 : FEC_DMA_CFG_1)
 
 #define DMA_CLASS_EN           (1 << 16)
-#define FEC_RCMR(X)            ((X == 2) ? FEC_RCMR_2 : FEC_RCMR_1)
+#define FEC_RCMR(X)            (((X) == 2) ? FEC_RCMR_2 : FEC_RCMR_1)
 #define IDLE_SLOPE_MASK                0xffff
 #define IDLE_SLOPE_1           0x200 /* BW fraction: 0.5 */
 #define IDLE_SLOPE_2           0x200 /* BW fraction: 0.5 */
-#define IDLE_SLOPE(X)          ((X == 1) ? (IDLE_SLOPE_1 & IDLE_SLOPE_MASK) : \
+#define IDLE_SLOPE(X)          (((X) == 1) ?                           \
+                               (IDLE_SLOPE_1 & IDLE_SLOPE_MASK) :      \
                                (IDLE_SLOPE_2 & IDLE_SLOPE_MASK))
 #define RCMR_MATCHEN           (0x1 << 16)
-#define RCMR_CMP_CFG(v, n)     ((v & 0x7) <<  (n << 2))
+#define RCMR_CMP_CFG(v, n)     (((v) & 0x7) <<  (n << 2))
 #define RCMR_CMP_1             (RCMR_CMP_CFG(0, 0) | RCMR_CMP_CFG(1, 1) | \
                                RCMR_CMP_CFG(2, 2) | RCMR_CMP_CFG(3, 3))
 #define RCMR_CMP_2             (RCMR_CMP_CFG(4, 0) | RCMR_CMP_CFG(5, 1) | \
                                RCMR_CMP_CFG(6, 2) | RCMR_CMP_CFG(7, 3))
-#define RCMR_CMP(X)            ((X == 1) ? RCMR_CMP_1 : RCMR_CMP_2)
-#define FEC_TX_BD_FTYPE(X)     ((X & 0xf) << 20)
+#define RCMR_CMP(X)            (((X) == 1) ? RCMR_CMP_1 : RCMR_CMP_2)
+#define FEC_TX_BD_FTYPE(X)     (((X) & 0xf) << 20)
 
 /* The number of Tx and Rx buffers.  These are allocated from the page
  * pool.  The code may assume these are power of two, so it it best
@@ -359,7 +360,7 @@ struct bufdesc_ex {
 /* ENET interrupt coalescing macro define */
 #define FEC_ITR_CLK_SEL                (0x1 << 30)
 #define FEC_ITR_EN             (0x1 << 31)
-#define FEC_ITR_ICFT(X)                ((X & 0xff) << 20)
+#define FEC_ITR_ICFT(X)                (((X) & 0xff) << 20)
 #define FEC_ITR_ICTT(X)                ((X) & 0xffff)
 #define FEC_ITR_ICFT_DEFAULT   200  /* Set 200 frame count threshold */
 #define FEC_ITR_ICTT_DEFAULT   1000 /* Set 1000us timer threshold */