I recently changed several drivers in drivers/net of the Linux kernel to
correctly check the return value of request_region.  Since you appear
to be the maintainer of this driver, I'm forwarding the patch to you.

Thanks
--
-Steven
In a time of universal deceit, telling the truth is a revolutionary act.
                        -- George Orwell
He's alive.  He's alive!  Oh, that fellow at RadioShack said I was mad!
Well, who's mad now?
                        -- Montgomery C. Burns

diff -Nru clean-2.4.17//drivers/net/irda/nsc-ircc.c
linux/drivers/net/irda/nsc-ircc.c
--- clean-2.4.17//drivers/net/irda/nsc-ircc.c   Mon Nov  5 19:23:13 2001
+++ linux/drivers/net/irda/nsc-ircc.c   Thu Dec 27 14:08:03 2001
@@ -282,15 +282,13 @@
         self->io.fifo_size = 32;

        /* Reserve the ioports that we need */
-       ret = check_region(self->io.fir_base, self->io.fir_ext);
-       if (ret < 0) {
+       if (!request_region(self->io.fir_base, self->io.fir_ext, driver_name)) {
                WARNING(__FUNCTION__ "(), can't get iobase of 0x%03x\n",
                        self->io.fir_base);
                dev_self[i] = NULL;
                kfree(self);
                return -ENODEV;
        }
-       request_region(self->io.fir_base, self->io.fir_ext, driver_name);

        /* Initialize QoS for this device */
        irda_init_max_qos_capabilies(&self->qos);



_______________________________________________
Linux-IrDA mailing list  -  [EMAIL PROTECTED]
http://www.pasta.cs.UiT.No/mailman/listinfo/linux-irda

Reply via email to