Hello Jonathan Zhang, David Hendricks, Philipp Deppenwiese, Lance Zhao, build bot (Jenkins), Lijian Zhao, Anjaneya "Reddy" Chagam, Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/38640
to look at the new patch set (#8).
Change subject: cpu: Allow to configure microcode at pre-defined address ......................................................................
cpu: Allow to configure microcode at pre-defined address
FSP-T takes microcode pointer and location parameters, and FSP-T is invoked before CAR is set-up and before memory is trained. So it is not possible to modify supplied microcode pointer in runtime. Because of that we have to hardcode the pointer in bootblock.
Also, current FSP-T on Xeons require microcode (it is not optional). Reasons for that are currently unclear and are being investigated.
However for the present time we need to be able to add microcode at a certain offset so FSP-T can be used.
TEST=test on OCP TiogaPass board, as well as out-of-tree CPU/board
Change-Id: I6c02601a7ac64078e556e2032baeccaf27f77da2 Signed-off-by: Andrey Petrov anpetrov@fb.com --- M src/cpu/Makefile.inc 1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/40/38640/8