> @@ -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.