lkml.org 
[lkml]   [2007]   [Aug]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] ser_a2232.c bug removed: handles "IRQ request denied" case


This patch handles the case in which the request for the
vertical blank interrupt could not be granted. Thanks to
Nazia Zaman for spotting it.

Applies to latest 2.6.x kernels.


--- linux/drivers/char/ser_a2232.c.org 2007-08-30 15:41:25.000000000 +0100
+++ linux/drivers/char/ser_a2232.c 2007-08-30 15:46:17.000000000 +0100
@@ -787,7 +787,7 @@ static int __init a2232board_init(void)

}

- printk("Total: %d A2232 boards initialized.\n", nr_a2232); /* Some status report if no card was found */
+ printk("Total: %d A2232 boards detected.\n", nr_a2232); /* Some status report if no card was found */

a2232_init_portstructs();

@@ -795,9 +795,13 @@ static int __init a2232board_init(void)
a2232_init_drivers also registers the drivers. Must be here because all boards
have to be detected first.
*/
- if (a2232_init_drivers()) return -ENODEV; // maybe we should use a different -Exxx?
+ if (a2232_init_drivers())
+ return -ENODEV; // maybe we should use a different -Exxx?

- request_irq(IRQ_AMIGA_VERTB, a2232_vbl_inter, 0, "A2232 serial VBL", a2232_driver_ID);
+ if (request_irq(IRQ_AMIGA_VERTB, a2232_vbl_inter, 0, "A2232 serial VBL", a2232_driver_ID))
+ return -ENODEV;
+
+ printk("A2232 driver initialized successfully.\n");
return 0;
}


Signed-off-by: Enver Haase <a2232@gmx.net>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-08-30 17:55    [W:0.142 / U:0.124 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site