soc/amd/common/vboot: Split transfer buffer methods into separate file
I want to reuse the transfer buffer methods in SMM, so I need to add
them into their own file. I renamed `setup_cbmem_console` to
`replay_transfer_buffer_cbmemc` so it has a more descriptive name. I
also fixed the comment on `verify_psp_transfer_buf`.
BUG=b:221231786
TEST=Boot guybrush to OS
Signed-off-by: Raul E Rangel <rrangel@chromium.org>
Change-Id: I4f3a8b414b91f601c3a9c3dc7af8f388286fe4da
Reviewed-on: https://review.coreboot.org/c/coreboot/+/62348
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
diff --git a/src/soc/amd/common/vboot/Makefile.inc b/src/soc/amd/common/vboot/Makefile.inc
index 8a6f116..6938d16 100644
--- a/src/soc/amd/common/vboot/Makefile.inc
+++ b/src/soc/amd/common/vboot/Makefile.inc
@@ -6,3 +6,4 @@
endif
bootblock-$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK) += vboot_bootblock.c
+bootblock-$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK) += transfer_buffer.c
diff --git a/src/soc/amd/common/vboot/transfer_buffer.c b/src/soc/amd/common/vboot/transfer_buffer.c
new file mode 100644
index 0000000..06e564a
--- /dev/null
+++ b/src/soc/amd/common/vboot/transfer_buffer.c
@@ -0,0 +1,87 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#include <amdblocks/reset.h>
+#include <console/cbmem_console.h>
+#include <console/console.h>
+#include <pc80/mc146818rtc.h>
+#include <security/vboot/vbnv.h>
+#include <security/vboot/symbols.h>
+#include <soc/psp_transfer.h>
+#include <timestamp.h>
+#include <2struct.h>
+
+int transfer_buffer_valid(const struct transfer_info_struct *ptr)
+{
+ if (ptr->magic_val == TRANSFER_MAGIC_VAL && ptr->struct_bytes == sizeof(*ptr))
+ return 1;
+ else
+ return 0;
+}
+
+void verify_psp_transfer_buf(void)
+{
+ if (*(uint32_t *)_vboot2_work == VB2_SHARED_DATA_MAGIC) {
+ cmos_write(0x00, CMOS_RECOVERY_BYTE);
+ return;
+ }
+
+ /*
+ * If CMOS is valid and the system has already been rebooted once, but
+ * still returns here, instead of rebooting to verstage again, assume
+ * that the system is in a reboot loop and halt.
+ */
+ if ((!vbnv_cmos_failed()) && cmos_read(CMOS_RECOVERY_BYTE) ==
+ CMOS_RECOVERY_MAGIC_VAL)
+ die("Error: Reboot into recovery was unsuccessful. Halting.");
+
+ printk(BIOS_ERR, "VBOOT workbuf not valid.\n");
+ printk(BIOS_DEBUG, "Signature: %#08x\n", *(uint32_t *)_vboot2_work);
+ cmos_init(0);
+ cmos_write(CMOS_RECOVERY_MAGIC_VAL, CMOS_RECOVERY_BYTE);
+ warm_reset();
+}
+
+void show_psp_transfer_info(void)
+{
+ struct transfer_info_struct *info = (struct transfer_info_struct *)
+ (void *)(uintptr_t)_transfer_buffer;
+
+ if (transfer_buffer_valid(info)) {
+ if ((info->psp_info & PSP_INFO_VALID) == 0) {
+ printk(BIOS_INFO, "No PSP info found in transfer buffer.\n");
+ return;
+ }
+
+ printk(BIOS_INFO, "PSP boot mode: %s\n",
+ info->psp_info & PSP_INFO_PRODUCTION_MODE ?
+ "Production" : "Development");
+ printk(BIOS_INFO, "Silicon level: %s\n",
+ info->psp_info & PSP_INFO_PRODUCTION_SILICON ?
+ "Production" : "Pre-Production");
+ }
+}
+
+void replay_transfer_buffer_cbmemc(const struct transfer_info_struct *info)
+{
+
+ void *cbmemc;
+ size_t cbmemc_size;
+
+ if (info->console_offset < sizeof(*info))
+ return;
+
+ if (info->timestamp_offset <= info->console_offset)
+ return;
+
+ cbmemc_size = info->timestamp_offset - info->console_offset;
+
+ if (info->console_offset + cbmemc_size > info->buffer_size)
+ return;
+
+ cbmemc = (void *)((uintptr_t)info + info->console_offset);
+
+ /* We need to manually initialize cbmemc so we can fill the new buffer. cbmemc_init()
+ * will also be called later in console_hw_init(), but it will be a no-op. */
+ cbmemc_init();
+ cbmemc_copy_in(cbmemc, cbmemc_size);
+}
diff --git a/src/soc/amd/common/vboot/vboot_bootblock.c b/src/soc/amd/common/vboot/vboot_bootblock.c
index e3705d1..7264836 100644
--- a/src/soc/amd/common/vboot/vboot_bootblock.c
+++ b/src/soc/amd/common/vboot/vboot_bootblock.c
@@ -1,91 +1,10 @@
/* SPDX-License-Identifier: GPL-2.0-only */
-#include <amdblocks/reset.h>
#include <bootblock_common.h>
-#include <console/cbmem_console.h>
#include <console/console.h>
-#include <pc80/mc146818rtc.h>
-#include <security/vboot/vbnv.h>
-#include <security/vboot/symbols.h>
#include <soc/psp_transfer.h>
+#include <symbols.h>
#include <timestamp.h>
-#include <2struct.h>
-
-static int transfer_buffer_valid(const struct transfer_info_struct *ptr)
-{
- if (ptr->magic_val == TRANSFER_MAGIC_VAL && ptr->struct_bytes == sizeof(*ptr))
- return 1;
- else
- return 0;
-}
-
-void verify_psp_transfer_buf(void)
-{
- if (*(uint32_t *)_vboot2_work == VB2_SHARED_DATA_MAGIC) {
- cmos_write(0x00, CMOS_RECOVERY_BYTE);
- return;
- }
-
- /*
- * If CMOS is valid and the system has already been rebooted once, but
- * still returns here, instead of rebooting to verstage again, assume
- * that the system is in a reboot loop and halt.
- */
- if ((!vbnv_cmos_failed()) && cmos_read(CMOS_RECOVERY_BYTE) ==
- CMOS_RECOVERY_MAGIC_VAL)
- die("Error: Reboot into recovery was unsuccessful. Halting.");
-
- printk(BIOS_ERR, "VBOOT workbuf not valid.\n");
- printk(BIOS_DEBUG, "Signature: %#08x\n", *(uint32_t *)_vboot2_work);
- cmos_init(0);
- cmos_write(CMOS_RECOVERY_MAGIC_VAL, CMOS_RECOVERY_BYTE);
- warm_reset();
-}
-
-void show_psp_transfer_info(void)
-{
- struct transfer_info_struct *info = (struct transfer_info_struct *)
- (void *)(uintptr_t)_transfer_buffer;
-
- if (transfer_buffer_valid(info)) {
- if ((info->psp_info & PSP_INFO_VALID) == 0) {
- printk(BIOS_INFO, "No PSP info found in transfer buffer.\n");
- return;
- }
-
- printk(BIOS_INFO, "PSP boot mode: %s\n",
- info->psp_info & PSP_INFO_PRODUCTION_MODE ?
- "Production" : "Development");
- printk(BIOS_INFO, "Silicon level: %s\n",
- info->psp_info & PSP_INFO_PRODUCTION_SILICON ?
- "Production" : "Pre-Production");
- }
-}
-
-static void setup_cbmem_console(const struct transfer_info_struct *info)
-{
-
- void *cbmemc;
- size_t cbmemc_size;
-
- if (info->console_offset < sizeof(*info))
- return;
-
- if (info->timestamp_offset <= info->console_offset)
- return;
-
- cbmemc_size = info->timestamp_offset - info->console_offset;
-
- if (info->console_offset + cbmemc_size > info->buffer_size)
- return;
-
- cbmemc = (void *)((uintptr_t)info + info->console_offset);
-
- /* We need to manually initialize cbmemc so we can fill the new buffer. cbmemc_init()
- * will also be called later in console_hw_init(), but it will be a no-op. */
- cbmemc_init();
- cbmemc_copy_in(cbmemc, cbmemc_size);
-}
void boot_with_psp_timestamp(uint64_t base_timestamp)
{
@@ -95,7 +14,7 @@
if (!transfer_buffer_valid(info) || info->timestamp == 0)
return;
- setup_cbmem_console(info);
+ replay_transfer_buffer_cbmemc(info);
/*
* info->timestamp is PSP's timestamp (in microseconds)