> A newline was missing and closing braces of functions do not need a > semicolon. Can there be a need to offer such changes by separate update steps? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.10-rc6#n81 … > +++ b/drivers/i2c/busses/i2c-rcar.c > @@ -192,7 +192,7 @@ static int rcar_i2c_get_scl(struct i2c_adapter *adap) > > return !!(rcar_i2c_read(priv, ICMCR) & FSCL); > > -}; > +} … How do you think about to omit any blank lines at such source code places? Regards, Markus
