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 ��.n��������+%������w��{.n�����{��(��)��jg��������ݢj����G�������j:+v���w�m������w�������h�����٥