sfc: Add parentheses around use of bitfield macro arguments
authorBen Hutchings <bhutchings@solarflare.com>
Tue, 11 Sep 2012 20:37:36 +0000 (21:37 +0100)
committerBen Hutchings <bhutchings@solarflare.com>
Tue, 2 Oct 2012 00:58:39 +0000 (01:58 +0100)
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
drivers/net/ethernet/sfc/bitfield.h

index b26a954c27fcba84764e4c4310e5dcb91dd17a61..5400a33f254f0d8727dd882036b033305eba0d9d 100644 (file)
@@ -120,10 +120,10 @@ typedef union efx_oword {
  * [0,high-low), with garbage in bits [high-low+1,...).
  */
 #define EFX_EXTRACT_NATIVE(native_element, min, max, low, high)                \
-       (((low > max) || (high < min)) ? 0 :                            \
-        ((low > min) ?                                                 \
-         ((native_element) >> (low - min)) :                           \
-         ((native_element) << (min - low))))
+       ((low) > (max) || (high) < (min) ? 0 :                          \
+        (low) > (min) ?                                                \
+        (native_element) >> ((low) - (min)) :                          \
+        (native_element) << ((min) - (low)))
 
 /*
  * Extract bit field portion [low,high) from the 64-bit little-endian
@@ -142,27 +142,27 @@ typedef union efx_oword {
 #define EFX_EXTRACT_OWORD64(oword, low, high)                          \
        ((EFX_EXTRACT64((oword).u64[0], 0, 63, low, high) |             \
          EFX_EXTRACT64((oword).u64[1], 64, 127, low, high)) &          \
-        EFX_MASK64(high + 1 - low))
+        EFX_MASK64((high) + 1 - (low)))
 
 #define EFX_EXTRACT_QWORD64(qword, low, high)                          \
        (EFX_EXTRACT64((qword).u64[0], 0, 63, low, high) &              \
-        EFX_MASK64(high + 1 - low))
+        EFX_MASK64((high) + 1 - (low)))
 
 #define EFX_EXTRACT_OWORD32(oword, low, high)                          \
        ((EFX_EXTRACT32((oword).u32[0], 0, 31, low, high) |             \
          EFX_EXTRACT32((oword).u32[1], 32, 63, low, high) |            \
          EFX_EXTRACT32((oword).u32[2], 64, 95, low, high) |            \
          EFX_EXTRACT32((oword).u32[3], 96, 127, low, high)) &          \
-        EFX_MASK32(high + 1 - low))
+        EFX_MASK32((high) + 1 - (low)))
 
 #define EFX_EXTRACT_QWORD32(qword, low, high)                          \
        ((EFX_EXTRACT32((qword).u32[0], 0, 31, low, high) |             \
          EFX_EXTRACT32((qword).u32[1], 32, 63, low, high)) &           \
-        EFX_MASK32(high + 1 - low))
+        EFX_MASK32((high) + 1 - (low)))
 
 #define EFX_EXTRACT_DWORD(dword, low, high)                    \
        (EFX_EXTRACT32((dword).u32[0], 0, 31, low, high) &      \
-        EFX_MASK32(high + 1 - low))
+        EFX_MASK32((high) + 1 - (low)))
 
 #define EFX_OWORD_FIELD64(oword, field)                                \
        EFX_EXTRACT_OWORD64(oword, EFX_LOW_BIT(field),          \
@@ -442,10 +442,10 @@ typedef union efx_oword {
        cpu_to_le32(EFX_INSERT_NATIVE(min, max, low, high, value))
 
 #define EFX_INPLACE_MASK64(min, max, low, high)                                \
-       EFX_INSERT64(min, max, low, high, EFX_MASK64(high + 1 - low))
+       EFX_INSERT64(min, max, low, high, EFX_MASK64((high) + 1 - (low)))
 
 #define EFX_INPLACE_MASK32(min, max, low, high)                                \
-       EFX_INSERT32(min, max, low, high, EFX_MASK32(high + 1 - low))
+       EFX_INSERT32(min, max, low, high, EFX_MASK32((high) + 1 - (low)))
 
 #define EFX_SET_OWORD64(oword, low, high, value) do {                  \
        (oword).u64[0] = (((oword).u64[0]                               \