From 6b36c836266d7864f39e18381e58c8961a3fe491 Mon Sep 17 00:00:00 2001 From: Kangheui Won Date: Wed, 21 Apr 2021 14:48:14 +1000 Subject: soc/amd/picasso: clean up Kconfig and header Clean up Kconfig and psp_trasfer.h files before copying over to cezanne. TEST=build, flash and boot on jelboz360 Signed-off-by: Kangheui Won Change-Id: Ib370d93e23c15a2fe4c46051ed3647d2d067bb10 Reviewed-on: https://review.coreboot.org/c/coreboot/+/52563 Reviewed-by: Raul Rangel Tested-by: build bot (Jenkins) --- src/soc/amd/picasso/Kconfig | 14 ++------------ src/soc/amd/picasso/include/soc/psp_transfer.h | 4 ++-- 2 files changed, 4 insertions(+), 14 deletions(-) (limited to 'src/soc') diff --git a/src/soc/amd/picasso/Kconfig b/src/soc/amd/picasso/Kconfig index 1918cb734f..63fa0103ac 100644 --- a/src/soc/amd/picasso/Kconfig +++ b/src/soc/amd/picasso/Kconfig @@ -109,9 +109,9 @@ config PSP_SHAREDMEM_BASE default 0x0 help This variable defines the base address in DRAM memory where PSP copies - vboot workbuf to. This is used in linker script to have a static + the vboot workbuf. This is used in the linker script to have a static allocation for the buffer as well as for adding relevant entries in - BIOS directory table for the PSP. + the BIOS directory table for the PSP. config PSP_SHAREDMEM_SIZE hex @@ -405,16 +405,6 @@ config PSP_WHITELIST_FILE depends on HAVE_PSP_WHITELIST_FILE default "3rdparty/amd_blobs/picasso/PSP/wtl-rvn.sbin" -config PSP_SHAREDMEM_SIZE - hex "Maximum size of shared memory area" - default 0x3000 if VBOOT - default 0x0 - help - Sets the maximum size for the PSP to pass the vboot workbuf and - any logs or timestamps back to coreboot. This will be copied - into main memory by the PSP and will be available when the x86 is - started. - config PSP_UNLOCK_SECURE_DEBUG bool "Unlock secure debug" default n diff --git a/src/soc/amd/picasso/include/soc/psp_transfer.h b/src/soc/amd/picasso/include/soc/psp_transfer.h index 51c9eacb57..b5dffe7b39 100644 --- a/src/soc/amd/picasso/include/soc/psp_transfer.h +++ b/src/soc/amd/picasso/include/soc/psp_transfer.h @@ -9,7 +9,7 @@ # error "Must set CONFIG_CMOS_RECOVERY_BYTE" # endif -#define CMOS_RECOVERY_MAGIC_VAL 0x96 +#define CMOS_RECOVERY_MAGIC_VAL 0x96 #define TRANSFER_INFO_SIZE 64 #define TIMESTAMP_BUFFER_SIZE 0x200 @@ -44,7 +44,7 @@ struct transfer_info_struct { uint32_t psp_info; /* Offset 0x3C */ }; -_Static_assert(sizeof(struct transfer_info_struct) == TRANSFER_INFO_SIZE, \ +_Static_assert(sizeof(struct transfer_info_struct) == TRANSFER_INFO_SIZE, "TRANSFER_INFO_SIZE is incorrect"); /* Make sure the PSP transferred information over to x86 side. */ -- cgit v1.2.3