Messages in this thread | | | Date | Sun, 29 Jul 2018 13:30:50 +0100 | From | Jonathan Cameron <> | Subject | Re: [PATCH v2 4/4] iio: adc: xilinx: Move request_irq before enabling interrupts |
| |
On Mon, 23 Jul 2018 20:32:03 +0530 Manish Narani <manish.narani@xilinx.com> wrote:
> Enabling the Interrupts before registering the irq handler is a bad > idea. This patch corrects the same for XADC driver. > > Signed-off-by: Manish Narani <manish.narani@xilinx.com> Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it.
Thanks,
Jonathan > --- > drivers/iio/adc/xilinx-xadc-core.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/iio/adc/xilinx-xadc-core.c b/drivers/iio/adc/xilinx-xadc-core.c > index 44a2519..3f6be5a 100644 > --- a/drivers/iio/adc/xilinx-xadc-core.c > +++ b/drivers/iio/adc/xilinx-xadc-core.c > @@ -1226,15 +1226,15 @@ static int xadc_probe(struct platform_device *pdev) > if (ret) > goto err_free_samplerate_trigger; > > - ret = xadc->ops->setup(pdev, indio_dev, xadc->irq); > - if (ret) > - goto err_clk_disable_unprepare; > - > ret = request_irq(xadc->irq, xadc->ops->interrupt_handler, 0, > dev_name(&pdev->dev), indio_dev); > if (ret) > goto err_clk_disable_unprepare; > > + ret = xadc->ops->setup(pdev, indio_dev, xadc->irq); > + if (ret) > + goto err_free_irq; > + > for (i = 0; i < 16; i++) > xadc_read_adc_reg(xadc, XADC_REG_THRESHOLD(i), > &xadc->threshold[i]);
| |