soc/amd: select RESET_VECTOR_IN_RAM from common non-CAR Kconfig
Select RESET_VECTOR_IN_RAM from the common SOC_AMD_COMMON_BLOCK_NONCAR Kconfig option instead of selecting it in each AMD SoC's Kconfig which selects SOC_AMD_COMMON_BLOCK_NONCAR. From family 17h on, the AMD SoCs don't use cache as RAM (CAR) any more. In most cases, including the coreboot case, the PSP puts coreboot's bootblock into DRAM, thus RESET_VECTOR_IN_RAM needs to be selected. There might be a case where the RESET_VECTOR_IN_RAM part isn't true, but that isn't specific to a SoC generation, so even this unlikely case doesn't prevent us from moving the selection of the Kconfig option to the common non-CAR Kconfig option. Change-Id: I87d7908f94505647f504f9d214e3c52f9c3a3715 Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/87322 Reviewed-by: Patrick Rudolph <patrick.rudolph@9elements.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
parent
deda4ee304
commit
321f30d89e
7 changed files with 1 additions and 6 deletions
|
|
@ -23,7 +23,6 @@ config SOC_AMD_CEZANNE
|
|||
select PROVIDES_ROM_SHARING
|
||||
select PSP_VERSTAGE_CCP_DMA if VBOOT_STARTS_BEFORE_BOOTBLOCK
|
||||
select PSP_VERSTAGE_MAP_ENTIRE_SPIROM if VBOOT_STARTS_BEFORE_BOOTBLOCK
|
||||
select RESET_VECTOR_IN_RAM
|
||||
select RTC
|
||||
select SOC_AMD_COMMON
|
||||
select SOC_AMD_COMMON_BLOCK_ACP_GEN1
|
||||
|
|
|
|||
|
|
@ -16,6 +16,7 @@ config SOC_AMD_COMMON_BLOCK_CAR
|
|||
config SOC_AMD_COMMON_BLOCK_NONCAR
|
||||
bool
|
||||
select RESERVED_PHYSICAL_ADDRESS_BITS_SUPPORT
|
||||
select RESET_VECTOR_IN_RAM
|
||||
help
|
||||
From family 17h on AMD CPUs/APUs don't use cache as RAM (CAR) any
|
||||
more, since the RAM initialization is already done by the PSP when
|
||||
|
|
|
|||
|
|
@ -13,7 +13,6 @@ config SOC_SPECIFIC_OPTIONS
|
|||
select HAVE_ACPI_TABLES
|
||||
select HAVE_X86_64_SUPPORT
|
||||
select HAVE_SMI_HANDLER
|
||||
select RESET_VECTOR_IN_RAM
|
||||
select SOC_AMD_COMMON
|
||||
select SOC_AMD_COMMON_BLOCK_ACPI
|
||||
select SOC_AMD_COMMON_BLOCK_ACPIMMIO
|
||||
|
|
|
|||
|
|
@ -25,7 +25,6 @@ config SOC_AMD_GLINDA
|
|||
select PROVIDES_ROM_SHARING
|
||||
select PSP_SUPPORTS_EFS2_RELATIVE_ADDR if VBOOT_STARTS_BEFORE_BOOTBLOCK
|
||||
select PSP_VERSTAGE_CCP_DMA if VBOOT_STARTS_BEFORE_BOOTBLOCK
|
||||
select RESET_VECTOR_IN_RAM
|
||||
select RTC
|
||||
select SOC_AMD_COMMON
|
||||
select SOC_AMD_COMMON_BLOCK_ACP_GEN2 # TODO: Check if this is still correct
|
||||
|
|
|
|||
|
|
@ -25,7 +25,6 @@ config SOC_AMD_REMBRANDT_BASE
|
|||
select PSP_INCLUDES_HSP
|
||||
select PSP_SUPPORTS_EFS2_RELATIVE_ADDR if VBOOT_STARTS_BEFORE_BOOTBLOCK
|
||||
select PSP_VERSTAGE_CCP_DMA if VBOOT_STARTS_BEFORE_BOOTBLOCK
|
||||
select RESET_VECTOR_IN_RAM
|
||||
select RTC
|
||||
select SOC_AMD_COMMON
|
||||
select SOC_AMD_COMMON_BLOCK_ACP_GEN2
|
||||
|
|
|
|||
|
|
@ -22,7 +22,6 @@ config SOC_AMD_PHOENIX_BASE
|
|||
select PSP_SUPPORTS_EFS2_RELATIVE_ADDR if VBOOT_STARTS_BEFORE_BOOTBLOCK
|
||||
# TODO: (b/303516266) Re-enable CCP DMA after addressing a stall
|
||||
# select PSP_VERSTAGE_CCP_DMA if VBOOT_STARTS_BEFORE_BOOTBLOCK
|
||||
select RESET_VECTOR_IN_RAM
|
||||
select RTC
|
||||
select SOC_AMD_COMMON
|
||||
select SOC_AMD_COMMON_BLOCK_ACP_GEN2
|
||||
|
|
|
|||
|
|
@ -19,7 +19,6 @@ config SOC_AMD_PICASSO
|
|||
select PARALLEL_MP_AP_WORK
|
||||
select PLATFORM_USES_FSP2_0
|
||||
select PROVIDES_ROM_SHARING
|
||||
select RESET_VECTOR_IN_RAM
|
||||
select RTC
|
||||
select SOC_AMD_COMMON
|
||||
select SOC_AMD_COMMON_BLOCK_ACP_GEN1
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue