On Wed, Dec 09, 2020 at 09:52:42PM +0100, Wolfram Sang wrote: > > > @@ -243,7 +243,7 @@ struct dw_i2c_dev { > > struct clk *clk; > > struct clk *pclk; > > struct reset_control *rst; > > - struct i2c_client *slave; > > + struct i2c_client *slave; > > u32 (*get_clk_rate_khz) (struct dw_i2c_dev *dev); > > int cmd_err; > > struct i2c_msg *msgs; > > What about using just a single space after the type? Won't need an > update ever again. Maybe, but this is really so minor patch, so if you are okay with other two (that have more importance) drop this one. -- With Best Regards, Andy Shevchenko