[coreboot-gerrit] Change in ...coreboot[master]: src/mainboard/pcengines/apu2/OemCustomize.c: Enable CPB feature

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31229 )

Change subject: src/mainboard/pcengines/apu2/OemCustomize.c: Enable CPB feature
......................................................................

Patch Set 2:
> Patch Set 1:
> > Patch Set 1:
> > > Patch Set 1:
> > > > Patch Set 1:
> > > >
> > > > Michal, did you resolve the instability? I think I have some old notes about this.
> > >
> > > I have not encountered any issues. Would You mind sending these notes on priv so I can verify them?
> >
> > Done; I understood apu2c4 was instable, apu3a2. Wondering if it was somehow memorymap or ECC related.
>
> Thank You. I have tested the patch on apu2c4 actually and instability seems to be gone.

Ok. It would be nice to have a definite answer if the previously applied patch on P-states was the culprit. So if you could revert the P-state change locally and see if it becomes unstable again and comment here.