[coreboot-gerrit] Change in coreboot[master]: ec/lenovo/h8: Add a config to indicate whether dock init is needed

[Bill XIE (Code Review)]

Bill XIE has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36385 )

Change subject: ec/lenovo/h8: Add a config to indicate whether dock init is needed
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36385/2/src/ec/lenovo/h8/h8.c 
File src/ec/lenovo/h8/h8.c:

https://review.coreboot.org/c/coreboot/+/36385/2/src/ec/lenovo/h8/h8.c@357 
PS2, Line 357: #if CONFIG(H8_DOCK_INIT)
> I would rather keep this scheme, as I have no condition to test all boards that use h8_mainboard_ini […]

The H8_DOCK_INIT could be treated as transitional, eliminating empty h8_mainboard_init_dock() declarations first, and migrate to your scheme board by board, just as C_ENVIRONMENT_BOOTBLOCK.