summaryrefslogtreecommitdiffstats
path: root/src/soc/amd/sabrina
diff options
context:
space:
mode:
authorKarthikeyan Ramasubramanian <kramasub@google.com>2022-07-25 13:40:21 -0600
committerPaul Fagerburg <pfagerburg@chromium.org>2022-07-27 13:41:30 +0000
commit153f976fff93b44127a07cce9df04f3e13680212 (patch)
treec6aa87534c4dc4aee0041bc04c4a616c235e94f0 /src/soc/amd/sabrina
parentb5ff9b9f3f6387df800ceec2dd0daee1d02c8939 (diff)
downloadcoreboot-153f976fff93b44127a07cce9df04f3e13680212.tar.gz
coreboot-153f976fff93b44127a07cce9df04f3e13680212.tar.bz2
coreboot-153f976fff93b44127a07cce9df04f3e13680212.zip
soc/amd/sabrina: Disable CCP DMA and HW MODEXP
Enabling them causes firmware keyblock/preamble and/or body verification failure. Hence disabling them to use software based verification. Re-enable them once the issue is root-caused. BUG=b:217414563 TEST=Build and boot to OS in Skyrim with PSP and x86 verstage. Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com> Change-Id: I7e259ae5d790977d08afcb0a77f8d4f38c85f39e Reviewed-on: https://review.coreboot.org/c/coreboot/+/66134 Reviewed-by: Raul Rangel <rrangel@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd/sabrina')
-rw-r--r--src/soc/amd/sabrina/Kconfig1
-rw-r--r--src/soc/amd/sabrina/psp_verstage/svc.c8
2 files changed, 2 insertions, 7 deletions
diff --git a/src/soc/amd/sabrina/Kconfig b/src/soc/amd/sabrina/Kconfig
index b67d76977a60..ed739abcbab9 100644
--- a/src/soc/amd/sabrina/Kconfig
+++ b/src/soc/amd/sabrina/Kconfig
@@ -44,7 +44,6 @@ config SOC_SPECIFIC_OPTIONS
select PARALLEL_MP_AP_WORK
select PLATFORM_USES_FSP2_0
select PROVIDES_ROM_SHARING
- select PSP_VERSTAGE_CCP_DMA if VBOOT_STARTS_BEFORE_BOOTBLOCK
select PSP_SUPPORTS_EFS2_RELATIVE_ADDR if VBOOT_STARTS_BEFORE_BOOTBLOCK
select RESET_VECTOR_IN_RAM
select RTC
diff --git a/src/soc/amd/sabrina/psp_verstage/svc.c b/src/soc/amd/sabrina/psp_verstage/svc.c
index 021b344a57e7..5e26300f5661 100644
--- a/src/soc/amd/sabrina/psp_verstage/svc.c
+++ b/src/soc/amd/sabrina/psp_verstage/svc.c
@@ -174,12 +174,8 @@ uint32_t svc_crypto_sha(struct sha_generic_data *sha_op, enum sha_operation_mode
uint32_t svc_modexp(struct mod_exp_params *mod_exp_param)
{
- uint32_t retval = 0;
- struct cmd_param_modexp param = {
- .ptr_modexp = mod_exp_param,
- };
- SVC_CALL2(SVC_VERSTAGE_CMD, CMD_MODEXP, (void *)&param, retval);
- return retval;
+ /* TODO: b/240175446 Re-enable CCP DMA and svc_modexp later. */
+ return 1;
}
uint32_t svc_ccp_dma(uint32_t spi_rom_offset, void *dest, uint32_t size)