Change in coreboot[master]: soc/intel/braswell: Refactor acpi_init_gnvs()

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/42863 )

Change subject: soc/intel/braswell: Refactor acpi_init_gnvs()
......................................................................

soc/intel/braswell: Refactor acpi_init_gnvs()

Move GNVS details to different function, called
from acpi_create_gnvs().

Change-Id: Ief02c078fe37753c0d29418394a351105a1aacc8
Signed-off-by: Kyösti Mälkki 
---
M src/soc/intel/braswell/acpi.c
1 file changed, 6 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/63/42863/1

diff --git a/src/soc/intel/braswell/acpi.c b/src/soc/intel/braswell/acpi.c
index cb8b1b5..b0179ee 100644
--- a/src/soc/intel/braswell/acpi.c
+++ b/src/soc/intel/braswell/acpi.c
@@ -70,6 +70,12 @@

/* Top of Low Memory (start of resource allocation) */
    gnvs->tolm = nc_read_top_of_low_memory();
+
+	/* Fill in the Wi-Fi Region ID */
+	if (CONFIG(HAVE_REGULATORY_DOMAIN))
+		gnvs->cid1 = wifi_regulatory_domain();
+	else
+		gnvs->cid1 = WRDD_DEFAULT_REGULATORY_DOMAIN;
 }

static int acpi_sci_irq(void)
@@ -483,12 +489,6 @@
    if (gnvs) {
    	acpi_create_gnvs(gnvs);

-		/* Fill in the Wi-Fi Region ID */
-		if (CONFIG(HAVE_REGULATORY_DOMAIN))
-			gnvs->cid1 = wifi_regulatory_domain();
-		else
-			gnvs->cid1 = WRDD_DEFAULT_REGULATORY_DOMAIN;
-
    	/* Add it to DSDT */
    	acpigen_write_scope("\");
    	acpigen_write_name_dword("NVSA", (u32) gnvs);

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/42863 )

Change subject: soc/intel/braswell: Refactor acpi_init_gnvs()
......................................................................

Abandoned