[coreboot-gerrit] Change in coreboot[master]: security/vboot: relocate vb2ex_abort and vb2ex_printf

[Joel Kitching (Code Review)]

Joel Kitching has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38706 )

Change subject: security/vboot: relocate vb2ex_abort and vb2ex_printf
......................................................................

Patch Set 3:
> Patch Set 3:
>
> Would adding vb2ex_abort() and vb2ex_printf() to vboot library be the preferred solution?

By design, the "vb2ex" functions are supposed to be implemented by the caller.  These are mostly lower-level functions interacting with drivers or hardware in some way.  This also allows implementation to differ across the firmware being linked, i.e. coreboot or depthcharge.