diff options
| author | Liu Ying <Ying.Liu@freescale.com> | 2015-11-30 17:01:49 +0800 |
|---|---|---|
| committer | Jason Liu <jason.hui.liu@nxp.com> | 2019-02-12 10:23:52 +0800 |
| commit | 467d1e5d79f7c61b22a7ebefec6a9e3cb0271a40 (patch) | |
| tree | 23c27ac69034247e864ebce76fc23f8c6537968c | |
| parent | 399e3954e92a94b563cbecd6fedb82b6389aea81 (diff) | |
MLK-11921-1 mxc IPUv3: common: Fix the macro idma_is_valid/idma_mask definitions
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>
| -rw-r--r-- | drivers/mxc/ipu3/ipu_common.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/mxc/ipu3/ipu_common.c b/drivers/mxc/ipu3/ipu_common.c index 749975f1ab4c..6d66ed5a311e 100644 --- a/drivers/mxc/ipu3/ipu_common.c +++ b/drivers/mxc/ipu3/ipu_common.c @@ -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) { |