[XL] Change in coreboot[main]: mb/protectli/vault_adl_p: Add initial support for VP6630/VP6650/VP6670

[Nico Huber (Code Review)]

Attention is currently required from: Felix Singer, Maciej Pijanowski, Michał Żygowski, Piotr Król.

Nico Huber has posted comments on this change by Michał Żygowski. ( https://review.coreboot.org/c/coreboot/+/80501?usp=email )

Change subject: mb/protectli/vault_adl_p: Add initial support for VP6630/VP6650/VP6670
......................................................................

Patch Set 8: Code-Review+1

(2 comments)

File src/mainboard/protectli/vault_adl_p/bootblock.c:

https://review.coreboot.org/c/coreboot/+/80501/comment/3f791baa_3ca990e8?usp... :
PS7, Line 41: polarity << pin
> Prepared a patch with a generic ITE SIo GPIO driver: CB:83355

Acknowledged

File src/mainboard/protectli/vault_adl_p/mainboard.c:

https://review.coreboot.org/c/coreboot/+/80501/comment/1a3f907b_c1febd63?usp... :
PS7, Line 29: }
> Exported it to a header file as static inline function

Why inline, though? We can add as many C files as we want. Inline only bloats
the binary.