Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45949 )
Change subject: soc/intel/common/block/i2c: Scan bridge devices behind I2C controllers ......................................................................
Patch Set 2: Code-Review+2
(1 comment)
https://review.coreboot.org/c/coreboot/+/45949/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/45949/2//COMMIT_MSG@9 PS2, Line 9: Scan bridge devices like I2C Mutiplexers behind I2C controllers using I think it would be good to provide some context in commit message that the original assumption was that there will only be non-bridge devices under the SoC I2C controller and hence it was using scan_generic_bus. However, we need to support I2C Multiplexers which act as bridges and have devices under them. That is why the change to scan_static_bus is required.