Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46144 )
Change subject: drivers/i2c/gpiomux: Add chip driver for multiplexed I2C bus ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/46144/1/src/drivers/i2c/gpiomux/bus... File src/drivers/i2c/gpiomux/bus/bus.c:
https://review.coreboot.org/c/coreboot/+/46144/1/src/drivers/i2c/gpiomux/bus... PS1, Line 14: dev->chip_info
config_of(dev)?
Done. Here and in the mux driver: https://review.coreboot.org/c/coreboot/+/45911/7/src/drivers/i2c/gpiomux/mux...
https://review.coreboot.org/c/coreboot/+/46144/1/src/drivers/i2c/gpiomux/bus... PS1, Line 22: name[ACPI_NAME_BUFFER_SIZE - 1] = '\0';
snprintf() already takes care of doing this.
Done. Here and in the mux driver: https://review.coreboot.org/c/coreboot/+/45911/7/src/drivers/i2c/gpiomux/mux...
https://review.coreboot.org/c/coreboot/+/46144/1/src/drivers/i2c/gpiomux/bus... PS1, Line 60: if (!config) : return; :
I don't understand the intent of this check.
Done. My bad. The new dev check is to satisfy the coverity.