Re: [PATCHv7 08/11] i2c: match vendorless strings on the internal string length

From: Javier Martinez Canillas
Date: Mon Nov 07 2016 - 14:22:44 EST


Hello Kieran,

On 11/07/2016 09:47 AM, Kieran Bingham wrote:
> If a user provides a shortened string to match a device to the sysfs i2c
> interface it will match on the first string that contains that string
> prefix.
>
> for example:
> echo a 0x68 > /sys/bus/i2c/devices/i2c-2/new_device
> will match as3711, as3722, and ak8975 incorrectly.
>
> Correct this by using sysfs_streq to match the string exactly
>
> Signed-off-by: Kieran Bingham <kieran@xxxxxxxxxxx>
>

Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America