[coreboot-gerrit] Change in coreboot[master]: soc/amd/cezanne/psp_verstage: Log the platform boot mode report

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/62668 )

Change subject: soc/amd/cezanne/psp_verstage: Log the platform boot mode report
......................................................................

soc/amd/cezanne/psp_verstage: Log the platform boot mode report

Log the platform boot mode reported by PSP verstage to PSP stage 1
bootloader. This helps to improve the debuggability.

BUG=b:193050286
TEST=Build and Boot to OS in Nipperkin. Ensure that the platform boot
mode is logged in the verstage logs.

Signed-off-by: Karthikeyan Ramasubramanian 
Change-Id: I752ee56f2af48215a770d799432d02f0609757cd
Reviewed-on: https://review.coreboot.org/c/coreboot/+/62668
Tested-by: build bot (Jenkins) 
Reviewed-by: Rob Barnes 
Reviewed-by: Jason Glenesk 
---
M src/soc/amd/cezanne/psp_verstage/chipset.c
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Rob Barnes: Looks good to me, approved
  Jason Glenesk: Looks good to me, but someone else must approve

diff --git a/src/soc/amd/cezanne/psp_verstage/chipset.c b/src/soc/amd/cezanne/psp_verstage/chipset.c
index 41d318e..372fa36 100644
--- a/src/soc/amd/cezanne/psp_verstage/chipset.c
+++ b/src/soc/amd/cezanne/psp_verstage/chipset.c
@@ -1,6 +1,7 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

#include <bl_uapp/bl_syscall_public.h>
+#include <console/console.h>
 #include <psp_verstage.h>

uint32_t update_psp_bios_dir(uint32_t *psp_dir_offset, uint32_t *bios_dir_offset)
@@ -35,6 +36,8 @@

void platform_report_mode(int developer_mode_enabled)
 {
+	printk(BIOS_INFO, "Reporting %s mode\n",
+			developer_mode_enabled ? "Developer" : "Production");
    if (developer_mode_enabled)
    	svc_set_platform_boot_mode(CHROME_BOOK_BOOT_MODE_DEVELOPER);
    else