soc/intel/{cnl,icl}: Move globalnvs.asl/nvs.h into common/block/

This patch creates a common instance of globalnvs.asl/nvs.h inside intel common
code (soc/intel/common/block/) and ask cnl & icl soc code to refer globalnvs.asl
and nvs.h from common code block.

TEST=Able to build and boot Hatch and ICL DE system. Dump DSDT.asl to verify
GNVS operation region presence after booting to OS.

Change-Id: Ia9fb12a75557bd7dc38f6d22ba2b32065d18b3ee
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36457
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Michael Niewöhner
Reviewed-by: Angel Pons <th3fanbus@gmail.com>

2 files changed, 1 insertions, 84 deletions

diff --git a/src/soc/intel/icelake/acpi/globalnvs.asl b/src/soc/intel/icelake/acpi/globalnvs.asl
deleted file mode 100644
index 678ce5ac23..0000000000
--- a/src/soc/intel/icelake/acpi/globalnvs.asl
+++ /dev/null
@@ -1,55 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2018 Intel Corp.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-/* Global Variables */
-
-Name (\PICM, 0)		// IOAPIC/8259
-
-/*
- * Global ACPI memory region. This region is used for passing information
- * between coreboot (aka "the system bios"), ACPI, and the SMI handler.
- * Since we don't know where this will end up in memory at ACPI compile time,
- * we have to fix it up in coreboot's ACPI creation phase.
- */
-
-External (NVSA)
-
-OperationRegion (GNVS, SystemMemory, NVSA, 0x2000)
-Field (GNVS, ByteAcc, NoLock, Preserve)
-{
-	/* Miscellaneous */
-	Offset (0x00),
-	OSYS,	16,	// 0x00 - Operating System
-	SMIF,	8,	// 0x02 - SMI function
-	PCNT,	8,	// 0x03 - Processor Count
-	PPCM,	8,	// 0x04 - Max PPC State
-	TLVL,	8,	// 0x05 - Throttle Level Limit
-	LIDS,	8,	// 0x06 - LID State
-	PWRS,	8,	// 0x07 - AC Power State
-	CBMC,	32,	// 0x08 - 0x0b AC Power State
-	PM1I,	64,	// 0x0c - 0x13 PM1 wake status bit
-	GPEI,	64,	// 0x14 - 0x17 GPE wake status bit
-	DPTE,	8,	// 0x1c - Enable DPTF
-	NHLA,	64,	// 0x1d - 0x24 NHLT Address
-	NHLL,	32,	// 0x25 - 0x28 NHLT Length
-	CID1,	16,	// 0x29 - 0x2a Wifi Country Identifier
-	U2WE,	16,	// 0x2b - 0x2c USB2 Wake Enable Bitmap
-	U3WE,	16,	// 0x2d - 0x2e USB3 Wake Enable Bitmap
-	UIOR,	8,	// 0x2f - UART debug controller init on S3 resume
-
-	/* ChromeOS specific */
-	Offset (0x100),
-	#include <vendorcode/google/chromeos/acpi/gnvs.asl>
-}
diff --git a/src/soc/intel/icelake/include/soc/nvs.h b/src/soc/intel/icelake/include/soc/nvs.h
index b2d903a4f5..c855df0305 100644
--- a/src/soc/intel/icelake/include/soc/nvs.h
+++ b/src/soc/intel/icelake/include/soc/nvs.h
@@ -16,34 +16,6 @@
 #ifndef _SOC_NVS_H_
 #define _SOC_NVS_H_
 
-#include <commonlib/helpers.h>
-#include <vendorcode/google/chromeos/gnvs.h>
-
-typedef struct global_nvs_t {
-	/* Miscellaneous */
-	u16	osys; /* 0x00 - 0x01 Operating System */
-	u8	smif; /* 0x02 - SMI function call ("TRAP") */
-	u8      pcnt; /* 0x03 - Processor Count */
-	u8	ppcm; /* 0x04 - Max PPC State */
-	u8	tlvl; /* 0x05 - Throttle Level Limit */
-	u8	lids; /* 0x06 - LID State */
-	u8	pwrs; /* 0x07 - AC Power State */
-	u32	cbmc; /* 0x08 - 0xb AC Power State */
-	u64	pm1i; /* 0x0c - 0x13 PM1 wake status bit */
-	u64	gpei; /* 0x14 - 0x1b GPE wake status bit */
-	u8	dpte; /* 0x1c - Enable DPTF */
-	u64	nhla; /* 0x1d - 0x24 NHLT Address */
-	u32	nhll; /* 0x25 - 0x28 NHLT Length */
-	u16	cid1; /* 0x29 - 0x2a Wifi Country Identifier */
-	u16	u2we; /* 0x2b - 0x2c USB2 Wake Enable Bitmap */
-	u16	u3we; /* 0x2d - 0x2e USB3 Wake Enable Bitmap */
-	u8	uior; /* 0x2f - UART debug controller init on S3 resume */
-	u8	unused[208];
-
-	/* ChromeOS specific (0x100 - 0xfff) */
-	chromeos_acpi_t chromeos;
-}  __packed global_nvs_t;
-check_member(global_nvs_t, chromeos, GNVS_CHROMEOS_ACPI_OFFSET);
-
+#include <intelblocks/nvs.h>
 
 #endif