summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorBen Gardner <gardner.ben@gmail.com>2016-03-11 14:50:35 -0600
committerMartin Roth <martinroth@google.com>2016-03-13 03:06:37 +0100
commit983608daa5b75538c1e11cfad67d5badff73dafa (patch)
tree5d4b728cdce08c64259086b043bcaa1ffc9df1c7 /src
parent10008107b7786c3c8b9a88d5b24a9b08e2896b47 (diff)
downloadcoreboot-983608daa5b75538c1e11cfad67d5badff73dafa.tar.xz
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 <gardner.ben@gmail.com> Reviewed-on: https://review.coreboot.org/14040 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src')
-rw-r--r--src/soc/intel/fsp_baytrail/acpi.c3
1 files changed, 0 insertions, 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();