Andre Heider has posted comments on this change. ( https://review.coreboot.org/23783 )
Change subject: soc/nvidia/tegra210: add missing bl31 params to ATF config ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/#/c/23783/3/src/soc/nvidia/tegra210/arm_tf.c File src/soc/nvidia/tegra210/arm_tf.c:
https://review.coreboot.org/#/c/23783/3/src/soc/nvidia/tegra210/arm_tf.c@46 PS3, Line 46: switch (console_uart_get_id()) {
I don't really care, whatever you prefer
Okay, so lets keep it this way then. The switch statement has the bonus of additional compiler checks in case the enum gets changed in the future. Thanks for the reviews!