From d81324eb74d880bcb434a74f337f9b08a0057c2f Mon Sep 17 00:00:00 2001 From: Patrick Rudolph Date: Thu, 21 Dec 2023 14:46:48 +0100 Subject: nb/intel/sandybridge/raminit: Drop write to BANDTIMERS_SNB MRC.bin doesn't write BANDTIMERS_SNB register, so drop the write. The bits written were targeting a reserved range, so assume it didn't do anything useful. Tested on Lenovo X220: Still boots and runs fine. Change-Id: I920aabd60831c791188af976914553787cc0ff18 Signed-off-by: Patrick Rudolph Reviewed-on: https://review.coreboot.org/c/coreboot/+/79760 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons Reviewed-by: Paul Menzel --- src/northbridge/intel/sandybridge/raminit_common.c | 1 - 1 file changed, 1 deletion(-) (limited to 'src/northbridge') diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c index f896541288..d6d545c10e 100644 --- a/src/northbridge/intel/sandybridge/raminit_common.c +++ b/src/northbridge/intel/sandybridge/raminit_common.c @@ -2865,7 +2865,6 @@ void final_registers(ramctr_timing *ctrl) mchbar_setbits32(MC_INIT_STATE_G, 1 << 0); mchbar_setbits32(MC_INIT_STATE_G, 1 << 7); - mchbar_write32(BANDTIMERS_SNB, 0xfa); /* Find a populated channel */ FOR_ALL_POPULATED_CHANNELS -- cgit v1.2.3