sb/intel/common/rcba.h: Guard RCBAx macro parameters

Add brackets around the parameters to avoid operation order problems.

Change-Id: I689983b5b937f66b1a520eea61a38fb96c13c007
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50035
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Angel Pons 2021-01-28 13:40:02 +01:00 committed by Patrick Georgi
parent 7d66b39513
commit cf445ea89b
1 changed files with 4 additions and 4 deletions

View File

@ -11,10 +11,10 @@
#define RCBA 0xf0 #define RCBA 0xf0
#define RCBA_ENABLE 0x01 #define RCBA_ENABLE 0x01
#define RCBA8(x) (*((volatile u8 *)(DEFAULT_RCBA + x))) #define RCBA8(x) (*((volatile u8 *)(DEFAULT_RCBA + (x))))
#define RCBA16(x) (*((volatile u16 *)(DEFAULT_RCBA + x))) #define RCBA16(x) (*((volatile u16 *)(DEFAULT_RCBA + (x))))
#define RCBA32(x) (*((volatile u32 *)(DEFAULT_RCBA + x))) #define RCBA32(x) (*((volatile u32 *)(DEFAULT_RCBA + (x))))
#define RCBA64(x) (*((volatile u64 *)(DEFAULT_RCBA + x))) #define RCBA64(x) (*((volatile u64 *)(DEFAULT_RCBA + (x))))
#define RCBA_AND_OR(bits, x, and, or) \ #define RCBA_AND_OR(bits, x, and, or) \
(RCBA##bits(x) = ((RCBA##bits(x) & (and)) | (or))) (RCBA##bits(x) = ((RCBA##bits(x) & (and)) | (or)))