build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33842 )
Change subject: src/superio/nuvoton: Add support for NCT5539D ......................................................................
Patch Set 19:
(3 comments)
https://review.coreboot.org/c/coreboot/+/33842/19/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/romstage.c:
https://review.coreboot.org/c/coreboot/+/33842/19/src/mainboard/asus/h110m-e... PS19, 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/+/33842/19/src/mainboard/asus/h110m-e... PS19, 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/+/33842/19/src/mainboard/asus/h110m-e... PS19, Line 58: { that open brace { should be on the previous line