From 343d1ab48234b4ae153be35311973069748f0182 Mon Sep 17 00:00:00 2001 From: Wenyang Duan <wenyang.duan@gmail.com> Date: Mon, 25 Sep 2023 14:32:56 -0700 Subject: [PATCH] Put parentheses around macro argument. It might be too paranoid but makes it safe to deal with operators with lower priorities than '+', '<<' for example. --- kernel/uart.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/uart.c b/kernel/uart.c index e3b3b8a..02b3a77 100644 --- a/kernel/uart.c +++ b/kernel/uart.c @@ -13,7 +13,7 @@ // the UART control registers are memory-mapped // at address UART0. this macro returns the // address of one of the registers. -#define Reg(reg) ((volatile unsigned char *)(UART0 + reg)) +#define Reg(reg) ((volatile unsigned char *)(UART0 + (reg))) // the UART control registers. // some have different meanings for