Message ID | 20210926162859.3567685-1-aardelean@deviqon.com (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | iio: adc: rn5t618-adc: use devm_iio_map_array_register() function | expand |
On Sun, 26 Sep 2021 19:28:59 +0300 Alexandru Ardelean <aardelean@deviqon.com> wrote: > This driver already hooks a similar unwind callback via > devm_add_action_or_reset(). > > They pretty much do the same thing, so this change converts it to the > devm_iio_map_array_register(). > > Signed-off-by: Alexandru Ardelean <aardelean@deviqon.com> > --- > drivers/iio/adc/rn5t618-adc.c | 13 +------------ > 1 file changed, 1 insertion(+), 12 deletions(-) yes, looks sane, the patches it depens on are not in linux-next yet. But it seems that they are accepted, so: Reviewed-by: Andreas Kemnade <andreas@kemnade.info>
On Mon, 27 Sep 2021 22:08:34 +0200 Andreas Kemnade <andreas@kemnade.info> wrote: > On Sun, 26 Sep 2021 19:28:59 +0300 > Alexandru Ardelean <aardelean@deviqon.com> wrote: > > > This driver already hooks a similar unwind callback via > > devm_add_action_or_reset(). > > > > They pretty much do the same thing, so this change converts it to the > > devm_iio_map_array_register(). > > > > Signed-off-by: Alexandru Ardelean <aardelean@deviqon.com> > > --- > > drivers/iio/adc/rn5t618-adc.c | 13 +------------ > > 1 file changed, 1 insertion(+), 12 deletions(-) > > yes, looks sane, the patches it depens on are not in linux-next yet. I was waiting for 0-day confirmation which for some reason has gone astray (not problems reported either). Ah well - I'll assume it's fine and risk next breakage. > But it seems that they are accepted, so: > > Reviewed-by: Andreas Kemnade <andreas@kemnade.info> Thanks and applied to the togreg branch of iio.git, pushed out as testing to see if 0-day notices this time. Thanks, Jonathan
diff --git a/drivers/iio/adc/rn5t618-adc.c b/drivers/iio/adc/rn5t618-adc.c index c56fccb2c8e1..7d891b4ea461 100644 --- a/drivers/iio/adc/rn5t618-adc.c +++ b/drivers/iio/adc/rn5t618-adc.c @@ -197,13 +197,6 @@ static struct iio_map rn5t618_maps[] = { { /* sentinel */ } }; -static void unregister_map(void *data) -{ - struct iio_dev *iio_dev = (struct iio_dev *) data; - - iio_map_array_unregister(iio_dev); -} - static int rn5t618_adc_probe(struct platform_device *pdev) { int ret; @@ -254,11 +247,7 @@ static int rn5t618_adc_probe(struct platform_device *pdev) return ret; } - ret = iio_map_array_register(iio_dev, rn5t618_maps); - if (ret < 0) - return ret; - - ret = devm_add_action_or_reset(adc->dev, unregister_map, iio_dev); + ret = devm_iio_map_array_register(adc->dev, iio_dev, rn5t618_maps); if (ret < 0) return ret;
This driver already hooks a similar unwind callback via devm_add_action_or_reset(). They pretty much do the same thing, so this change converts it to the devm_iio_map_array_register(). Signed-off-by: Alexandru Ardelean <aardelean@deviqon.com> --- drivers/iio/adc/rn5t618-adc.c | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-)