Hello Patrick Rudolph, Huang Jin, Frans Hendriks, Lee Leahy, Philipp Deppenwiese, build bot (Jenkins), Nico Huber, Michał Żygowski, Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/32381
to look at the new patch set (#9).
Change subject: soc/intel/braswell: add default option to use public FSP ......................................................................
soc/intel/braswell: add default option to use public FSP
The current Braswell FSP 1.1 header in vendorcode/intel, for which there is no publicly available FSP binary, contains silicon init UPDs which are not found in the publicly available header/binary in the FSP Github repo. This prevents new boards from being added which use the public Braswell FSP header/binary.
To resolve this, move the UPDs not found in the public header from the soc's chip.c to ramstage.c for the boards which use them. Add a Kconfig option to use the current non-public FSP header and select it for boards which need it (google/cyan variants); set the public FSP option as the default. Use the Kconfig option to set FSP_HEADER_PATH to ensure the correct header is used.
Additionally, reorder FSP 1.1 Kconfig entries for improved flow/ readability, and set the default path for the FSP binary when the public FSP is selected.
Test: build google/cyan and intel/strago using non-public and public FSP header/binaries respectively.
Change-Id: I43cf18b98c844175a87b61fdbe4b0b24484e5702 Signed-off-by: Matt DeVillier matt.devillier@gmail.com --- M src/drivers/intel/fsp1_1/Kconfig M src/mainboard/google/cyan/Kconfig M src/mainboard/google/cyan/variants/celes/devicetree.cb M src/mainboard/google/cyan/variants/celes/ramstage.c M src/mainboard/google/cyan/variants/kefka/Makefile.inc M src/mainboard/google/cyan/variants/kefka/devicetree.cb A src/mainboard/google/cyan/variants/kefka/ramstage.c M src/mainboard/google/cyan/variants/relm/devicetree.cb M src/mainboard/google/cyan/variants/relm/ramstage.c M src/soc/intel/braswell/Kconfig M src/soc/intel/braswell/Makefile.inc M src/soc/intel/braswell/chip.c M src/soc/intel/braswell/chip.h 13 files changed, 87 insertions(+), 65 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/32381/9