[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl,icl}: Drop insane defaults for CAR implementation

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34166 )

Change subject: soc/intel/{cnl,icl}: Drop insane defaults for CAR implementation
......................................................................

Patch Set 1:

I don't follow. This patch already sets `USE_CANNONLAKE_
CAR_NEM_ENHANCED` as default (just implicitly because the
first entry of a choice is always the default). If you
want me to add an unnecessary but explicit `default
USE_CANNONLAKE_CAR_NEM_ENHANCED`, let me know.

However, in that case I would also like to make it a con-
vention. General Kconfig style shouldn't be part of this
review. Somebody would have to write a lint test to check
for choices without explicit default and we should discuss
if that is what we want for the project.