* [PATCH 04/15] drivers/net: Do not free an IRQ if its request failed [not found] <1334867804-31942-1-git-send-email-lee.jones@linaro.org> @ 2012-04-19 20:36 ` Lee Jones 2012-05-04 14:10 ` Lee Jones 0 siblings, 1 reply; 4+ messages in thread From: Lee Jones @ 2012-04-19 20:36 UTC (permalink / raw) To: linux-arm-kernel, arnd, linus.walleij, grant.likely, cjb, linux Cc: Lee Jones, netdev Refrain from attempting to free an interrupt line if the request fails and hence, there is no IRQ to free. CC: netdev@vger.kernel.org Signed-off-by: Lee Jones <lee.jones@linaro.org> --- drivers/net/ethernet/smsc/smsc911x.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/smsc/smsc911x.c b/drivers/net/ethernet/smsc/smsc911x.c index 4a69710..f17a76e 100644 --- a/drivers/net/ethernet/smsc/smsc911x.c +++ b/drivers/net/ethernet/smsc/smsc911x.c @@ -2382,7 +2382,6 @@ static int __devinit smsc911x_drv_probe(struct platform_device *pdev) SET_NETDEV_DEV(dev, &pdev->dev); pdata = netdev_priv(dev); - dev->irq = irq_res->start; irq_flags = irq_res->flags & IRQF_TRIGGER_MASK; pdata->ioaddr = ioremap_nocache(res->start, res_size); @@ -2446,7 +2445,7 @@ static int __devinit smsc911x_drv_probe(struct platform_device *pdev) if (retval) { SMSC_WARN(pdata, probe, "Unable to claim requested irq: %d", dev->irq); - goto out_free_irq; + goto out_disable_resources; } retval = register_netdev(dev); -- 1.7.9.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 04/15] drivers/net: Do not free an IRQ if its request failed 2012-04-19 20:36 ` [PATCH 04/15] drivers/net: Do not free an IRQ if its request failed Lee Jones @ 2012-05-04 14:10 ` Lee Jones 2012-05-04 18:22 ` Linus Walleij 0 siblings, 1 reply; 4+ messages in thread From: Lee Jones @ 2012-05-04 14:10 UTC (permalink / raw) To: Lee Jones Cc: linux-arm-kernel, arnd, linus.walleij, grant.likely, cjb, linux, netdev On 19/04/12 21:36, Lee Jones wrote: > Refrain from attempting to free an interrupt line if the request > fails and hence, there is no IRQ to free. > > CC: netdev@vger.kernel.org > Signed-off-by: Lee Jones<lee.jones@linaro.org> > --- > drivers/net/ethernet/smsc/smsc911x.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/ethernet/smsc/smsc911x.c b/drivers/net/ethernet/smsc/smsc911x.c > index 4a69710..f17a76e 100644 > --- a/drivers/net/ethernet/smsc/smsc911x.c > +++ b/drivers/net/ethernet/smsc/smsc911x.c > @@ -2382,7 +2382,6 @@ static int __devinit smsc911x_drv_probe(struct platform_device *pdev) > SET_NETDEV_DEV(dev,&pdev->dev); > > pdata = netdev_priv(dev); > - > dev->irq = irq_res->start; > irq_flags = irq_res->flags& IRQF_TRIGGER_MASK; > pdata->ioaddr = ioremap_nocache(res->start, res_size); > @@ -2446,7 +2445,7 @@ static int __devinit smsc911x_drv_probe(struct platform_device *pdev) > if (retval) { > SMSC_WARN(pdata, probe, > "Unable to claim requested irq: %d", dev->irq); > - goto out_free_irq; > + goto out_disable_resources; > } > > retval = register_netdev(dev); Anything on this from the Net guys? Kind regards, Lee -- Lee Jones Linaro ST-Ericsson Landing Team Lead M: +44 77 88 633 515 Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 04/15] drivers/net: Do not free an IRQ if its request failed 2012-05-04 14:10 ` Lee Jones @ 2012-05-04 18:22 ` Linus Walleij 2012-05-04 21:18 ` Lee Jones 0 siblings, 1 reply; 4+ messages in thread From: Linus Walleij @ 2012-05-04 18:22 UTC (permalink / raw) To: Lee Jones Cc: linux, linus.walleij, arnd, netdev, grant.likely, cjb, linux-arm-kernel On Fri, May 4, 2012 at 4:10 PM, Lee Jones <lee.jones@linaro.org> wrote: > On 19/04/12 21:36, Lee Jones wrote: >> - goto out_free_irq; >> + goto out_disable_resources; >> } >> >> retval = register_netdev(dev); > > > Anything on this from the Net guys? This was merged for 3.4-rc5, check: https://lkml.org/lkml/2012/4/29/2 Linus Walleij ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 04/15] drivers/net: Do not free an IRQ if its request failed 2012-05-04 18:22 ` Linus Walleij @ 2012-05-04 21:18 ` Lee Jones 0 siblings, 0 replies; 4+ messages in thread From: Lee Jones @ 2012-05-04 21:18 UTC (permalink / raw) To: Linus Walleij Cc: linux, linus.walleij, arnd, netdev, grant.likely, cjb, linux-arm-kernel On 04/05/12 19:22, Linus Walleij wrote: > On Fri, May 4, 2012 at 4:10 PM, Lee Jones<lee.jones@linaro.org> wrote: >> On 19/04/12 21:36, Lee Jones wrote: > >>> - goto out_free_irq; >>> + goto out_disable_resources; >>> } >>> >>> retval = register_netdev(dev); >> >> >> Anything on this from the Net guys? > > This was merged for 3.4-rc5, check: > https://lkml.org/lkml/2012/4/29/2 I actually found that out a few hours ago by accident. Thanks for letting me know though. Kind regards, Lee -- Lee Jones Linaro ST-Ericsson Landing Team Lead M: +44 77 88 633 515 Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-05-04 21:18 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1334867804-31942-1-git-send-email-lee.jones@linaro.org>
2012-04-19 20:36 ` [PATCH 04/15] drivers/net: Do not free an IRQ if its request failed Lee Jones
2012-05-04 14:10 ` Lee Jones
2012-05-04 18:22 ` Linus Walleij
2012-05-04 21:18 ` Lee Jones
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).