[coreboot-gerrit] Change in coreboot[master]: src/soc/intel/denverton_ns: Use improvement in coreboot since 4.9

[Mariusz Szafrański (Code Review)]

Mariusz Szafrański has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47340 )

Change subject: src/soc/intel/denverton_ns: Use improvement in coreboot since 4.9
......................................................................

Patch Set 3:
> Patch Set 3:
> > Patch Set 3: Code-Review+1
> >
> > There are issues with setting NEM on Denverton platform reliable.
> > FSP-CAR works correctly and reliable so it the suggested method to setup CAR.
> > NEM could be removed in the future.
>
> Um, no. If coreboot CAR init isn't working well on Denverton, the expected approach is to investigate why. Given that APL uses CQOS, maybe that's what DNV should also be using instead?

To be clear. In the past I`ve only deeply checked setting temp ram using FSP-T TempRamInit call. And can confirm that this methods works correctly and is reliable on DNV. So my suggestion to use it instead of others.