From 3136424e48b533a29258d5b2412439db066ff744 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Fri, 23 Jul 2021 19:18:02 +0200 Subject: soc/amd/common/block/acpimmio: add Kconfig option for biosram accessors The biosram accessor support in soc/amd/common/block/acpimmio/biosram.c is only used on Stoneyridge and the old amd/southbridge code and not on Picasso or Cezanne. It also only builds as a 32 bit binary and breaks when trying to build as a 64 bit binary, since the size of an uintptr_t is different on those two. There is no support for using the 32 bit binaryPI with a 64 bit coreboot while there is code to use a 32 bit FSP with 64 bit coreboot, so not building this for FSP-based SoC support moves us one step closer to be able to build coreboot as 64 bit binary for Picasso and Cezanne. Signed-off-by: Felix Held Change-Id: I2d87ec2fa1b217eaf55d865e4390308812502e56 Reviewed-on: https://review.coreboot.org/c/coreboot/+/56570 Tested-by: build bot (Jenkins) Reviewed-by: Marshall Dawson --- src/soc/amd/common/block/acpimmio/Kconfig | 10 ++++++++++ src/soc/amd/common/block/acpimmio/Makefile.inc | 4 ++-- 2 files changed, 12 insertions(+), 2 deletions(-) (limited to 'src/soc/amd/common/block') diff --git a/src/soc/amd/common/block/acpimmio/Kconfig b/src/soc/amd/common/block/acpimmio/Kconfig index d74a00bcb9..794ae3ed5e 100644 --- a/src/soc/amd/common/block/acpimmio/Kconfig +++ b/src/soc/amd/common/block/acpimmio/Kconfig @@ -3,3 +3,13 @@ config SOC_AMD_COMMON_BLOCK_ACPIMMIO help Select this option to enable hardware blocks in the AcpiMmio address space (0xfed8xxxx). + +if SOC_AMD_COMMON_BLOCK_ACPIMMIO + +config SOC_AMD_COMMON_BLOCK_ACPIMMIO_BIOSRAM + bool + help + Add functions to access settings stored in the biosram region. + This is only used by the SoCs using binaryPI and the old AGESA. + +endif # SOC_AMD_COMMON_BLOCK_ACPIMMIO diff --git a/src/soc/amd/common/block/acpimmio/Makefile.inc b/src/soc/amd/common/block/acpimmio/Makefile.inc index 6537d5625d..bbc52fbd3d 100644 --- a/src/soc/amd/common/block/acpimmio/Makefile.inc +++ b/src/soc/amd/common/block/acpimmio/Makefile.inc @@ -3,8 +3,8 @@ ifeq ($(CONFIG_SOC_AMD_COMMON_BLOCK_ACPIMMIO),y) all-y += mmio_util.c smm-y += mmio_util.c -all-y += biosram.c -smm-y += biosram.c +all-$(CONFIG_SOC_AMD_COMMON_BLOCK_ACPIMMIO_BIOSRAM) += biosram.c +smm-$(CONFIG_SOC_AMD_COMMON_BLOCK_ACPIMMIO_BIOSRAM) += biosram.c bootblock-y += print_reset_status.c -- cgit v1.2.3