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

[Paul Menzel (Code Review)]

Paul Menzel 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:

(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?

No, it’s not. Thank you for spotting this.

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? […]

Sure, do you want it below or above the line?

I haven’t analyzed yet, what the cause is. coreboot logs `Enabling resources` above the output.