diff options
| author | Matt DeVillier <matt.devillier@gmail.com> | 2016-01-08 17:22:09 -0800 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2016-12-14 12:10:25 +0100 |
| commit | a1df15efc82a0ec03bd7353f5f51a2f288c01685 (patch) | |
| tree | 445aef6928a15074e9c5a6254b694598c2c0bea2 /src/soc/intel/broadwell/lpc.c | |
| parent | 93eb8c48b654ab100491822212238909fa1a2962 (diff) | |
| download | coreboot-a1df15efc82a0ec03bd7353f5f51a2f288c01685.tar.xz | |
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 <matt.devillier@gmail.com>
Reviewed-on: https://review.coreboot.org/17828
Tested-by: build bot (Jenkins)
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/soc/intel/broadwell/lpc.c')
| -rw-r--r-- | src/soc/intel/broadwell/lpc.c | 1 |
1 files changed, 0 insertions, 1 deletions
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 */ |