lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 27 Apr 2019 14:08:40 +0100 From: Jonathan Cameron <jic23@...nel.org> To: Fabrice Gasnier <fabrice.gasnier@...com> Cc: Fabien Dessenne <fabien.dessenne@...com>, Hartmut Knaack <knaack.h@....de>, Lars-Peter Clausen <lars@...afoo.de>, Peter Meerwald-Stadler <pmeerw@...erw.net>, Maxime Coquelin <mcoquelin.stm32@...il.com>, Alexandre Torgue <alexandre.torgue@...com>, <linux-iio@...r.kernel.org>, <linux-stm32@...md-mailman.stormreply.com>, <linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org> Subject: Re: [PATCH 1/2] iio: adc: stm32-dfsdm: manage the get_irq error case On Wed, 24 Apr 2019 14:55:09 +0200 Fabrice Gasnier <fabrice.gasnier@...com> wrote: > On 4/24/19 2:51 PM, Fabien Dessenne wrote: > > During probe, check the "get_irq" error value. > > > > Signed-off-by: Fabien Dessenne <fabien.dessenne@...com> > > --- > > Hi Fabien, > > Acked-by: Fabrice Gasnier <fabrice.gasnier@...com> Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > > Thanks, > Fabrice > > > drivers/iio/adc/stm32-dfsdm-adc.c | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/drivers/iio/adc/stm32-dfsdm-adc.c b/drivers/iio/adc/stm32-dfsdm-adc.c > > index 19adc2b..588907c 100644 > > --- a/drivers/iio/adc/stm32-dfsdm-adc.c > > +++ b/drivers/iio/adc/stm32-dfsdm-adc.c > > @@ -1456,6 +1456,12 @@ static int stm32_dfsdm_adc_probe(struct platform_device *pdev) > > * So IRQ associated to filter instance 0 is dedicated to the Filter 0. > > */ > > irq = platform_get_irq(pdev, 0); > > + if (irq < 0) { > > + if (irq != -EPROBE_DEFER) > > + dev_err(dev, "Failed to get IRQ: %d\n", irq); > > + return irq; > > + } > > + > > ret = devm_request_irq(dev, irq, stm32_dfsdm_irq, > > 0, pdev->name, adc); > > if (ret < 0) { > >
Powered by blists - more mailing lists