diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2022-02-16 07:00:29 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-03-25 19:59:20 +0000 |
| commit | cdad992f0f3230bf27150290c47ecf72d20c121b (patch) | |
| tree | d2cc5cd6d4269e6acc6ad9582f67192f17a9c029 | |
| parent | 0dd03687402458b25a33c8b5b3dff241e13357a5 (diff) | |
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 <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/62172
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>
| -rw-r--r-- | src/northbridge/intel/gm45/pm.c | 2 |
1 files changed, 1 insertions, 1 deletions
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); |