Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/85007?usp=email )
Change subject: soc/amd/common/psp/rpmc: bring debug output in line with fmap section ......................................................................
soc/amd/common/psp/rpmc: bring debug output in line with fmap section
Call the PSP RPMC NVRAM 'PSP RPMC NVRAM' instead of 'PSP NVRAM' in the debug console output to not be misleading, since the RPMC feature uses the 'PSP_RPMC_NVRAM' fmap section and not the 'PSP_NVRAM' fmap section.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: Ie89dfcfe4b8780f422c222477bb627e03bd3662d Reviewed-on: https://review.coreboot.org/c/coreboot/+/85007 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Matt DeVillier matt.devillier@amd.corp-partner.google.com --- M src/soc/amd/common/block/psp/rpmc.c 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Matt DeVillier: Looks good to me, approved
diff --git a/src/soc/amd/common/block/psp/rpmc.c b/src/soc/amd/common/block/psp/rpmc.c index a557459..dd097c5 100644 --- a/src/soc/amd/common/block/psp/rpmc.c +++ b/src/soc/amd/common/block/psp/rpmc.c @@ -68,8 +68,8 @@
static void print_rpmc_general_status(uint8_t healthy, uint8_t rpmc_protected) { - printk(BIOS_SPEW, "PSP NVRAM %s healthy\n", healthy ? "is" : "isn't"); - printk(BIOS_SPEW, "PSP NVRAM %s using RPMC protection\n", + printk(BIOS_SPEW, "PSP RPMC NVRAM %s healthy\n", healthy ? "is" : "isn't"); + printk(BIOS_SPEW, "PSP RPMC NVRAM %s using RPMC protection\n", rpmc_protected ? "is" : " isn't"); }