Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39926 )
Change subject: [TEST] mb/asus/p5ql_pro: Re-add second southbridge to dt ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/39926/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39926/1//COMMIT_MSG@9 PS1, Line 9: This was lost during review. We don't need it, though!
(comment to prevent accidental submission)
https://review.coreboot.org/c/coreboot/+/39926/1/src/mainboard/asus/p5qc/var... File src/mainboard/asus/p5qc/variants/p5ql_pro/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/39926/1/src/mainboard/asus/p5qc/var... PS1, Line 66: chip superio/winbond/w83667hg-a # Super I/O There used to be a subsystemid entry here, which was wrongly indented and had a "corresponding" end statement
https://review.coreboot.org/c/coreboot/+/39926/1/src/mainboard/asus/p5qc/var... PS1, Line 117: end The additional `end` was here.