From 34c4f9e6b1eb2ce0c3756739270dbca565af25ba Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Benjamin=20Cab=C3=A9?= Date: Sat, 14 Jun 2025 21:13:58 +0200 Subject: [PATCH] drivers: i3c: cdns: fix bitwise operation in CONF_STATUS0_DEV_ROLE macro MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Updated the CONF_STATUS0_DEV_ROLE macro to ensure proper bitwise operation by adding parentheses around the bitmask operation. Signed-off-by: Benjamin Cabé --- drivers/i3c/i3c_cdns.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i3c/i3c_cdns.c b/drivers/i3c/i3c_cdns.c index c0e271da532..0914b14e81b 100644 --- a/drivers/i3c/i3c_cdns.c +++ b/drivers/i3c/i3c_cdns.c @@ -34,7 +34,7 @@ #define CONF_STATUS0_SUPPORTS_DDR BIT(5) #define CONF_STATUS0_SEC_MASTER BIT(4) /* And it was replaced with a Dev Role mask */ -#define CONF_STATUS0_DEV_ROLE(x) ((x) & GENMASK(5, 4) >> 4) +#define CONF_STATUS0_DEV_ROLE(x) (((x) & GENMASK(5, 4)) >> 4) #define CONF_STATUS0_DEV_ROLE_MAIN_MASTER 0 #define CONF_STATUS0_DEV_ROLE_SEC_MASTER 1 #define CONF_STATUS0_DEV_ROLE_SLAVE 2