[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:

(2 comments)

I was referring to SVI2 P-State patch:
https://review.coreboot.org/c/coreboot/+/30367

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'

https://review.coreboot.org/#/c/31229/2//COMMIT_MSG@17 
PS2, Line 17: memory bandwidth.
> Will "memtest86+ showing increased memory speed when boost is enabled" be ok?

Yet from specs we know that enabling CPB does not increase overall memory bandwidth?

'TEST: Higher single-core CPU performance is indicated by increased memory bandwidth as reported by memtest86+'