diff options
author | Felix Held <felix-coreboot@felixheld.de> | 2022-02-23 17:54:20 +0100 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2022-02-25 17:42:45 +0000 |
commit | 4b2464fc90d60f01b0d890e1a0dc6dcdbd119617 (patch) | |
tree | d6b552cd62528e73c38a3bfbd5088feb5b7e2170 /src/soc/intel | |
parent | 46a3a044adfc8ec15faafd529e27c718754861c3 (diff) |
arch/x86: factor out and commonize HPET_BASE_ADDRESS definition
All x86 chipsets and SoCs have the HPET MMIO base address at 0xfed00000,
so define this once in arch/x86 and include this wherever needed. The
old AMD AGESA code in vendorcode that has its own definition is left
unchanged, but sb/amd/cimx/sb800/cfg.c is changed to use the new common
definition.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Ifc624051cc6c0f125fa154e826cfbeaf41b4de83
Reviewed-on: https://review.coreboot.org/c/coreboot/+/62304
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Diffstat (limited to 'src/soc/intel')
-rw-r--r-- | src/soc/intel/alderlake/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/apollolake/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/baytrail/include/soc/iomap.h | 3 | ||||
-rw-r--r-- | src/soc/intel/braswell/include/soc/iomap.h | 3 | ||||
-rw-r--r-- | src/soc/intel/broadwell/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/broadwell/pch/bootblock.c | 1 | ||||
-rw-r--r-- | src/soc/intel/cannonlake/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/common/block/acpi/acpi/lpc.asl | 1 | ||||
-rw-r--r-- | src/soc/intel/common/block/acpi/acpi/northbridge.asl | 1 | ||||
-rw-r--r-- | src/soc/intel/denverton_ns/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/elkhartlake/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/icelake/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/jasperlake/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/skylake/acpi/systemagent.asl | 1 | ||||
-rw-r--r-- | src/soc/intel/skylake/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/tigerlake/include/soc/iomap.h | 2 | ||||
-rw-r--r-- | src/soc/intel/xeon_sp/include/soc/iomap.h | 3 | ||||
-rw-r--r-- | src/soc/intel/xeon_sp/nb_acpi.c | 1 |
18 files changed, 5 insertions, 29 deletions
diff --git a/src/soc/intel/alderlake/include/soc/iomap.h b/src/soc/intel/alderlake/include/soc/iomap.h index 23b7c5bc94..9ff8f54d01 100644 --- a/src/soc/intel/alderlake/include/soc/iomap.h +++ b/src/soc/intel/alderlake/include/soc/iomap.h @@ -59,8 +59,6 @@ #define REG_BASE_ADDRESS 0xfb000000 #define REG_BASE_SIZE 0x1000 -#define HPET_BASE_ADDRESS 0xfed00000 - #define PCH_PWRM_BASE_ADDRESS 0xfe000000 #define PCH_PWRM_BASE_SIZE 0x10000 diff --git a/src/soc/intel/apollolake/include/soc/iomap.h b/src/soc/intel/apollolake/include/soc/iomap.h index e92227bc1d..209e1c2674 100644 --- a/src/soc/intel/apollolake/include/soc/iomap.h +++ b/src/soc/intel/apollolake/include/soc/iomap.h @@ -10,8 +10,6 @@ #define MCH_BASE_ADDRESS 0xfed10000 #define MCH_BASE_SIZE (32 * KiB) -#define HPET_BASE_ADDRESS 0xfed00000 - #define ACPI_BASE_ADDRESS 0x400 #define ACPI_BASE_SIZE 0x100 #define R_ACPI_PM1_TMR 0x8 diff --git a/src/soc/intel/baytrail/include/soc/iomap.h b/src/soc/intel/baytrail/include/soc/iomap.h index b5bea50007..c11a20aa19 100644 --- a/src/soc/intel/baytrail/include/soc/iomap.h +++ b/src/soc/intel/baytrail/include/soc/iomap.h @@ -42,9 +42,6 @@ #define RCBA_BASE_ADDRESS 0xfed1c000 #define RCBA_BASE_SIZE 0x400 -/* High Performance Event Timer */ -#define HPET_BASE_ADDRESS 0xfed00000 - /* Temporary Base Address */ #define TEMP_BASE_ADDRESS 0xfd000000 diff --git a/src/soc/intel/braswell/include/soc/iomap.h b/src/soc/intel/braswell/include/soc/iomap.h index 65fe4ac0bd..9220f695b5 100644 --- a/src/soc/intel/braswell/include/soc/iomap.h +++ b/src/soc/intel/braswell/include/soc/iomap.h @@ -43,9 +43,6 @@ #define RCBA_BASE_ADDRESS 0xfed1c000 #define RCBA_BASE_SIZE 0x400 -/* High Performance Event Timer */ -#define HPET_BASE_ADDRESS 0xfed00000 - /* Temporary Base Address */ #define TEMP_BASE_ADDRESS 0xfd000000 diff --git a/src/soc/intel/broadwell/include/soc/iomap.h b/src/soc/intel/broadwell/include/soc/iomap.h index 0f9f38f145..551f659ada 100644 --- a/src/soc/intel/broadwell/include/soc/iomap.h +++ b/src/soc/intel/broadwell/include/soc/iomap.h @@ -5,8 +5,6 @@ #include <northbridge/intel/haswell/memmap.h> -#define HPET_BASE_ADDRESS 0xfed00000 - #define ACPI_BASE_ADDRESS 0x1000 #define ACPI_BASE_SIZE 0x100 diff --git a/src/soc/intel/broadwell/pch/bootblock.c b/src/soc/intel/broadwell/pch/bootblock.c index 2f76111c5e..80c77212cf 100644 --- a/src/soc/intel/broadwell/pch/bootblock.c +++ b/src/soc/intel/broadwell/pch/bootblock.c @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: GPL-2.0-only */ #include <arch/bootblock.h> +#include <arch/hpet.h> #include <device/pci_ops.h> #include <soc/iomap.h> #include <soc/lpc.h> diff --git a/src/soc/intel/cannonlake/include/soc/iomap.h b/src/soc/intel/cannonlake/include/soc/iomap.h index e09cec27d6..a7149523ff 100644 --- a/src/soc/intel/cannonlake/include/soc/iomap.h +++ b/src/soc/intel/cannonlake/include/soc/iomap.h @@ -39,8 +39,6 @@ #define REG_BASE_ADDRESS 0xfc000000 #define REG_BASE_SIZE 0x1000 -#define HPET_BASE_ADDRESS 0xfed00000 - #define PCH_PWRM_BASE_ADDRESS 0xfe000000 #define PCH_PWRM_BASE_SIZE 0x10000 diff --git a/src/soc/intel/common/block/acpi/acpi/lpc.asl b/src/soc/intel/common/block/acpi/acpi/lpc.asl index 6c75a4756b..c4a7dd3aaf 100644 --- a/src/soc/intel/common/block/acpi/acpi/lpc.asl +++ b/src/soc/intel/common/block/acpi/acpi/lpc.asl @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: GPL-2.0-only */ /* Intel LPC/eSPI Bus Device - 0:1f.0 */ +#include <arch/hpet.h> #include <soc/iomap.h> Device (LPCB) diff --git a/src/soc/intel/common/block/acpi/acpi/northbridge.asl b/src/soc/intel/common/block/acpi/acpi/northbridge.asl index 89495a9e45..be14d180bb 100644 --- a/src/soc/intel/common/block/acpi/acpi/northbridge.asl +++ b/src/soc/intel/common/block/acpi/acpi/northbridge.asl @@ -1,5 +1,6 @@ /* SPDX-License-Identifier: GPL-2.0-or-later */ +#include <arch/hpet.h> #include <soc/iomap.h> Name (_HID, EisaId ("PNP0A08") /* PCI Express Bus */) // _HID: Hardware ID diff --git a/src/soc/intel/denverton_ns/include/soc/iomap.h b/src/soc/intel/denverton_ns/include/soc/iomap.h index ca14719079..827444a9c6 100644 --- a/src/soc/intel/denverton_ns/include/soc/iomap.h +++ b/src/soc/intel/denverton_ns/include/soc/iomap.h @@ -16,8 +16,6 @@ #define ACPI_BASE_ADDRESS DEFAULT_PMBASE #define DEFAULT_TCO_BASE 0x400 -#define HPET_BASE_ADDRESS 0xfed00000 - /* Southbridge internal device MEM BARs (Set to match FSP settings) */ #define DEFAULT_PCR_BASE 0xfd000000 #define DEFAULT_PWRM_BASE 0xfe000000 diff --git a/src/soc/intel/elkhartlake/include/soc/iomap.h b/src/soc/intel/elkhartlake/include/soc/iomap.h index dbbd2292ca..0e36c9faa8 100644 --- a/src/soc/intel/elkhartlake/include/soc/iomap.h +++ b/src/soc/intel/elkhartlake/include/soc/iomap.h @@ -30,8 +30,6 @@ #define REG_BASE_ADDRESS 0xfb000000 #define REG_BASE_SIZE 0x1000 -#define HPET_BASE_ADDRESS 0xfed00000 - #define PCH_PWRM_BASE_ADDRESS 0xfe000000 #define PCH_PWRM_BASE_SIZE 0x10000 diff --git a/src/soc/intel/icelake/include/soc/iomap.h b/src/soc/intel/icelake/include/soc/iomap.h index cee9411ae6..becb045b8b 100644 --- a/src/soc/intel/icelake/include/soc/iomap.h +++ b/src/soc/intel/icelake/include/soc/iomap.h @@ -30,8 +30,6 @@ #define REG_BASE_ADDRESS 0xfc000000 #define REG_BASE_SIZE 0x1000 -#define HPET_BASE_ADDRESS 0xfed00000 - #define PCH_PWRM_BASE_ADDRESS 0xfe000000 #define PCH_PWRM_BASE_SIZE 0x10000 diff --git a/src/soc/intel/jasperlake/include/soc/iomap.h b/src/soc/intel/jasperlake/include/soc/iomap.h index a6ac8367a3..88ccdcfe88 100644 --- a/src/soc/intel/jasperlake/include/soc/iomap.h +++ b/src/soc/intel/jasperlake/include/soc/iomap.h @@ -41,8 +41,6 @@ #define REG_BASE_ADDRESS 0xfb000000 #define REG_BASE_SIZE 0x1000 -#define HPET_BASE_ADDRESS 0xfed00000 - #define PCH_PWRM_BASE_ADDRESS 0xfe000000 #define PCH_PWRM_BASE_SIZE 0x10000 diff --git a/src/soc/intel/skylake/acpi/systemagent.asl b/src/soc/intel/skylake/acpi/systemagent.asl index 709e1f78c5..36d49327c6 100644 --- a/src/soc/intel/skylake/acpi/systemagent.asl +++ b/src/soc/intel/skylake/acpi/systemagent.asl @@ -1,5 +1,6 @@ /* SPDX-License-Identifier: GPL-2.0-only */ +#include <arch/hpet.h> #include <soc/iomap.h> Name (_HID, EisaId ("PNP0A08") /* PCI Express Bus */) // _HID: Hardware ID diff --git a/src/soc/intel/skylake/include/soc/iomap.h b/src/soc/intel/skylake/include/soc/iomap.h index d3fb9579fd..9245f4b6b0 100644 --- a/src/soc/intel/skylake/include/soc/iomap.h +++ b/src/soc/intel/skylake/include/soc/iomap.h @@ -35,8 +35,6 @@ #define VTVC0_BASE_ADDRESS 0xfed91000 #define VTVC0_BASE_SIZE 0x1000 -#define HPET_BASE_ADDRESS 0xfed00000 - #define PCH_PWRM_BASE_ADDRESS 0xfe000000 #define PCH_PWRM_BASE_SIZE 0x10000 diff --git a/src/soc/intel/tigerlake/include/soc/iomap.h b/src/soc/intel/tigerlake/include/soc/iomap.h index dec1950964..51f637a819 100644 --- a/src/soc/intel/tigerlake/include/soc/iomap.h +++ b/src/soc/intel/tigerlake/include/soc/iomap.h @@ -59,8 +59,6 @@ #define REG_BASE_ADDRESS 0xfb000000 #define REG_BASE_SIZE 0x4000 -#define HPET_BASE_ADDRESS 0xfed00000 - #define PCH_PWRM_BASE_ADDRESS 0xfe000000 #define PCH_PWRM_BASE_SIZE 0x10000 diff --git a/src/soc/intel/xeon_sp/include/soc/iomap.h b/src/soc/intel/xeon_sp/include/soc/iomap.h index ceaf271eb2..5daf62bf5c 100644 --- a/src/soc/intel/xeon_sp/include/soc/iomap.h +++ b/src/soc/intel/xeon_sp/include/soc/iomap.h @@ -26,9 +26,6 @@ #define VGA_BASE_ADDRESS 0xa0000 #define VGA_BASE_SIZE 0x20000 -/* High Performance Event Timer */ -#define HPET_BASE_ADDRESS 0xfed00000 - #define HECI1_BASE_ADDRESS 0xfed1a000 #define PCH_PWRM_BASE_ADDRESS CONFIG_INTEL_PCH_PWRM_BASE_ADDRESS diff --git a/src/soc/intel/xeon_sp/nb_acpi.c b/src/soc/intel/xeon_sp/nb_acpi.c index bb98f0b5fb..0519d4451b 100644 --- a/src/soc/intel/xeon_sp/nb_acpi.c +++ b/src/soc/intel/xeon_sp/nb_acpi.c @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: GPL-2.0-or-later */ #include <acpi/acpigen.h> +#include <arch/hpet.h> #include <assert.h> #include <cbmem.h> #include <device/mmio.h> |