Re: [PATCH v1 1/1] iio: proximity: sx_common: Allow IIO core to take care of firmware node

[Jonathan Cameron <jic23@kernel.org>]

On Mon, 30 May 2022 20:43:26 +0300
Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:

> IIO core correctly will take care of firmware node if it's not set in
> the driver. Drop ACPI and OF specifics from the driver and allow IIO
> core to handle this.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Looks fine to me.  As such I'll apply it now, but Gwendal, you've
been active with this driver recently so if you have time to sanity check
that would be great.  Once I've caught up with new stuff I plan to check
where we are with your various series on this driver.

Thanks,

Jonathan

> ---
>  drivers/iio/proximity/sx_common.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/iio/proximity/sx_common.c b/drivers/iio/proximity/sx_common.c
> index 8ad814d96b7e..6bb68c1835da 100644
> --- a/drivers/iio/proximity/sx_common.c
> +++ b/drivers/iio/proximity/sx_common.c
> @@ -5,7 +5,6 @@
>   * Common part of most Semtech SAR sensor.
>   */
>  
> -#include <linux/acpi.h>
>  #include <linux/bitops.h>
>  #include <linux/byteorder/generic.h>
>  #include <linux/delay.h>
> @@ -520,8 +519,6 @@ int sx_common_probe(struct i2c_client *client,
>  	if (ret)
>  		return dev_err_probe(dev, ret, "error reading WHOAMI\n");
>  
> -	ACPI_COMPANION_SET(&indio_dev->dev, ACPI_COMPANION(dev));
> -	indio_dev->dev.of_node = client->dev.of_node;
>  	indio_dev->modes = INDIO_DIRECT_MODE;
>  
>  	indio_dev->channels =  data->chip_info->iio_channels;