Input: HIL - handle erros from input_register_device()

Also some whitespace cleanup.

Signed-off-by: Cyrill V. Gorcunov <gorcunov@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>

1 files changed, 66 insertions, 48 deletions

diff --git a/drivers/input/keyboard/hilkbd.c b/drivers/input/keyboard/hilkbd.c
index 35461eab2faf..255a6ec75a48 100644
--- a/drivers/input/keyboard/hilkbd.c
+++ b/drivers/input/keyboard/hilkbd.c
@@ -6,10 +6,10 @@
  *  Copyright (C) 1999-2006 Helge Deller <deller@gmx.de>
  *
  *  Very basic HP Human Interface Loop (HIL) driver.
- *  This driver handles the keyboard on HP300 (m68k) and on some 
+ *  This driver handles the keyboard on HP300 (m68k) and on some
  *  HP700 (parisc) series machines.
  *
- * 
+ *
  * This file is subject to the terms and conditions of the GNU General Public
  * License version 2.  See the file COPYING in the main directory of this
  * archive for more details.
@@ -64,9 +64,9 @@ MODULE_LICENSE("GPL v2");
 #endif
 
 
- 
+
 /* HIL helper functions */
- 
+
 #define hil_busy()              (hil_readb(HILBASE + HIL_CMD) & HIL_BUSY)
 #define hil_data_available()    (hil_readb(HILBASE + HIL_CMD) & HIL_DATA_RDY)
 #define hil_status()            (hil_readb(HILBASE + HIL_CMD))
@@ -75,7 +75,7 @@ MODULE_LICENSE("GPL v2");
 #define hil_write_data(x)       do { hil_writeb((x), HILBASE + HIL_DATA); } while (0)
 
 /* HIL constants */
- 
+
 #define HIL_BUSY                0x02
 #define HIL_DATA_RDY            0x01
 
@@ -86,10 +86,10 @@ MODULE_LICENSE("GPL v2");
 #define HIL_INTON               0x5C            /* Turn on interrupts. */
 #define HIL_INTOFF              0x5D            /* Turn off interrupts. */
 
-#define HIL_READKBDSADR         0xF9
-#define HIL_WRITEKBDSADR        0xE9
+#define HIL_READKBDSADR         0xF9
+#define HIL_WRITEKBDSADR        0xE9
 
-static unsigned int hphilkeyb_keycode[HIL_KEYCODES_SET1_TBLSIZE] = 
+static unsigned int hphilkeyb_keycode[HIL_KEYCODES_SET1_TBLSIZE] =
         { HIL_KEYCODES_SET1 };
 
 /* HIL structure */
@@ -97,11 +97,11 @@ static struct {
         struct input_dev *dev;
 
         unsigned int curdev;
-        
+
         unsigned char s;
         unsigned char c;
         int valid;
-        
+
         unsigned char data[16];
         unsigned int ptr;
         spinlock_t lock;
@@ -115,7 +115,7 @@ static void poll_finished(void)
         int down;
         int key;
         unsigned char scode;
-        
+
         switch (hil_dev.data[0]) {
         case 0x40:
                 down = (hil_dev.data[1] & 1) == 0;
@@ -127,6 +127,7 @@ static void poll_finished(void)
         hil_dev.curdev = 0;
 }
 
+
 static inline void handle_status(unsigned char s, unsigned char c)
 {
         if (c & 0x8) {
@@ -143,6 +144,7 @@ static inline void handle_status(unsigned char s, unsigned char c)
         }
 }
 
+
 static inline void handle_data(unsigned char s, unsigned char c)
 {
         if (hil_dev.curdev) {
@@ -152,13 +154,11 @@ static inline void handle_data(unsigned char s, unsigned char c)
 }
 
 
-/* 
- * Handle HIL interrupts.
- */
+/* handle HIL interrupts */
 static irqreturn_t hil_interrupt(int irq, void *handle)
 {
         unsigned char s, c;
-        
+
         s = hil_status();
         c = hil_read_data();
 
@@ -179,10 +179,8 @@ static irqreturn_t hil_interrupt(int irq, void *handle)
         return IRQ_HANDLED;
 }
 
-/*
- * Send a command to the HIL
- */
 
+/* send a command to the HIL */
 static void hil_do(unsigned char cmd, unsigned char *data, unsigned int len)
 {
         unsigned long flags;
@@ -200,16 +198,14 @@ static void hil_do(unsigned char cmd, unsigned char *data, unsigned int len)
 }
 
 
-/*
- * Initialise HIL. 
- */
-
+/* initialise HIL */
 static int __init
 hil_keyb_init(void)
 {
         unsigned char c;
         unsigned int i, kbid;
         wait_queue_head_t hil_wait;
+        int err;
 
         if (hil_dev.dev) {
                 return -ENODEV; /* already initialized */
@@ -219,15 +215,25 @@ hil_keyb_init(void)
         if (!hil_dev.dev)
                 return -ENOMEM;
         hil_dev.dev->private = &hil_dev;
-        
+
 #if defined(CONFIG_HP300)
-        if (!hwreg_present((void *)(HILBASE + HIL_DATA)))
-                return -ENODEV;
-        
-        request_region(HILBASE+HIL_DATA, 2, "hil");
+        if (!hwreg_present((void *)(HILBASE + HIL_DATA))) {
+                printk(KERN_ERR "HIL: hardware register was not found\n");
+                err = -ENODEV;
+                goto err1;
+        }
+        if (!request_region(HILBASE + HIL_DATA, 2, "hil")) {
+                printk(KERN_ERR "HIL: IOPORT region already used\n");
+                err = -EIO;
+                goto err1;
+        }
 #endif
-        
-        request_irq(HIL_IRQ, hil_interrupt, 0, "hil", hil_dev.dev_id);
+
+        err = request_irq(HIL_IRQ, hil_interrupt, 0, "hil", hil_dev.dev_id);
+        if (err) {
+                printk(KERN_ERR "HIL: Can't get IRQ\n");
+                goto err2;