Kangheui Won has uploaded this change for review.

View Change

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) {

To view, visit change 44751. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I14bc34e6af501240a6f633db3999a7759e88d60b
Gerrit-Change-Number: 44751
Gerrit-PatchSet: 1
Gerrit-Owner: Kangheui Won <khwon@chromium.org>
Gerrit-MessageType: newchange