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 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46144/2/src/drivers/i2c/gpiomux/bus... File src/drivers/i2c/gpiomux/bus/bus.c:
https://review.coreboot.org/c/coreboot/+/46144/2/src/drivers/i2c/gpiomux/bus... PS2, Line 20: malloc
Just curious: You used a static buffer in earlier CL, but malloc here. […]
I wasn't sure if the names of one of the mux adapters always overrides the other one because of the static buffer. Hence used buffer from the heap.
I verified that the names are distinct and are not overwritten. Hence updated to the static buffer here.