Marc Jones has posted comments on this change. ( https://review.coreboot.org/21333 )
Change subject: amd/stoneyridge: Remove duplicate PCI definitions ......................................................................
Patch Set 1:
(1 comment)
Where are the other definitions?
https://review.coreboot.org/#/c/21333/1/src/soc/amd/stoneyridge/include/soc/... File src/soc/amd/stoneyridge/include/soc/pci_devs.h:
https://review.coreboot.org/#/c/21333/1/src/soc/amd/stoneyridge/include/soc/... PS1, Line 22: A comment about where these are declared would be helpful here. I'd assume that they needed to be in this file with the others.