Andre Heider has posted comments on this change. ( https://review.coreboot.org/23783 )
Change subject: soc/nvidia/tegra210: add missing bl31 params ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/23783/1/src/soc/nvidia/tegra210/arm_tf.c File src/soc/nvidia/tegra210/arm_tf.c:
https://review.coreboot.org/#/c/23783/1/src/soc/nvidia/tegra210/arm_tf.c@45 PS1, Line 45: t210_plat_params.uart_id = 1;
Shouldn't this depend on the CONSOLE_SERIAL_TEGRA210_UARTx Kconfigs?
It should, yes ;)
But this was just the first step without functional change. UARTA is currently hard wired, see e.g. src/soc/nvidia/tegra210/clock.c:clock_early_uart()
I have follow up patches which need a bit of polishing, but the end result is a working serial based on CONSOLE_SERIAL_TEGRA210_UARTx