From ef6278b0ff42d95faf7bc06eef7206ea8bc6faef Mon Sep 17 00:00:00 2001 From: fangxinyong Date: Tue, 19 Aug 2025 21:03:07 +0800 Subject: [PATCH] arch/arm: armv8-r irq type-safe bit ops Use 1u/~1u constants for TPIDRPRW bit operations. Avoid implicit conversion to a different underlying type (MISRA Rule 10.3). No functional change to task pointer tagging logic. Signed-off-by: fangxinyong --- arch/arm/include/armv8-r/irq.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/include/armv8-r/irq.h b/arch/arm/include/armv8-r/irq.h index 8d44efb6442..9244843c2ff 100644 --- a/arch/arm/include/armv8-r/irq.h +++ b/arch/arm/include/armv8-r/irq.h @@ -483,12 +483,12 @@ static inline_function uintptr_t up_getusrsp(void *regs) noinstrument_function static inline_function void up_set_interrupt_context(bool flag) { - CP15_MODIFY(flag, 1ul, TPIDRPRW); + CP15_MODIFY(flag, 1u, TPIDRPRW); } -#define up_this_task() ((struct tcb_s *)(CP15_GET(TPIDRPRW) & ~1ul)) -#define up_update_task(t) CP15_MODIFY(t, ~1ul, TPIDRPRW) -#define up_interrupt_context() (CP15_GET(TPIDRPRW) & 1) +#define up_this_task() ((struct tcb_s *)(CP15_GET(TPIDRPRW) & ~1u)) +#define up_update_task(t) CP15_MODIFY(t, ~1u, TPIDRPRW) +#define up_interrupt_context() (CP15_GET(TPIDRPRW) & 1u) /**************************************************************************** * Public Data