From 5b9eca17cb981a08738e9d9ec3d6b91113daeb0e Mon Sep 17 00:00:00 2001 From: Xenia Ragiadakou Date: Tue, 2 Aug 2022 10:54:33 +0300 Subject: [PATCH] xen/char: imx-lpuart: Fix MISRA C 2012 Rule 20.7 violation 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 Reviewed-by: Stefano Stabellini --- xen/drivers/char/imx-lpuart.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/drivers/char/imx-lpuart.c b/xen/drivers/char/imx-lpuart.c index 2709136081..9c1f3b71a3 100644 --- a/xen/drivers/char/imx-lpuart.c +++ b/xen/drivers/char/imx-lpuart.c @@ -26,8 +26,8 @@ #include #include -#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; -- 2.30.2