diff options
author | Ryan Salsamendi <rsalsamendi@hotmail.com> | 2017-07-04 13:14:16 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-07-10 18:15:11 +0000 |
commit | 3f2fe18965fc5404e4d095a25dcb1be48e4040a5 (patch) | |
tree | f17cc962dad56d84ff9eb481564406e8fcced61d /src/southbridge/intel/lynxpoint/lpc.c | |
parent | 2fdf895867bc258690943475f0d2e8fc7daa6ee5 (diff) | |
download | coreboot-3f2fe18965fc5404e4d095a25dcb1be48e4040a5.tar.xz |
southbridge/intel/lynxpoint: Fix undefined behavior
Fix undefined behavior found by clang's -Wshift-sign-overflow, grep,
and source inspection. Left shifting an int where the right operand is
>= the width of the type is undefined. Add UL suffix since it's safe
for unsigned types.
Change-Id: I10db2566199200ceb3068721cfb35eadb2be1f68
Signed-off-by: Ryan Salsamendi <rsalsamendi@hotmail.com>
Reviewed-on: https://review.coreboot.org/20464
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/southbridge/intel/lynxpoint/lpc.c')
-rw-r--r-- | src/southbridge/intel/lynxpoint/lpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/southbridge/intel/lynxpoint/lpc.c b/src/southbridge/intel/lynxpoint/lpc.c index 513063edc4..ff78ea9eeb 100644 --- a/src/southbridge/intel/lynxpoint/lpc.c +++ b/src/southbridge/intel/lynxpoint/lpc.c @@ -475,7 +475,7 @@ static void enable_lp_clock_gating(device_t dev) reg32 &= ~(1 << 29); // LPC Dynamic else reg32 |= (1 << 29); // LPC Dynamic - reg32 |= (1 << 31); // LP LPC + reg32 |= (1UL << 31); // LP LPC reg32 |= (1 << 30); // LP BLA reg32 |= (1 << 28); // GPIO Dynamic reg32 |= (1 << 27); // HPET Dynamic |