Hi Andy, On Mon, 31 Jan 2022 17:13:44 +0200, Andy Shevchenko wrote: > Since we have a common P2SB accessor in tree we may use it instead of > open coded variants. > > Replace custom code by p2sb_bar() call. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > Acked-by: Hans de Goede <hdegoede@xxxxxxxxxx> > Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > drivers/i2c/busses/Kconfig | 1 + > drivers/i2c/busses/i2c-i801.c | 39 +++++++------------------------ > drivers/platform/x86/intel/p2sb.c | 6 +++++ > 3 files changed, 16 insertions(+), 30 deletions(-) > (...) Reviewed-by: Jean Delvare <jdelvare@xxxxxxx> And thank you for taking the time to write this neat P2SB API and to convert all the code that was doing the same so far. -- Jean Delvare SUSE L3 Support