arcnet: fix signed bug in probe function
authorDan Carpenter <error27@gmail.com>
Sat, 17 Jul 2010 07:21:28 +0000 (07:21 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 18 Jul 2010 22:07:15 +0000 (15:07 -0700)
probe_irq_off() returns the first irq found or if two irqs are found
then it returns the negative of the first irq found.  We can cast
dev->irq to an int so that the test for negative values works.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/arcnet/com20020-isa.c
drivers/net/arcnet/com90io.c

index 0402da30a4ed22c9781a051a2b65cda98767bae2..37272827ee555cf87f6712069f2ba1410f6dbcee 100644 (file)
@@ -90,14 +90,14 @@ static int __init com20020isa_probe(struct net_device *dev)
                outb(0, _INTMASK);
                dev->irq = probe_irq_off(airqmask);
 
-               if (dev->irq <= 0) {
+               if ((int)dev->irq <= 0) {
                        BUGMSG(D_INIT_REASONS, "Autoprobe IRQ failed first time\n");
                        airqmask = probe_irq_on();
                        outb(NORXflag, _INTMASK);
                        udelay(5);
                        outb(0, _INTMASK);
                        dev->irq = probe_irq_off(airqmask);
-                       if (dev->irq <= 0) {
+                       if ((int)dev->irq <= 0) {
                                BUGMSG(D_NORMAL, "Autoprobe IRQ failed.\n");
                                err = -ENODEV;
                                goto out;
index 4cb401813b7e7ec9af98ec10334ec8a99b53069f..eb27976dab373134478409f64b9f65c6a6ea4219 100644 (file)
@@ -213,7 +213,7 @@ static int __init com90io_probe(struct net_device *dev)
                outb(0, _INTMASK);
                dev->irq = probe_irq_off(airqmask);
 
-               if (dev->irq <= 0) {
+               if ((int)dev->irq <= 0) {
                        BUGMSG(D_INIT_REASONS, "Autoprobe IRQ failed\n");
                        goto err_out;
                }