Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42830 )
Change subject: soc/amd/picasso: Add console & timestamp buffers to psp_verstage ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/42830/3/src/soc/amd/picasso/memlayo... File src/soc/amd/picasso/memlayout_psp_verstage.ld:
https://review.coreboot.org/c/coreboot/+/42830/3/src/soc/amd/picasso/memlayo... PS3, Line 57: vboot2_work
I'm not sure. I'll take a look, but even if there isn't we can add something. […]
I guess you could add a struct at the beginning of the transfer buf that describes what got transferred so we don't have to change the console and timestamp structs to include a magic value.
https://review.coreboot.org/c/coreboot/+/42830/3/src/soc/amd/picasso/psp_ver... File src/soc/amd/picasso/psp_verstage/psp_verstage.c:
https://review.coreboot.org/c/coreboot/+/42830/3/src/soc/amd/picasso/psp_ver... PS3, Line 155: _vboot2_work
how about _transfer_buf? naming it _workbuf but including things in it that aren't the workbuf are […]
I like that.