Re: [PATCHv6 07/11] i2c: match dt-style device names from sysfs interface

From: Lee Jones
Date: Wed Oct 26 2016 - 04:53:17 EST


On Tue, 25 Oct 2016, Kieran Bingham wrote:

> A user can choose to instantiate a device on an i2c bus using the sysfs
> interface by providing a string and address to match and communicate
> with the device on the bus. Presently this string is only matched
> against the old i2c device id style strings, even in the presence of
> full device tree compatible strings with vendor prefixes.
>
> Providing a vendor-prefixed string to the sysfs interface will not match
> against the device tree of_match_device() calls as there is no device
> tree node to parse from the sysfs interface.
>
> Convert i2c_of_match_device_strip_vendor() such that it can match both
> vendor prefixed and stripped compatible strings on the sysfs interface.
>
> Signed-off-by: Kieran Bingham <kieran@xxxxxxxxxxx>

Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>

> ---
> drivers/i2c/i2c-core.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
> index 2003334b5246..01bce56f733a 100644
> --- a/drivers/i2c/i2c-core.c
> +++ b/drivers/i2c/i2c-core.c
> @@ -1692,6 +1692,16 @@ i2c_of_match_device_strip_vendor(const struct of_device_id *matches,
> const char *name;
>
> for (; matches->compatible[0]; matches++) {
> + /*
> + * Adding devices through the i2c sysfs interface provides us
> + * a string to match which may be compatible with the device
> + * tree compatible strings, however with no actual of_node the
> + * of_match_device() will not match
> + */
> + if (!strncasecmp(client->name, matches->compatible,
> + strlen(matches->compatible)))
> + return matches;
> +
> name = strchr(matches->compatible, ',');
> if (!name)
> name = matches->compatible;

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog