[coreboot-gerrit] Change in coreboot[master]: mb/gigabyte/ga-h61m-s2pv: Add ga-h61m-ds2v as a variant

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37483 )

Change subject: mb/gigabyte/ga-h61m-s2pv: Add ga-h61m-ds2v as a variant
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/37483/1/src/mainboard/gigabyte/ga-h... 
File src/mainboard/gigabyte/ga-h61m-s2pv/variants/ga-h61m-ds2v/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/37483/1/src/mainboard/gigabyte/ga-h... 
PS1, Line 75: 						irq 0xf0 = 0x08
> Killed.

We might be talking past each other: If the `superio.c` registers
i/o resources _and_ the LDN is enabled here without setting these
resources, the allocator will try to assign them. However, it does
a shitty job and usually breaks i/o allocation for PCI devices while
doing so.

Please check if you see any i/o resource related errors in the log.