Hi Andy, Thanks for the patch. I have verified this patch on 4.13-rc1 kernel for on my TPS68470 PMIC v4 patch. Feel free to add Tested-by: Rajmohan Mani <rajmohan.mani@xxxxxxxxx> Raj > -----Original Message----- > From: linux-acpi-owner@xxxxxxxxxxxxxxx [mailto:linux-acpi- > owner@xxxxxxxxxxxxxxx] On Behalf Of Andy Shevchenko > Sent: Monday, July 17, 2017 7:13 AM > To: Wolfram Sang <wsa@xxxxxxxxxxxxx>; linux-i2c@xxxxxxxxxxxxxxx; Mika > Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>; Hans de Goede > <hdegoede@xxxxxxxxxx>; Rafael J . Wysocki <rjw@xxxxxxxxxxxxx>; linux- > acpi@xxxxxxxxxxxxxxx > Cc: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > Subject: [Resend, PATCH v2] i2c: core: Allow empty id_table in ACPI case as > well > > For now empty ID table is not allowed with ACPI and prevents driver to be > probed. > > Add a check to allow empty ID table. > > This introduces a helper i2c_acpi_match_device(). > > Note, we rename some static function in i2c-core-acpi.c to distinguish with > public API. > > Fixes: da10c06a044b ("i2c: Make I2C ID tables non-mandatory for DT'ed > devices") > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > - twice linux-i2c@ instead of i2c and acpi MLs, thus, resent drivers/i2c/i2c- > core-acpi.c | 19 +++++++++++++++---- drivers/i2c/i2c-core-base.c | 1 + > drivers/i2c/i2c-core.h | 9 +++++++++ > 3 files changed, 25 insertions(+), 4 deletions(-) > > diff --git a/drivers/i2c/i2c-core-acpi.c b/drivers/i2c/i2c-core-acpi.c index > 4842ec3a5451..a9126b3cda61 100644 > --- a/drivers/i2c/i2c-core-acpi.c > +++ b/drivers/i2c/i2c-core-acpi.c > @@ -230,6 +230,16 @@ void i2c_acpi_register_devices(struct i2c_adapter > *adap) > dev_warn(&adap->dev, "failed to enumerate I2C slaves\n"); > } > > +const struct acpi_device_id * > +i2c_acpi_match_device(const struct acpi_device_id *matches, > + struct i2c_client *client) > +{ > + if (!(client && matches)) > + return NULL; > + > + return acpi_match_device(matches, &client->dev); } > + > static acpi_status i2c_acpi_lookup_speed(acpi_handle handle, u32 level, > void *data, void **return_value) { > @@ -289,7 +299,7 @@ u32 i2c_acpi_find_bus_speed(struct device *dev) } > EXPORT_SYMBOL_GPL(i2c_acpi_find_bus_speed); > > -static int i2c_acpi_match_adapter(struct device *dev, void *data) > +static int i2c_acpi_find_match_adapter(struct device *dev, void *data) > { > struct i2c_adapter *adapter = i2c_verify_adapter(dev); > > @@ -299,7 +309,7 @@ static int i2c_acpi_match_adapter(struct device *dev, > void *data) > return ACPI_HANDLE(dev) == (acpi_handle)data; } > > -static int i2c_acpi_match_device(struct device *dev, void *data) > +static int i2c_acpi_find_match_device(struct device *dev, void *data) > { > return ACPI_COMPANION(dev) == data; > } > @@ -309,7 +319,7 @@ static struct i2c_adapter > *i2c_acpi_find_adapter_by_handle(acpi_handle handle) > struct device *dev; > > dev = bus_find_device(&i2c_bus_type, NULL, handle, > - i2c_acpi_match_adapter); > + i2c_acpi_find_match_adapter); > return dev ? i2c_verify_adapter(dev) : NULL; } > > @@ -317,7 +327,8 @@ static struct i2c_client > *i2c_acpi_find_client_by_adev(struct acpi_device *adev) { > struct device *dev; > > - dev = bus_find_device(&i2c_bus_type, NULL, adev, > i2c_acpi_match_device); > + dev = bus_find_device(&i2c_bus_type, NULL, adev, > + i2c_acpi_find_match_device); > return dev ? i2c_verify_client(dev) : NULL; } > > diff --git a/drivers/i2c/i2c-core-base.c b/drivers/i2c/i2c-core-base.c index > c89dac7fd2e7..12822a4b8f8f 100644 > --- a/drivers/i2c/i2c-core-base.c > +++ b/drivers/i2c/i2c-core-base.c > @@ -357,6 +357,7 @@ static int i2c_device_probe(struct device *dev) > * Tree match table entry is supplied for the probing device. > */ > if (!driver->id_table && > + !i2c_acpi_match_device(dev->driver->acpi_match_table, client) && > !i2c_of_match_device(dev->driver->of_match_table, client)) > return -ENODEV; > > diff --git a/drivers/i2c/i2c-core.h b/drivers/i2c/i2c-core.h index > 3b63f5e5b89c..3d3d9bf02101 100644 > --- a/drivers/i2c/i2c-core.h > +++ b/drivers/i2c/i2c-core.h > @@ -31,9 +31,18 @@ int i2c_check_addr_validity(unsigned addr, unsigned > short flags); int i2c_check_7bit_addr_validity_strict(unsigned short addr); > > #ifdef CONFIG_ACPI > +const struct acpi_device_id * > +i2c_acpi_match_device(const struct acpi_device_id *matches, > + struct i2c_client *client); > void i2c_acpi_register_devices(struct i2c_adapter *adap); #else /* > CONFIG_ACPI */ static inline void i2c_acpi_register_devices(struct > i2c_adapter *adap) { } > +static inline const struct acpi_device_id * i2c_acpi_match_device(const > +struct acpi_device_id *matches, > + struct i2c_client *client) > +{ > + return NULL; > +} > #endif /* CONFIG_ACPI */ > extern struct notifier_block i2c_acpi_notifier; > > -- > 2.11.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the > body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at > http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
