Attention is currently required from: Ahamed Husni.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67342 )
Change subject: drivers/smbus: SC16IS750 I2C to UART converter chip initialization ......................................................................
Patch Set 2: Verified-1
(3 comments)
File src/drivers/smbus/sc16is750_init.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-158066): https://review.coreboot.org/c/coreboot/+/67342/comment/5df79a5e_51ab2085 PS2, Line 7: do_smbus_write_byte(CONFIG_FIXED_SMBUS_IO_BASE, \ Avoid unnecessary line continuations
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-158066): https://review.coreboot.org/c/coreboot/+/67342/comment/a2d61ab9_b4b4fdaf PS2, Line 21: sc16is750_write_byte(REG_LCR << 3, 0b10000011); Avoid gcc v4.3+ binary constant extension: <0b10000011>
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-158066): https://review.coreboot.org/c/coreboot/+/67342/comment/b0206964_196e6c5b PS2, Line 27: sc16is750_write_byte(REG_LCR << 3, 0b00000011); Avoid gcc v4.3+ binary constant extension: <0b00000011>