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

[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}: Always use CAR NEM enhanced by default
......................................................................

Patch Set 3:

Folks, I know now that I shouldn't have used the word `insane`
in the commit message. I looked it up, realized that it is a much
stronger word than I expected and changed it immediately.

But... spending hours with a review that fixes a regression that
only made it through the review process because the reviewing
party got their comforting cushion in the patch? That really shows
that something is wrong in the community. And then covering up for
the tension in the community with an adapted commit message? I
think that would even make it worse.

Please, don't ignore tension in the community. Please, make it
visible wherever you can! If we cover it up, we can never learn
from past faults. And I think we all have to learn and hope we
will.