Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37816 )
Change subject: AGESA fam14: Remove early PCI subsystem ID setting ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37816/1/src/cpu/amd/agesa/family14/... File src/cpu/amd/agesa/family14/fixme.c:
https://review.coreboot.org/c/coreboot/+/37816/1/src/cpu/amd/agesa/family14/... PS1, Line 78: /* Set Ontario Link Data */ : PciAddress.AddressValue = MAKE_SBDFO(0, 0, 0, 0, 0xE0); : PciData = 0x01308002; : LibAmdPciWrite(AccessWidth32, PciAddress, &PciData, &StdHeader); Remove these too. The subsystem ID is set via indexed registers.