[coreboot-gerrit] [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 10: Code-Review+2

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/80501/comment/1fd91190_4506888a?usp... :
PS7, Line 29: }
> Created a C file with the beep function

Acknowledged

https://review.coreboot.org/c/coreboot/+/80501/comment/897aeab8_8fb16b6d?usp... :
PS7, Line 51: 	}
> Uhh, I didn't realize there is a function to get a processor name already under cpu/x86/name. […]

Acknowledged