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

[Subrata Banik (Code Review)]

Subrata Banik 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:
> 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.

`default USE_CANNONLAKE_CAR_NEM_ENHANCED` this is what i meant just to make sure we are running with pre-validated configuration and noone need to debug some unnecessary hang issue