[PATCH] iio: adc: max1363: Drop provision to provide an IIO channel map via platform data

[PATCH] iio: adc: max1363: Drop provision to provide an IIO channel map via platform data

[Jonathan Cameron]

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Back in the days of board files, platform data was used to provide
information on the mapping from ADC channel to an analog signal
from another device. We've long since moved to doing this via
device tree. Hence drop the support from the max1363 driver which is
the only driver still providing this.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 drivers/iio/adc/max1363.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/iio/adc/max1363.c b/drivers/iio/adc/max1363.c
index eef55ed4814a..a28cf86cdce8 100644
--- a/drivers/iio/adc/max1363.c
+++ b/drivers/iio/adc/max1363.c
@@ -29,7 +29,6 @@
 #include <linux/iio/sysfs.h>
 #include <linux/iio/events.h>
 #include <linux/iio/buffer.h>
-#include <linux/iio/driver.h>
 #include <linux/iio/kfifo_buf.h>
 #include <linux/iio/trigger_consumer.h>
 #include <linux/iio/triggered_buffer.h>
@@ -1595,11 +1594,6 @@ static int max1363_probe(struct i2c_client *client,
 	if (!indio_dev)
 		return -ENOMEM;
 
-	ret = devm_iio_map_array_register(&client->dev, indio_dev,
-					  client->dev.platform_data);
-	if (ret < 0)
-		return ret;
-
 	st = iio_priv(indio_dev);
 
 	mutex_init(&st->lock);
-- 
2.37.2

Re: [PATCH] iio: adc: max1363: Drop provision to provide an IIO channel map via platform data

[Jonathan Cameron]

On Sun, 21 Aug 2022 17:10:58 +0100
Jonathan Cameron <jic23@kernel.org> wrote:

> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> Back in the days of board files, platform data was used to provide
> information on the mapping from ADC channel to an analog signal
> from another device. We've long since moved to doing this via
> device tree. Hence drop the support from the max1363 driver which is
> the only driver still providing this.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

This one is trivial enough that I'll take it without review.

Applied to the togreg branch of iio.git and pushed out as testing
for 0-day to see if I'm wrong :)

Thanks,

Jonathan

> ---
>  drivers/iio/adc/max1363.c | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/drivers/iio/adc/max1363.c b/drivers/iio/adc/max1363.c
> index eef55ed4814a..a28cf86cdce8 100644
> --- a/drivers/iio/adc/max1363.c
> +++ b/drivers/iio/adc/max1363.c
> @@ -29,7 +29,6 @@
>  #include <linux/iio/sysfs.h>
>  #include <linux/iio/events.h>
>  #include <linux/iio/buffer.h>
> -#include <linux/iio/driver.h>
>  #include <linux/iio/kfifo_buf.h>
>  #include <linux/iio/trigger_consumer.h>
>  #include <linux/iio/triggered_buffer.h>
> @@ -1595,11 +1594,6 @@ static int max1363_probe(struct i2c_client *client,
>  	if (!indio_dev)
>  		return -ENOMEM;
>  
> -	ret = devm_iio_map_array_register(&client->dev, indio_dev,
> -					  client->dev.platform_data);
> -	if (ret < 0)
> -		return ret;
> -
>  	st = iio_priv(indio_dev);
>  
>  	mutex_init(&st->lock);