Change in coreboot[master]: soc/amd/picasso: Add fields for the PSP to the transfer struct

[Martin Roth (Code Review)]

Martin Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45803 )

Change subject: soc/amd/picasso: Add fields for the PSP to the transfer struct
......................................................................

soc/amd/picasso: Add fields for the PSP to the transfer struct

The PSP will be adding information into these fields after verstage
runs.  This allows data to be passed directly to coreboot very early
in the boot process.

BUG=b:168895748
TEST=None
Branch=Zork

Signed-off-by: Martin Roth 
Change-Id: Idbd1dfece59e99f6f15dfd8d002529ea6417cdbe
---
M src/soc/amd/picasso/include/soc/psp_transfer.h
1 file changed, 7 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/03/45803/1

diff --git a/src/soc/amd/picasso/include/soc/psp_transfer.h b/src/soc/amd/picasso/include/soc/psp_transfer.h
index 0996f64..be88ce8 100644
--- a/src/soc/amd/picasso/include/soc/psp_transfer.h
+++ b/src/soc/amd/picasso/include/soc/psp_transfer.h
@@ -30,7 +30,13 @@
    uint32_t	console_offset;
    uint32_t	timestamp_offset;
    uint32_t	fmap_offset;
-	uint32_t	unused[9];		/* Pad to 64 bytes */
+
+	uint32_t	unused1[5];
+
+	/* Fields reserved for the PSP */
+	uint64_t	timestamp;		/* Offset 0x30 */
+	uint32_t	psp_unused;		/* Offset 0x38 */
+	uint32_t	psp_info;		/* Offset 0x3C */
 };

_Static_assert(sizeof(struct transfer_info_struct) == TRANSFER_INFO_SIZE, \

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45803 )

Change subject: soc/amd/picasso: Add fields for the PSP to the transfer struct
......................................................................

Patch Set 1: Code-Review+2

[Felix Held (Code Review)]

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

Change subject: soc/amd/picasso: Add fields for the PSP to the transfer struct
......................................................................

soc/amd/picasso: Add fields for the PSP to the transfer struct

The PSP will be adding information into these fields after verstage
runs.  This allows data to be passed directly to coreboot very early
in the boot process.

BUG=b:168895748
TEST=None
Branch=Zork

Signed-off-by: Martin Roth 
Change-Id: Idbd1dfece59e99f6f15dfd8d002529ea6417cdbe
Reviewed-on: https://review.coreboot.org/c/coreboot/+/45803
Tested-by: build bot (Jenkins) 
Reviewed-by: Felix Held 
---
M src/soc/amd/picasso/include/soc/psp_transfer.h
1 file changed, 7 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Felix Held: Looks good to me, approved

diff --git a/src/soc/amd/picasso/include/soc/psp_transfer.h b/src/soc/amd/picasso/include/soc/psp_transfer.h
index 0996f64..be88ce8 100644
--- a/src/soc/amd/picasso/include/soc/psp_transfer.h
+++ b/src/soc/amd/picasso/include/soc/psp_transfer.h
@@ -30,7 +30,13 @@
    uint32_t	console_offset;
    uint32_t	timestamp_offset;
    uint32_t	fmap_offset;
-	uint32_t	unused[9];		/* Pad to 64 bytes */
+
+	uint32_t	unused1[5];
+
+	/* Fields reserved for the PSP */
+	uint64_t	timestamp;		/* Offset 0x30 */
+	uint32_t	psp_unused;		/* Offset 0x38 */
+	uint32_t	psp_info;		/* Offset 0x3C */
 };

_Static_assert(sizeof(struct transfer_info_struct) == TRANSFER_INFO_SIZE, \