From 983608daa5b75538c1e11cfad67d5badff73dafa Mon Sep 17 00:00:00 2001 From: Ben Gardner Date: Fri, 11 Mar 2016 14:50:35 -0600 Subject: [PATCH] intel/fsp_baytrail: Don't clear gnvs in acpi_init_gnvs() That wipes out all previously stored settings and breaks running devices in ACPI mode. This more closely matches what is done in intel/baytrail. Change-Id: Ie993c9f9e1eceb73d016d2df72770a27abb26ec1 Signed-off-by: Ben Gardner Reviewed-on: https://review.coreboot.org/14040 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/soc/intel/fsp_baytrail/acpi.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/soc/intel/fsp_baytrail/acpi.c b/src/soc/intel/fsp_baytrail/acpi.c index e55709cb49..015681652e 100644 --- a/src/soc/intel/fsp_baytrail/acpi.c +++ b/src/soc/intel/fsp_baytrail/acpi.c @@ -85,9 +85,6 @@ static acpi_cstate_t cstate_map[] = { void acpi_init_gnvs(global_nvs_t *gnvs) { - /* Clear gnvs area so uninitialized portions are defined */ - memset(gnvs, 0, sizeof(*gnvs)); - /* CPU core count */ gnvs->pcnt = dev_count_cpu();