[coreboot-gerrit] Change in ...coreboot[master]: mb/lenovo: Add Lenovo Thinkpad Helix

[Felix Singer (Code Review)]

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33867 )

Change subject: mb/lenovo: Add Lenovo Thinkpad Helix
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/33867/1/src/mainboard/lenovo/helix/Kconfig 
File src/mainboard/lenovo/helix/Kconfig:

PS1:
> MAINBOARD_VENDOR is selected in top-level Kconfig (from mb/lenovo directory) […]

* PXE_ROM_ID Why not? If someone wants to use iPXE on it, it's preconfigured.
* CBFS_SIZE Maybe, depends on the partition layout. In my experience, it's much more bigger.