build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33009 )
Change subject: superio/smsc/smscsuperio: Add SCH5317 ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/33009/1/src/superio/smsc/smscsuperio/superio... File src/superio/smsc/smscsuperio/superio.c:
https://review.coreboot.org/#/c/33009/1/src/superio/smsc/smscsuperio/superio... PS1, Line 140: {SCH5317, {0, 3, 4, 5, -1, 7, -1, -1, 8, -1, -1, -1, 10, -1, -1}}, // TODO: check correctness line over 80 characters
https://review.coreboot.org/#/c/33009/1/src/superio/smsc/smscsuperio/superio... PS1, Line 140: {SCH5317, {0, 3, 4, 5, -1, 7, -1, -1, 8, -1, -1, -1, 10, -1, -1}}, // TODO: check correctness space required after that close brace '}'