Hi Chris, sorry for looking into this patch so late. On Fri, Jan 31, 2025 at 05:50:16PM +0000, Chris Babroski wrote: > Add support for SMBus repeated start conditions to the Mellanox I2C > driver. This support is specifically enabled for the > I2C_FUNC_SMBUS_WRITE_I2C_BLOCK implementation which is required for > communication with an I2C device on BlueField 3. > > The I2C bus timing configuration values have also been updated to fix > an issue found when testing block reads requiring a repeated start > condition. The "timeout" value, which determines how long the clock > can be held low before the I2C transaction is aborted, was too short > and caused unwanted failures. It was also found that other timing values > do not match the configuration that is recommended and has been verified by > the BlueField HW team so those values have been updated as well. Could we split this patches into two parts? It looks good to me, so that once you send it, I will apply it right away. Thanks, Andi