xen/char: imx-lpuart: Fix MISRA C 2012 Rule 20.7 violation
authorXenia Ragiadakou <burzalodowa@gmail.com>
Tue, 2 Aug 2022 07:54:33 +0000 (10:54 +0300)
committerStefano Stabellini <stefano.stabellini@amd.com>
Thu, 4 Aug 2022 21:37:22 +0000 (14:37 -0700)
The macro parameter 'off' is used as an expression and it is good to be
enclosed in parentheses to prevent against unintended expansion.

Signed-off-by: Xenia Ragiadakou <burzalodowa@gmail.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
xen/drivers/char/imx-lpuart.c

index 2709136081671c9dd168c6590f55dc2904d354f3..9c1f3b71a387b8da5c2345d11f08425dacf3547f 100644 (file)
@@ -26,8 +26,8 @@
 #include <asm/imx-lpuart.h>
 #include <asm/io.h>
 
-#define imx_lpuart_read(uart, off)       readl((uart)->regs + off)
-#define imx_lpuart_write(uart, off, val) writel((val), (uart)->regs + off)
+#define imx_lpuart_read(uart, off)       readl((uart)->regs + (off))
+#define imx_lpuart_write(uart, off, val) writel((val), (uart)->regs + (off))
 
 static struct imx_lpuart {
     uint32_t baud, clock_hz, data_bits, parity, stop_bits, fifo_size;