On 05/07/17 23:49, Andy Shevchenko wrote: > On Wed, Jul 5, 2017 at 1:13 PM, Chris Packham > <chris.packham@xxxxxxxxxxxxxxxxxxx> wrote: >> This series addresses some of the points identified by Andy. The series is >> based on top of i2c/for-next. 2 of the patches from v1 have already been >> applied so I've dropped them from this version. >> >> I have compile tested patch 1/1 but I don't have access to sh hardware >> to actually test the changes on. > > > Entire series looks good to me: > Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > > Further improvement is to get timeout value via device properties API > and get rid of legacy platform data completely. This can be done in > further series. Actually the timeout can probably go now. The two in-tree platforms that set it via platform_data set it to HZ which is (now) the default. > >> >> Chris Packham (4): >> sh: sh7785lcr: add GPIO lookup table for i2c controller reset >> i2c: pca-platform: unconditionally use devm_gpiod_get_optional >> i2c: pca-platform: use device_property_read_u32 >> i2c: pca-platform: drop gpio from platform data >> >> arch/blackfin/mach-bf561/boards/acvilon.c | 1 - >> arch/sh/boards/board-sh7785lcr.c | 11 +++++++++- >> drivers/i2c/busses/i2c-pca-platform.c | 34 +++++++++---------------------- >> include/linux/i2c-pca-platform.h | 3 --- >> 4 files changed, 20 insertions(+), 29 deletions(-) >> >> -- >> 2.13.0 >> > > >
