build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43614 )
Change subject: drivers/i2c/tas5825m: Add driver for TI TAS5825M amplifier ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/43614/4/src/drivers/i2c/tas5825m/ch... File src/drivers/i2c/tas5825m/chip.h:
https://review.coreboot.org/c/coreboot/+/43614/4/src/drivers/i2c/tas5825m/ch... PS4, Line 5: int id; please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/43614/4/src/drivers/i2c/tas5825m/ta... File src/drivers/i2c/tas5825m/tas5825m.c:
https://review.coreboot.org/c/coreboot/+/43614/4/src/drivers/i2c/tas5825m/ta... PS4, Line 56: if (res) { braces {} are not necessary for any arm of this statement