Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33522 )
Change subject: src/mb/gigabyte/ga-h61m-s2pv: Correct devicetree ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/33522/2/src/mainboard/gigabyte/ga-h61m-s2pv/... File src/mainboard/gigabyte/ga-h61m-s2pv/devicetree.cb:
https://review.coreboot.org/#/c/33522/2/src/mainboard/gigabyte/ga-h61m-s2pv/... PS2, Line 96: irq 0xf1 = 0x40
Ack, it seems the PNP_MSCx are only necessary for warnings when […]
Right now it's "vendor used this, if I use it I don't think anything will short out". Not ideal, most of these don't make much sense. At least this isn't using 'io' on the wrong places anymore ('io' sets two bytes).