build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/29476 )
Change subject: ec/kontron: Add support for Kontron kempld ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/29476/1/src/ec/kontron/kempld/kempld_i2c.c File src/ec/kontron/kempld/kempld_i2c.c:
https://review.coreboot.org/#/c/29476/1/src/ec/kontron/kempld/kempld_i2c.c@1... PS1, Line 169: } else { else is not generally useful after a break or return
https://review.coreboot.org/#/c/29476/1/src/ec/kontron/kempld/kempld_i2c.c@2... PS1, Line 214: printk(BIOS_SPEW, "kempld_i2c: Processed %zu segments.\n", num); line over 80 characters