From cdad992f0f3230bf27150290c47ecf72d20c121b Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Wed, 16 Feb 2022 07:00:29 +0100 Subject: [PATCH] nb/intel/gm45/pm.c: Make clang happy Clang complains that the terniary '?' operator is executed before the bitwise '|'. This is true and desired in this case. Being explicit about won't hurt however. Change-Id: I27d1fc1c19e1dab3d1c82e407151eaa46f8c7b03 Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/c/coreboot/+/62172 Tested-by: build bot (Jenkins) Reviewed-by: Elyes Haouas --- src/northbridge/intel/gm45/pm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/northbridge/intel/gm45/pm.c b/src/northbridge/intel/gm45/pm.c index f1b7abc89b..656a632639 100644 --- a/src/northbridge/intel/gm45/pm.c +++ b/src/northbridge/intel/gm45/pm.c @@ -171,7 +171,7 @@ void init_pm(const sysinfo_t *const sysinfo, int do_freq_scaling_cfg) mchbar_setbits32(0xf80, 1 << 31); mchbar_write32(0x40, (mchbar_read32(0x40) & ~(0x3f << 24)) | - (sysinfo->cores == 4) ? (1 << 24) : 0); + ((sysinfo->cores == 4) ? (1 << 24) : 0)); mchbar_clrbits32(0x40, 1 << 19); mchbar_setbits32(0x40, 1 << 13);