Kangheui Won has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44751 )
Change subject: soc/amd/picasso: copy local info to transfer buf ......................................................................
soc/amd/picasso: copy local info to transfer buf
buffer_info is local variable, we need to copy it to _transfer_buffer before we hand it over to PSP.
BUG=b:159220781 TEST=check transfer_info_struct is correctly populated on romstage
Signed-off-by: Kangheui Won khwon@chromium.org Change-Id: I14bc34e6af501240a6f633db3999a7759e88d60b --- M src/soc/amd/picasso/psp_verstage/psp_verstage.c 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/51/44751/1
diff --git a/src/soc/amd/picasso/psp_verstage/psp_verstage.c b/src/soc/amd/picasso/psp_verstage/psp_verstage.c index 005c8b0..ef4884f 100644 --- a/src/soc/amd/picasso/psp_verstage/psp_verstage.c +++ b/src/soc/amd/picasso/psp_verstage/psp_verstage.c @@ -186,6 +186,8 @@ buffer_info.workbuf_offset = (uint32_t)((uintptr_t)_fmap_cache - (uintptr_t)_vboot2_work);
+ memcpy(_transfer_buffer, &buffer_info, sizeof(buffer_info)); + retval = svc_save_uapp_data(UAPP_COPYBUF_CHROME_WORKBUF, (void *)_transfer_buffer, buffer_size); if (retval) {