Hi, On Fri, Mar 10, 2023 at 03:10:46PM +0800, liao.zenghui@xxxxxxxxxx wrote: > static inline void __i2c_dw_enable(struct dw_i2c_dev *dev) > { > + u32 enabled; > + > dev->status |= STATUS_ACTIVE; > - regmap_write(dev->map, DW_IC_ENABLE, 1); > + regmap_read(dev->map, DW_IC_ENABLE, &enabled); > + regmap_write(dev->map, DW_IC_ENABLE, enabled | DW_IC_ENABLE_ENABLE); Looks like this patch has some whitespace damage. Probably good to use git send-email or so to make sure the formatting stays intact.