Messages in this thread | | | Date | Tue, 29 Apr 2008 01:59:44 -0400 | From | Jeff Garzik <> | Subject | Re: [PATCH] ARM: am79c961a: platform_get_irq() may return signed unnoticed |
| |
Roel Kluin wrote: > dev->irq is unsigned, platform_get_irq() may return signed unnoticed > > Signed-off-by: Roel Kluin <12o3l@tiscali.nl> > --- > diff --git a/drivers/net/arm/am79c961a.c b/drivers/net/arm/am79c961a.c > index ba6bd03..a637910 100644 > --- a/drivers/net/arm/am79c961a.c > +++ b/drivers/net/arm/am79c961a.c > @@ -693,11 +693,15 @@ static int __init am79c961_probe(struct platform_device *pdev) > * done by the ether bootp loader. > */ > dev->base_addr = res->start; > - dev->irq = platform_get_irq(pdev, 0); > + ret = platform_get_irq(pdev, 0); > > - ret = -ENODEV; > - if (dev->irq < 0) > + if (ret < 0) { > + ret = -ENODEV; > goto nodev; > + } > + dev->irq = ret; > + > + ret = -ENODEV; > if (!request_region(dev->base_addr, 0x18, dev->name)) > goto nodev;
applied
| |