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

[Michał Żygowski (Code Review)]

Michał Żygowski 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 3:

(2 comments)

https://review.coreboot.org/#/c/31229/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/31229/2//COMMIT_MSG@13 
PS2, Line 13: during huge load. Processor has additional boosted P-states when
> 'during high load if other cores idle'

Done. I have also changed the max frequency to 1400MHz, since memtest86+ shows overall ~40% increase in memory speed and the processor's specification says max frequency 1400MHz. The benchmark results showed ~20% increase in performance, but these are probably biased by background operations that OS performs.

https://review.coreboot.org/#/c/31229/2//COMMIT_MSG@17 
PS2, Line 17: memory bandwidth.
> Yet from specs we know that enabling CPB does not increase overall memory bandwidth? […]

Done