On Sat, 2016-12-10 at 23:43 +0100, Hans de Goede wrote: > If (!shared_host) simply return 0, this avoids delaying the probe if > iosf_mbi_available() returns false when an i2c bus is not using the > punit semaphore. > > Also move the if (!iosf_mbi_available()) check to above the > dev_info, so that we do not repeat the dev_info on every probe > until iosf_mbi_available() returns true. > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > --- > Changes in v2: > -New patch in v2 of this set > Changes in v3: > -Use if (!shared_host) return 0, to simplify the non-shared_host path > and to avoid nested ifs > --- > drivers/i2c/busses/i2c-designware-baytrail.c | 13 +++++++------ > 1 file changed, 7 insertions(+), 6 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-designware-baytrail.c > b/drivers/i2c/busses/i2c-designware-baytrail.c > index a3f581c..cf02222 100644 > --- a/drivers/i2c/busses/i2c-designware-baytrail.c > +++ b/drivers/i2c/busses/i2c-designware-baytrail.c > @@ -138,15 +138,16 @@ int i2c_dw_eval_lock_support(struct dw_i2c_dev > *dev) > if (ACPI_FAILURE(status)) > return 0; > > - if (shared_host) { > - dev_info(dev->dev, "I2C bus managed by PUNIT\n"); > - dev->acquire_lock = baytrail_i2c_acquire; > - dev->release_lock = baytrail_i2c_release; > - dev->pm_runtime_disabled = true; > - } > + if (!shared_host) > + return 0; > > if (!iosf_mbi_available()) > return -EPROBE_DEFER; > > + dev_info(dev->dev, "I2C bus managed by PUNIT\n"); > + dev->acquire_lock = baytrail_i2c_acquire; > + dev->release_lock = baytrail_i2c_release; > + dev->pm_runtime_disabled = true; > + > return 0; > } -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html