On Tue, 25 May 2021 22:07:17 +0200, Heiner Kallweit wrote: > i801_setup_hstcfg() leaves the bits in priv->original_hstcfg that > we're interested in intact. Therefore we can remove the return value > from the function and use priv->original_hstcfg directly. > > Signed-off-by: Heiner Kallweit <hkallweit1@xxxxxxxxx> > --- > drivers/i2c/busses/i2c-i801.c | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) > (...) I like it. Reviewed-by: Jean Delvare <jdelvare@xxxxxxx> Tested-by: Jean Delvare <jdelvare@xxxxxxx> -- Jean Delvare SUSE L3 Support
