Hi, the patch is looks fine to me, except these lines: > @@ -414,8 +452,8 @@ > > static u32 pcf_func(struct i2c_adapter *adap) > { > - return I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL | > - I2C_FUNC_10BIT_ADDR | I2C_FUNC_PROTOCOL_MANGLING; > + return I2C_FUNC_SMBUS_EMUL | I2C_FUNC_10BIT_ADDR | > + I2C_FUNC_PROTOCOL_MANGLING; > } Why you've removed I2C_FUNC_I2C? Regards, Oleg. P.S. I'm currently not subscribed to the list, so please CC me.