RE: [PATCH 4/4] iio: adc: xilinx: Use devm_ functions while requesting irq
From: Manish Narani
Date: Mon Jul 23 2018 - 05:27:34 EST
Hi Lars,
> -----Original Message-----
> From: Lars-Peter Clausen [mailto:lars@xxxxxxxxxx]
> Sent: Thursday, July 19, 2018 10:08 PM
> To: Manish Narani <MNARANI@xxxxxxxxxx>; jic23@xxxxxxxxxx;
> knaack.h@xxxxxx; pmeerw@xxxxxxxxxx; Michal Simek
> <michals@xxxxxxxxxx>; linux-iio@xxxxxxxxxxxxxxx; linux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Cc: Anirudha Sarangi <anirudh@xxxxxxxxxx>; Srinivas Goud
> <sgoud@xxxxxxxxxx>
> Subject: Re: [PATCH 4/4] iio: adc: xilinx: Use devm_ functions while requesting
> irq
>
> > @@ -1310,7 +1308,6 @@ static int xadc_remove(struct platform_device
> > *pdev) {
> > struct iio_dev *indio_dev = platform_get_drvdata(pdev);
> > struct xadc *xadc = iio_priv(indio_dev);
> > - int irq = platform_get_irq(pdev, 0);
> >
> > iio_device_unregister(indio_dev);
> > if (xadc->ops->flags & XADC_FLAGS_BUFFERED) { @@ -1318,7 +1315,6
> @@
> > static int xadc_remove(struct platform_device *pdev)
> > iio_trigger_free(xadc->convst_trigger);
> > iio_triggered_buffer_cleanup(indio_dev);
> > }
> > - free_irq(irq, indio_dev);
>
> This opens up a race condition. The IRQ needs to be freed before any of these
> other things below the free_irq() are executed.
Will look into it and send with taking care of the same.
>
> > clk_disable_unprepare(xadc->clk);
> > cancel_delayed_work(&xadc->zynq_unmask_work);
> > kfree(xadc->data);
> >
Thanks,
Manish