From 66fbeaec981707fa74e7296b0ef23d802dcfedea Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Fri, 6 May 2016 13:48:39 -0700 Subject: intel/pineview: Don't try to store 34 bits in 32 Mask out the bit that doesn't fit in 32bits, so gcc 6.1 is happy Change-Id: I13e2b41742206b8d86b90314b80cc324c00ae637 Signed-off-by: Stefan Reinauer Reviewed-on: https://review.coreboot.org/14639 Reviewed-by: Damien Zammit Tested-by: build bot (Jenkins) --- src/northbridge/intel/pineview/raminit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/northbridge/intel/pineview/raminit.c b/src/northbridge/intel/pineview/raminit.c index a94a2ebb97..e4957d2484 100644 --- a/src/northbridge/intel/pineview/raminit.c +++ b/src/northbridge/intel/pineview/raminit.c @@ -822,7 +822,7 @@ static void sdram_timings(struct sysinfo *s) MCHBAR8(0x26c) = (MCHBAR8(0x26c) & ~0xfa) | ((u8)(reg32 >> 24)); MCHBAR8(0x271) = MCHBAR8(0x271) & ~(1 << 7); MCHBAR8(0x274) = MCHBAR8(0x274) & ~0x6; - reg32 = (u32) ((6 << 30) | (4 << 25) | (1 << 20) | (8 << 15) | + reg32 = (u32) (((6 & 0x03) << 30) | (4 << 25) | (1 << 20) | (8 << 15) | (6 << 10) | (4 << 5) | 1); MCHBAR32(0x278) = reg32; -- cgit v1.2.3