Wim Vervoorn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38590 )
Change subject: vendorcode/eltan/security: Switch to vb2 vboot library ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/38590/2/src/vendorcode/eltan/securi... File src/vendorcode/eltan/security/lib/Makefile.inc:
PS2:
Can't you just remove this whole file and select CONFIG_VBOOT_LIB now?
Thanks for the suggestion. We have removed what's left of this library so now we're only using the vboot library.
https://review.coreboot.org/c/coreboot/+/38590/2/src/vendorcode/eltan/securi... File src/vendorcode/eltan/security/verified_boot/vboot_check.c:
https://review.coreboot.org/c/coreboot/+/38590/2/src/vendorcode/eltan/securi... PS2, Line 17: #define NEED_VB20_INTERNALS /* Peeking into vb2_shared_data */
What is the long-term plan here? We do not want future development on vboot to be restricted by rand […]
Understood. For the next project we moved to a vboot implementation. For this one we don't have plans to switch at the moment. Is there a timeline from your side?