Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37231 )
Change subject: security/vboot: Remove struct vboot_working_data ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/37231/2/src/security/vboot/common.c File src/security/vboot/common.c:
https://review.coreboot.org/c/coreboot/+/37231/2/src/security/vboot/common.c... PS2, Line 85: #if CONFIG(VBOOT_STARTS_IN_BOOTBLOCK)
I think we could now just rewrite this whole section as […]
Done
https://review.coreboot.org/c/coreboot/+/37231/2/src/security/vboot/common.c... PS2, Line 98: vb2api_relocate(wb_cbmem, wb_preram, cbmem_size,
(Which also reminds me, we probably want to check the return value here... […]
Ack