Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73087 )
Change subject: mb/intel/dq67sw: Add LGA1155 microATX mainboard ......................................................................
Patch Set 7:
(1 comment)
Patchset:
PS7: Thank you again for making this port, and also for [uploading the logs to the board status repository](https://review.coreboot.org/plugins/gitiles/board-status/+/a25a97fee15a26b1d...). Three observations:
1. Despite `CONFIG_COMPRESSED_PAYLOAD_LZMA=y` the `cbfstool` output says, that [`fallback/payload` is not compressed](https://review.coreboot.org/plugins/gitiles/board-status/+/a25a97fee15a26b1d...):
fallback/payload 0x440c0 simple elf 780146 none
2. With log level `SPEW` the CBMEM pre-console buffer is too small and might need to be increased for your board.
3. There are warnings about leftover static devices:
[WARN ] PCI: Leftover static devices: [WARN ] PCI: 00:01.1 [WARN ] PCI: 00:01.2 [WARN ] PCI: 00:04.0 [WARN ] PCI: 00:06.0 [WARN ] PCI: 00:14.0 [WARN ] PCI: 00:16.1 [WARN ] PCI: 00:16.2 [WARN ] PCI: 00:16.3 [WARN ] PCI: 00:1c.1 [WARN ] PCI: 00:1c.2 [WARN ] PCI: 00:1c.3 [WARN ] PCI: 00:1c.7 [WARN ] PCI: Check your devicetree.cb.