diff mbox series

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

Message ID 20220821161058.2207185-1-jic23@kernel.org (mailing list archive)
State Accepted
Headers show
Series iio: adc: max1363: Drop provision to provide an IIO channel map via platform data | expand

Commit Message

Jonathan Cameron Aug. 21, 2022, 4:10 p.m. UTC
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(-)

Comments

Jonathan Cameron Sept. 4, 2022, 4:40 p.m. UTC | #1
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);
diff mbox series

Patch

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);