[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl,icl}: Always use CAR NEM enhanced by default

[Patrick Georgi (Code Review)]

Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34287 )

Change subject: soc/intel/{cnl,icl}: Always use CAR NEM enhanced by default
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34287/2/src/soc/intel/cannonlake/Kc... 
File src/soc/intel/cannonlake/Kconfig:

https://review.coreboot.org/c/coreboot/+/34287/2/src/soc/intel/cannonlake/Kc... 
PS2, Line 272: 	default USE_CANNONLAKE_CAR_NEM_ENHANCED
> 'always set a default' -- I'm confused. […]

The default in a choice is to pick the first entry, so in this case, there's a default, just no statement for it. As far as I understand Nico he prefers to not use statements that merely restate default behavior.

What to standardize on is better discussed on the list than here though.