From a1df15efc82a0ec03bd7353f5f51a2f288c01685 Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Fri, 8 Jan 2016 17:22:09 -0800 Subject: soc/intel/broadwell/lpc.c: don't zeroize existing gnvs table The gnvs table only needs to be zeroized after init; zeroizing an existing/populated table renders all I2C devices completely non-functional. TEST: boot Linux and observe all I2C devices functional Change-Id: Id149ad645dfe5ed999a65d10e786e17585abc477 Signed-off-by: Matt DeVillier Reviewed-on: https://review.coreboot.org/17828 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber --- src/soc/intel/broadwell/lpc.c | 1 - 1 file changed, 1 deletion(-) (limited to 'src/soc/intel/broadwell') diff --git a/src/soc/intel/broadwell/lpc.c b/src/soc/intel/broadwell/lpc.c index 850583b763..e4db498e30 100644 --- a/src/soc/intel/broadwell/lpc.c +++ b/src/soc/intel/broadwell/lpc.c @@ -599,7 +599,6 @@ static void southcluster_inject_dsdt(device_t device) } if (gnvs) { - memset(gnvs, 0, sizeof(*gnvs)); acpi_create_gnvs(gnvs); acpi_save_gnvs((unsigned long)gnvs); /* And tell SMI about it */ -- cgit v1.2.3