Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47977 )
Change subject: soc/amd/common/vboot: use transfer_buffer_valid function ......................................................................
soc/amd/common/vboot: use transfer_buffer_valid function
show_psp_transfer_info reimplemented the functionality of transfer_buffer_valid, so use replace that with a function call.
Change-Id: Ie3d373b10bdb0ab00640dabeea12b13ec25406cc Signed-off-by: Felix Held felix-coreboot@felixheld.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/47977 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Martin Roth martinroth@google.com --- M src/soc/amd/common/vboot/vboot_bootblock.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Martin Roth: Looks good to me, approved
diff --git a/src/soc/amd/common/vboot/vboot_bootblock.c b/src/soc/amd/common/vboot/vboot_bootblock.c index 45ad9de..da652eb 100644 --- a/src/soc/amd/common/vboot/vboot_bootblock.c +++ b/src/soc/amd/common/vboot/vboot_bootblock.c @@ -47,7 +47,7 @@ struct transfer_info_struct *info = (struct transfer_info_struct *) (void *)(uintptr_t)_transfer_buffer;
- if (info->magic_val == TRANSFER_MAGIC_VAL) { + 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;