MLK-11921-1 mxc IPUv3: common: Fix the macro idma_is_valid/idma_mask definitions
authorLiu Ying <Ying.Liu@freescale.com>
Mon, 30 Nov 2015 09:01:49 +0000 (17:01 +0800)
committerNitin Garg <nitin.garg@nxp.com>
Mon, 19 Mar 2018 19:49:20 +0000 (14:49 -0500)
This patch adds missing parentheses around the argument of the macro
idma_is_valid and idma_mask to avoid any potential macro expansion issue.

Signed-off-by: Liu Ying <Ying.Liu@freescale.com>
drivers/mxc/ipu3/ipu_common.c

index 605b38f..d472ad4 100644 (file)
@@ -131,8 +131,8 @@ static inline uint32_t tri_cur_buf_shift(uint32_t dma_chan)
        return ffs(mask) - 1;
 }
 
-#define idma_is_valid(ch)      (ch != NO_DMA)
-#define idma_mask(ch)          (idma_is_valid(ch) ? (1UL << (ch & 0x1F)) : 0)
+#define idma_is_valid(ch)      ((ch) != NO_DMA)
+#define idma_mask(ch)          (idma_is_valid(ch) ? (1UL << ((ch) & 0x1F)) : 0)
 
 static inline bool idma_is_set(struct ipu_soc *ipu, uint32_t reg, uint32_t dma)
 {