Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49025 )
Change subject: mb/clevo/cml-u: Rework Kconfig ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/49025/3/src/mainboard/clevo/cml-u/K... File src/mainboard/clevo/cml-u/Kconfig:
https://review.coreboot.org/c/coreboot/+/49025/3/src/mainboard/clevo/cml-u/K... PS3, Line 21: _OPTIONS
we do, since it is already declared in Kconfig. […]
well, vboot below (and others) is also already declared
https://review.coreboot.org/c/coreboot/+/49025/3/src/mainboard/clevo/cml-u/K... PS3, Line 21: onfig BOARD_CLEVO_L140CU_OPTIONS : bool : select BOARD_CLEVO_CMLU_COMMON : select EC_SYSTEM76_EC : select HAVE_SPD_IN_CBFS
I thought about that, but IMO the amount of config options is too small compared to the complexity t […]
not yet :D but ack, we can revisit that later