[coreboot-gerrit] Change in coreboot[master]: mb/asus/f2a85-m_pro: Enable PCIe bridge 00:15.2 in AGESA

[Werner Zeh (Code Review)]

Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46527 )

Change subject: mb/asus/f2a85-m_pro: Enable PCIe bridge 00:15.2 in AGESA
......................................................................

Patch Set 4: Code-Review+1

(2 comments)

https://review.coreboot.org/c/coreboot/+/46527/4//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/46527/4//COMMIT_MSG@12 
PS4, Line 12: ,
Is that needed here?

https://review.coreboot.org/c/coreboot/+/46527/4/src/mainboard/asus/f2a85-m/... 
File src/mainboard/asus/f2a85-m/devicetree_f2a85-m_pro.cb:

https://review.coreboot.org/c/coreboot/+/46527/4/src/mainboard/asus/f2a85-m/... 
PS4, Line 114: # FIXME: serial console stops working when enabling resources,
This is a long line...mind to break it down a bit and move part of it to the next line?
And by "enabling resources" you mean when the BARs of the PCI device are enumerated? Or is it realy the PCI device that causes the issues?