[coreboot-gerrit] Change in ...coreboot[master]: mb/gigabyte/ga-h61ma-d3v: Add new mainboard as variant

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31832 )

Change subject: mb/gigabyte/ga-h61ma-d3v: Add new mainboard as variant
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/#/c/31832/4/src/mainboard/gigabyte/ga-h61m-s2pv/... 
File src/mainboard/gigabyte/ga-h61m-s2pv/variants/ga-h61ma-d3v/devicetree.cb:

https://review.coreboot.org/#/c/31832/4/src/mainboard/gigabyte/ga-h61m-s2pv/... 
PS4, Line 78: 					device pnp 2e.7 off	# GPIO
Is the 'off' here correct?

https://review.coreboot.org/#/c/31832/4/src/mainboard/gigabyte/ga-h61m-s2pv/... 
PS4, Line 88: 						io 0xf6 = 0x1c
Not sure... I think we used whacky 'irq' syntax for these generic registers.