On 06/10/2016 10:04 AM, Wolfram Sang wrote: > On Thu, Jun 09, 2016 at 04:06:03PM +0300, Crestez Dan Leonard wrote: >> When devices are instatiated through devicetree the i2c_client->name is >> set to the compatible string with company name stripped out. This is >> then matched to the i2c_device_id table to pass the device_id to the >> probe function. This id parameter is used by some device drivers to >> differentiate between model numbers. >> >> When using ACPI this id parameter is NULL and the driver usually needs >> to do ACPI-specific differentiation. >> >> This patch attempts to find a valid i2c_device_id when using ACPI with >> DT-like compatible strings. > > Note that this DT behaviour is about to be dropped to match I2C with > the "generic" behaviour". > > https://lkml.org/lkml/2016/5/4/534 Looking at that series it seems that the intention is to eventually remove the i2c_device_id argument from probe completely? That would cause a lot of code churn. It would also require every driver that needs to differentiate between models to pretty much duplicate the matching logic performed by the core. This does seem better than receiving an i2c_device_id parameter argument which may or may not be NULL. Still, in order to support multiple models using ACPI DT ids every driver would have to attempt some sort of acpi_of_match_device, right? Have you considered adding .probe_of(dev, of_device_id) and .probe_acpi(dev, acpi_device_id) instead, with arguments which are always guaranteed to be non-NULL? The main advantage would be that drivers don't need to do their own matching and all rules are only present ever in the core. Then ACPI with DT ids could be made to "just work" without per-driver support. -- Regards, Leonard -- 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
