Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/58687 )
Change subject: soc/intel/denverton_ns: Fetch addr bits at runtime ......................................................................
soc/intel/denverton_ns: Fetch addr bits at runtime
Change-Id: Ic46a7d56cbaf45724ebc2a1911f5096af2fe461a Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/58687 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Mariusz Szafrański mariuszx.szafranski@intel.com Reviewed-by: Angel Pons th3fanbus@gmail.com --- M src/soc/intel/denverton_ns/acpi.c 1 file changed, 2 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Mariusz Szafrański: Looks good to me, approved Angel Pons: Looks good to me, approved
diff --git a/src/soc/intel/denverton_ns/acpi.c b/src/soc/intel/denverton_ns/acpi.c index 82b8cf1..ab6e7cc 100644 --- a/src/soc/intel/denverton_ns/acpi.c +++ b/src/soc/intel/denverton_ns/acpi.c @@ -4,6 +4,7 @@ #include <acpi/acpi_gnvs.h> #include <acpi/acpigen.h> #include <arch/smp/mpspec.h> +#include <cpu/cpu.h> #include <cpu/x86/smm.h> #include <string.h> #include <device/pci.h> @@ -60,7 +61,7 @@ gnvs->mmiob = (u32)get_top_of_low_memory(); gnvs->mmiol = (u32)(get_pciebase() - 1); gnvs->mmiohb = (u64)get_top_of_upper_memory(); - gnvs->mmiohl = (u64)(((u64)1 << CONFIG_CPU_ADDR_BITS) - 1); + gnvs->mmiohl = (u64)(((u64)1 << cpu_phys_address_size()) - 1); gnvs->tsegb = (u32)get_tseg_memory(); gnvs->tsegl = (u32)(get_top_of_low_memory() - get_tseg_memory()); }