Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40322 )
Change subject: soc/amd/picasso: Clean up legacy UART config ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/40322/2/src/soc/amd/picasso/Kconfig File src/soc/amd/picasso/Kconfig:
https://review.coreboot.org/c/coreboot/+/40322/2/src/soc/amd/picasso/Kconfig... PS2, Line 169: uart
on the Mandolin reference board there are I think all four UARTs routed to headers, so this probably […]
fixed the UART comment, but unsure if I should mark Nico's comment as solved
https://review.coreboot.org/c/coreboot/+/40322/2/src/soc/amd/picasso/uart.c File src/soc/amd/picasso/uart.c:
https://review.coreboot.org/c/coreboot/+/40322/2/src/soc/amd/picasso/uart.c@... PS2, Line 42:
double empty line
Done