[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:
>
> When I leave the blob default, I get prompted (in a different
> location non-obvious location) for CPU_MICROCODE_CBFS_LEN
> and CPU_MICROCODE_CBFS_LOC. Those have non-working default
> values (0). Resulting in a brick.
>
> With a reasonable default here, the open-source NEM-enhanced,
> everything works fine.

We can make NEM-enhanced a default option then but in past IOTG folks wishes to make FSP-T usage for them. i believe default can be overriden so they can still use FSP-CAR if wishes.