Simon Glass has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31887 )
Change subject: vboot: make vboot workbuf available to payload ......................................................................
Patch Set 10: Code-Review+1
(2 comments)
https://review.coreboot.org/#/c/31887/9/payloads/libpayload/libc/coreboot.c File payloads/libpayload/libc/coreboot.c:
https://review.coreboot.org/#/c/31887/9/payloads/libpayload/libc/coreboot.c@... PS9, Line 91: struct
In that case, we can add some consts in a follow-up CL.
OK but start small, as I am not an authority in coreboot at all. It has different conventions from U-Boot where I am more familiar.
https://review.coreboot.org/#/c/31887/9/src/security/vboot/misc.h File src/security/vboot/misc.h:
https://review.coreboot.org/#/c/31887/9/src/security/vboot/misc.h@32 PS9, Line 32: CPU architecture agnostic
Well, I suppose it would be more accurate to say that vboot_working_data "is placed just before the […]
Ack