[PATCH] pc keyboard driver minor cleanup

[PATCH] pc keyboard driver minor cleanup

[pazke]

[-- Attachment #1.1: Type: text/plain, Size: 316 bytes --]

Hi,

added misc_register() return value checking and removed panic() in case of 
kmalloc failure (IMHO it's possible to live without PS/2 mouse :)

Best regards.

-- 
Andrey Panin            | Embedded systems software engineer
pazke@orbita1.ru        | PGP key: http://www.orbita1.ru/~pazke/AndreyPanin.asc

[-- Attachment #1.2: patch-pckeyb --]
[-- Type: text/plain, Size: 848 bytes --]

diff -u -X /usr/dontdiff /linux.vanilla/drivers/char/pc_keyb.c /linux/drivers/char/pc_keyb.c
--- /linux.vanilla/drivers/char/pc_keyb.c	Tue Jun 12 10:51:25 2001
+++ /linux/drivers/char/pc_keyb.c	Tue Jun 12 13:51:26 2001
@@ -1011,13 +1011,20 @@
 
 static int __init psaux_init(void)
 {
+	int retval;
+
 	if (!detect_auxiliary_port())
 		return -EIO;
 
-	misc_register(&psaux_mouse);
+	if ((retval = misc_register(&psaux_mouse)))
+		return retval;
+
 	queue = (struct aux_queue *) kmalloc(sizeof(*queue), GFP_KERNEL);
-	if (queue == NULL)
-		panic("psaux_init(): out of memory");
+	if (queue == NULL) {
+		printk(KERN_ERR "psaux_init(): out of memory\n");
+		misc_deregister(&psaux_mouse);
+		return -ENOMEM;
+	}
 	memset(queue, 0, sizeof(*queue));
 	queue->head = queue->tail = 0;
 	init_waitqueue_head(&queue->proc_list);

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]