[coreboot-gerrit] Change in ...coreboot[master]: [WIP]device/pci_rom: Add ATRM method

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31502 )

Change subject: [WIP]device/pci_rom: Add ATRM method
......................................................................

Patch Set 1:
> Thank you for reply, although this change above seems too hard for me to execute properly

We can help you with that. It's not a reason to give up.
Either we fix that AMD code properly, or there'll be no
further upstreaming.
> and to be honest it is hard for me to justify working on it since my dGPU is already working, unless these ATRMs could be somehow helpful later for Crossfire.

If you want it upstream, it is not only about your dGPU
and we'll have to find a maintainable solution (some
trial-by-error code that already nobody understands when
merging is not what I'd consider maintainable).