build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34603 )
Change subject: src/mainboard/asus: Add h110m-e_m2 mainboard support ......................................................................
Patch Set 5:
(3 comments)
https://review.coreboot.org/c/coreboot/+/34603/5/src/mainboard/asus/h110m-e_... File src/mainboard/asus/h110m-e_m2/romstage.c:
https://review.coreboot.org/c/coreboot/+/34603/5/src/mainboard/asus/h110m-e_... PS5, Line 28: {0x33, 0xCC, 0x00, 0xCC, 0x33, 0xCC, 0x33, 0x00, 0xFF, 0x00, 0xFF, 0x00}}; space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/34603/5/src/mainboard/asus/h110m-e_... PS5, Line 35: const u8 dqs_map[2][8] = {{0, 1, 3, 2, 4, 5, 6, 7}, {1, 0, 4, 5, 2, 3, 6, 7}}; space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/34603/5/src/mainboard/asus/h110m-e_... PS5, Line 58: { that open brace { should be on the previous line