Re: [PATCH 15/15] iio: sx9310: use the data of the I2C table

[Jonathan Cameron <jic23@kernel.org>]

On Tue, 27 Oct 2026 17:19:19 +0100
Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote:

> From: Josef Bacik <josef@toxicpanda.com>
> 
> sx9310_check_whoami() takes the chip information from
> device_get_match_data(), which only looks at the OF and ACPI tables, and
> dereferences it. For a client which was bound through the i2c_device_id
> table, for instance one created with
> 
>   echo sx9310 0x28 > /sys/bus/i2c/devices/i2c-1/new_device
> 
> it returns NULL, and the kernel oopses as soon as something at that address
> answers the read of the WHOAMI register.
> 
> The I2C table has the same pointers. Use i2c_get_match_data(), which falls
> back to it.
> 
> Fixes: 21fd3b1373f6 ("iio: proximity: sx9310: Convert enum->pointer for match data table")
> Cc: stable <stable@kernel.org>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: David Lechner <dlechner@baylibre.com>
> Cc: "Nuno Sá" <nuno.sa@analog.com>
> Cc: Andy Shevchenko <andy@kernel.org>
> Cc: "Uwe Kleine-König (The Capable Hub)" <u.kleine-koenig@baylibre.com>
> Cc: Siratul Islam <email@sirat.me>
> Cc: Matti Vaittinen <mazziesaccount@gmail.com>
> Cc: Biju Das <biju.das.jz@bp.renesas.com>
> Cc: Stephen Boyd <swboyd@chromium.org>
Applied to the fixes-togreg branch (I'll rebase and send a pull
for that after rc1), though I pruned most of these out to reduce noise
in the git log

Thanks,

Jonathan

> Signed-off-by: Josef Bacik <josef@toxicpanda.com>
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> ---
>  drivers/iio/proximity/sx9310.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/proximity/sx9310.c b/drivers/iio/proximity/sx9310.c
> index 79ba46d8a0aa..2585b9c6fb69 100644
> --- a/drivers/iio/proximity/sx9310.c
> +++ b/drivers/iio/proximity/sx9310.c
> @@ -891,7 +891,7 @@ static int sx9310_check_whoami(struct device *dev,
>  	if (ret)
>  		return ret;
>  
> -	ddata = device_get_match_data(dev);
> +	ddata = i2c_get_match_data(data->client);
>  	if (ddata->whoami != whoami)
>  		return -ENODEV;
>  
>