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