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 6:
(3 comments)
https://review.coreboot.org/c/coreboot/+/37483/6//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37483/6//COMMIT_MSG@14 PS6, Line 14: a an? (depends on how you pronounce FT2232H)
https://review.coreboot.org/c/coreboot/+/37483/6//COMMIT_MSG@27 PS6, Line 27: - Audio: Only front/rear outputs have been tested. What other outputs are there?
https://review.coreboot.org/c/coreboot/+/37483/6/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/6/src/mainboard/gigabyte/ga-h... PS6, Line 93: device pci 1f.6 on end # Thermal What difference does it make? Does Linux have a driver for it that does something on its own?