[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:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/34166/1/src/soc/intel/cannonlake/Kc... 
PS1, Line 274:
> This had a prompt all along, so I expect everyone to select what suits their […]

i don;t think ppl know what suites their platform!  we have seen issue randomly with CAR NEM hence i would recommend to fixed it with NEM enhance. I had 2 default to satisfy IOT ask in past and now if they are okay with it. i would like to go with USE_CANNONLAKE_CAR_NEM_ENHANCED