Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45479 )
Change subject: soc/amd/picasso/acpi: add UART DMA controller devices ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/45479/1/src/soc/amd/picasso/acpi/sb... File src/soc/amd/picasso/acpi/sb_fch.asl:
https://review.coreboot.org/c/coreboot/+/45479/1/src/soc/amd/picasso/acpi/sb... PS1, Line 182: Device (DMA3) {
How does the driver associate these devices w/ the proper uart? Is that going to be open coded someh […]
The mapping is in the first parameter of FixedDMA. I'm not sure how that works in detail though
https://review.coreboot.org/c/coreboot/+/45479/1/src/soc/amd/picasso/acpi/sb... PS1, Line 221: Exclusive
You changed FUR1, FUR2, and FUR3 to Shared.
good catch