Name of user not set #1002789 has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44102 )
Change subject: [WIP]mb/intel/minnowmax: Add mainboard ......................................................................
Patch Set 3:
(3 comments)
A few more comments. Sorry for the flood, just want to get things working. I can confirm the board boots Windows 10 just fine (tested with Tianocore) though I couldn't get all the drivers sorted out and it won't boot off of the SD card like the stock firmware would (even after telling the ACPI table to make the SD card not removable). Ubuntu 20.04 runs great with no known issues booting off of the SD card (so long as the devicetree.cb says 0 to setting ACPI mode, again I tested with Tianocore as the payload). A handful of devices show when I do lspci that aren't present in the stock firmware (and contribute to some of the Windows driver issues) but they don't appear to hurt anything or cause errors in dmesg.
https://review.coreboot.org/c/coreboot/+/44102/3/src/mainboard/intel/minnowm... File src/mainboard/intel/minnowmax/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/44102/3/src/mainboard/intel/minnowm... PS3, Line 12: 0 Set to 1 to enable USB3. Not sure why this was like this in the original devicetree.cb.
https://review.coreboot.org/c/coreboot/+/44102/3/src/mainboard/intel/minnowm... PS3, Line 42: on My Minnowboard doesn't have this, so I set mine to off. Does yours?
https://review.coreboot.org/c/coreboot/+/44102/3/src/mainboard/intel/minnowm... PS3, Line 53: off My Minnowboard has this, and is the only way to get audio, so I set mine to on.