From d3a65deb255be26bdfbdae3b4b3625e4e3183a13 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Fri, 19 Feb 2021 16:08:36 +0100 Subject: [PATCH] soc/intel: Guard macro parameters in pm.h Guard against unintended operator precedence and associativity issues. Change-Id: I342682a57fde9942cdf7be10756ee21c10af802a Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/50917 Reviewed-by: Furquan Shaikh Tested-by: build bot (Jenkins) --- src/soc/intel/apollolake/include/soc/pm.h | 4 ++-- src/soc/intel/broadwell/include/soc/pm.h | 4 ++-- src/soc/intel/skylake/include/soc/pm.h | 4 ++-- src/soc/intel/xeon_sp/include/soc/pm.h | 6 +++--- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/soc/intel/apollolake/include/soc/pm.h b/src/soc/intel/apollolake/include/soc/pm.h index 748f76adba..b40a12fce3 100644 --- a/src/soc/intel/apollolake/include/soc/pm.h +++ b/src/soc/intel/apollolake/include/soc/pm.h @@ -120,7 +120,7 @@ #define GPE0_REG_MAX 4 #define GPE0_REG_SIZE 32 -#define GPE0_STS(x) (0x20 + (x * 4)) +#define GPE0_STS(x) (0x20 + ((x) * 4)) #define GPE0_A 0 #define GPE0_B 1 #define GPE0_C 2 @@ -136,7 +136,7 @@ #define BATLOW_STS (1 << 10) #define PCIE_GPE_STS (1 << 9) #define SWGPE_STS (1 << 2) -#define GPE0_EN(x) (0x30 + (x * 4)) +#define GPE0_EN(x) (0x30 + ((x) * 4)) #define ESPI_EN (1 << 20) /* This bit is present in GLK */ #define SATA_PME_EN (1 << 17) #define SMB_WAK_EN (1 << 16) diff --git a/src/soc/intel/broadwell/include/soc/pm.h b/src/soc/intel/broadwell/include/soc/pm.h index 40467ed1dd..52fd364430 100644 --- a/src/soc/intel/broadwell/include/soc/pm.h +++ b/src/soc/intel/broadwell/include/soc/pm.h @@ -62,7 +62,7 @@ #define GPE0_REG_MAX 4 #define GPE0_REG_SIZE 32 -#define GPE0_STS(x) (0x80 + (x * 4)) +#define GPE0_STS(x) (0x80 + ((x) * 4)) #define GPE_31_0 0 /* 0x80/0x90 = GPE[31:0] */ #define GPE_63_32 1 /* 0x84/0x94 = GPE[63:32] */ #define GPE_94_64 2 /* 0x88/0x98 = GPE[94:64] */ @@ -78,7 +78,7 @@ #define TCOSCI_STS (1 << 6) #define SWGPE_STS (1 << 2) #define HOT_PLUG_STS (1 << 1) -#define GPE0_EN(x) (0x90 + (x * 4)) +#define GPE0_EN(x) (0x90 + ((x) * 4)) #define WADT_en (1 << 18) #define GP27_EN (1 << 16) #define PME_B0_EN (1 << 13) diff --git a/src/soc/intel/skylake/include/soc/pm.h b/src/soc/intel/skylake/include/soc/pm.h index 92f33b90b5..cd5c43e9b7 100644 --- a/src/soc/intel/skylake/include/soc/pm.h +++ b/src/soc/intel/skylake/include/soc/pm.h @@ -86,7 +86,7 @@ #define GPE0_REG_MAX 4 #define GPE0_REG_SIZE 32 -#define GPE0_STS(x) (0x80 + (x * 4)) +#define GPE0_STS(x) (0x80 + ((x) * 4)) #define GPE_31_0 0 /* 0x80/0x90 = GPE[31:0] */ #define GPE_63_32 1 /* 0x84/0x94 = GPE[63:32] */ #define GPE_95_64 2 /* 0x88/0x98 = GPE[95:64] */ @@ -105,7 +105,7 @@ #define TCOSCI_STS (1 << 6) #define SWGPE_STS (1 << 2) #define HOT_PLUG_STS (1 << 1) -#define GPE0_EN(x) (0x90 + (x * 4)) +#define GPE0_EN(x) (0x90 + ((x) * 4)) #define WADT_EN (1 << 18) #define LAN_WAK_EN (1 << 16) #define GPIO_T2_EN (1 << 15) diff --git a/src/soc/intel/xeon_sp/include/soc/pm.h b/src/soc/intel/xeon_sp/include/soc/pm.h index b699e1c26c..2673320664 100644 --- a/src/soc/intel/xeon_sp/include/soc/pm.h +++ b/src/soc/intel/xeon_sp/include/soc/pm.h @@ -66,8 +66,8 @@ #define PM2_CNT 0x50 #define GPE0_REG_MAX 4 #define GPE0_REG_SIZE 32 -#define GPE0_STS(x) (0x80 + (x * 4)) -#define GPE0_EN(x) (0x90 + (x * 4)) +#define GPE0_STS(x) (0x80 + ((x) * 4)) +#define GPE0_EN(x) (0x90 + ((x) * 4)) #define GPE_STD 3 /* 0x8c/0x9c = Standard GPE */ #define GPE_STS_RSVD GPE_STD #define GPIO_T2_STS (1 << 15) @@ -76,7 +76,7 @@ #define PCI_EXP_STS (1 << 9) #define SMB_WAK_STS (1 << 7) #define TCOSCI_STS (1 << 6) -#define GPE0_EN(x) (0x90 + (x * 4)) +#define GPE0_EN(x) (0x90 + ((x) * 4)) #define GPIO_T2_EN (1 << 15) #define ESPI_EN (1 << 14) #define PME_B0_EN (1 << 13)