Hello Felix Singer, build bot (Jenkins), Nico Huber, Patrick Georgi, Martin Roth, Werner Zeh, Andrey Petrov, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/48800
to look at the new patch set (#6).
Change subject: soc/intel/apl: move UART pad config to boards ......................................................................
soc/intel/apl: move UART pad config to boards
UART pad configuration should not be done in common code, because that may cause short circuits, when the user sets a wrong UART index. Thus, add the corresponding pads to the early UART gpio table for the boards.
Also switch to `bootblock_mainboard_early_init` to configure the pads in early bootblock before console initialization, to make the console work as early as possible. None of the boards do any other gpio configuration in bootblock, so this should not influence behaviour in a negative way (e.g. breaking overrides).
Only boards selecting INTEL_LPSS_UART_FOR_CONSOLE have been adapted.
Further, clear the `.gpios` field for the SoCs, so that the common code still works but no pad configuration is done anymore and add a temporary note that `.gpios` is intentionally left empty.
As soon as all platforms have moved the pad configuration to the boards, we can adapt the common code an drop the remaining structs all at once.
Change-Id: I28a0375a0a27ddaa758498473e37af38821fdd52 Signed-off-by: Michael Niewöhner foss@mniewoehner.de --- M src/mainboard/google/octopus/bootblock.c M src/mainboard/google/octopus/variants/baseboard/gpio.c M src/mainboard/google/reef/bootblock.c M src/mainboard/google/reef/variants/baseboard/gpio.c M src/mainboard/intel/apollolake_rvp/Makefile.inc A src/mainboard/intel/apollolake_rvp/bootblock.c A src/mainboard/intel/apollolake_rvp/gpio.h M src/mainboard/intel/glkrvp/bootblock.c M src/mainboard/intel/glkrvp/variants/baseboard/gpio.c M src/mainboard/intel/leafhill/bootblock.c M src/mainboard/intel/leafhill/brd_gpio.h M src/mainboard/intel/minnow3/bootblock.c M src/mainboard/intel/minnow3/gpio.c M src/mainboard/siemens/mc_apl1/bootblock.c M src/mainboard/siemens/mc_apl1/variants/baseboard/gpio.c M src/mainboard/up/squared/bootblock.c M src/mainboard/up/squared/gpio.h M src/soc/intel/apollolake/uart.c 18 files changed, 98 insertions(+), 38 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/48800/6