Merge ../linux-2.6-watchdog-mm
[linux-drm-fsl-dcu.git] / drivers / input / keyboard / lkkbd.c
index 9481132532d017ee42d36cecc789a0007e7bca2b..979b93e33da7a9e0461594887267d17c8e3891a0 100644 (file)
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- *
- * Should you need to contact me, the author, you can do so either by
- * email or by paper mail:
- * Jan-Benedict Glaw, Lilienstraße 16, 33790 Hörste (near Halle/Westf.),
- * Germany.
  */
 
 #include <linux/delay.h>
@@ -273,11 +268,11 @@ static lk_keycode_t lkkbd_keycode[LK_NUM_KEYCODES] = {
        [0xfb] = KEY_APOSTROPHE,
 };
 
-#define CHECK_LED(LED, BITS) do {              \
-       if (test_bit (LED, lk->dev->led))       \
-               leds_on |= BITS;                \
-       else                                    \
-               leds_off |= BITS;               \
+#define CHECK_LED(LK, VAR_ON, VAR_OFF, LED, BITS) do {         \
+       if (test_bit (LED, (LK)->dev->led))                     \
+               VAR_ON |= BITS;                                 \
+       else                                                    \
+               VAR_OFF |= BITS;                                \
        } while (0)
 
 /*
@@ -298,6 +293,42 @@ struct lkkbd {
        int ctrlclick_volume;
 };
 
+#ifdef LKKBD_DEBUG
+/*
+ * Responses from the keyboard and mapping back to their names.
+ */
+static struct {
+       unsigned char value;
+       unsigned char *name;
+} lk_response[] = {
+#define RESPONSE(x) { .value = (x), .name = #x, }
+       RESPONSE (LK_STUCK_KEY),
+       RESPONSE (LK_SELFTEST_FAILED),
+       RESPONSE (LK_ALL_KEYS_UP),
+       RESPONSE (LK_METRONOME),
+       RESPONSE (LK_OUTPUT_ERROR),
+       RESPONSE (LK_INPUT_ERROR),
+       RESPONSE (LK_KBD_LOCKED),
+       RESPONSE (LK_KBD_TEST_MODE_ACK),
+       RESPONSE (LK_PREFIX_KEY_DOWN),
+       RESPONSE (LK_MODE_CHANGE_ACK),
+       RESPONSE (LK_RESPONSE_RESERVED),
+#undef RESPONSE
+};
+
+static unsigned char *
+response_name (unsigned char value)
+{
+       int i;
+
+       for (i = 0; i < ARRAY_SIZE (lk_response); i++)
+               if (lk_response[i].value == value)
+                       return lk_response[i].name;
+
+       return "<unknown>";
+}
+#endif /* LKKBD_DEBUG */
+
 /*
  * Calculate volume parameter byte for a given volume.
  */
@@ -348,18 +379,21 @@ lkkbd_detection_done (struct lkkbd *lk)
         */
        switch (lk->id[4]) {
                case 1:
-                       sprintf (lk->name, "DEC LK201 keyboard");
+                       strlcpy (lk->name, "DEC LK201 keyboard",
+                                sizeof (lk->name));
 
                        if (lk201_compose_is_alt)
                                lk->keycode[0xb1] = KEY_LEFTALT;
                        break;
 
                case 2:
-                       sprintf (lk->name, "DEC LK401 keyboard");
+                       strlcpy (lk->name, "DEC LK401 keyboard",
+                                sizeof (lk->name));
                        break;
 
                default:
-                       sprintf (lk->name, "Unknown DEC keyboard");
+                       strlcpy (lk->name, "Unknown DEC keyboard",
+                                sizeof (lk->name));
                        printk (KERN_ERR "lkkbd: keyboard on %s is unknown, "
                                        "please report to Jan-Benedict Glaw "
                                        "<jbglaw@lug-owl.de>\n", lk->phys);
@@ -414,8 +448,7 @@ lkkbd_detection_done (struct lkkbd *lk)
  * is received.
  */
 static irqreturn_t
-lkkbd_interrupt (struct serio *serio, unsigned char data, unsigned int flags,
-               struct pt_regs *regs)
+lkkbd_interrupt (struct serio *serio, unsigned char data, unsigned int flags)
 {
        struct lkkbd *lk = serio_get_drvdata (serio);
        int i;
@@ -434,54 +467,33 @@ lkkbd_interrupt (struct serio *serio, unsigned char data, unsigned int flags,
 
        switch (data) {
                case LK_ALL_KEYS_UP:
-                       input_regs (lk->dev, regs);
                        for (i = 0; i < ARRAY_SIZE (lkkbd_keycode); i++)
                                if (lk->keycode[i] != KEY_RESERVED)
                                        input_report_key (lk->dev, lk->keycode[i], 0);
                        input_sync (lk->dev);
                        break;
-               case LK_METRONOME:
-                       DBG (KERN_INFO "Got LK_METRONOME and don't "
-                                       "know how to handle...\n");
+
+               case 0x01:
+                       DBG (KERN_INFO "Got 0x01, scheduling re-initialization\n");
+                       lk->ignore_bytes = LK_NUM_IGNORE_BYTES;
+                       lk->id[LK_NUM_IGNORE_BYTES - lk->ignore_bytes--] = data;
+                       schedule_work (&lk->tq);
                        break;
+
+               case LK_METRONOME:
                case LK_OUTPUT_ERROR:
-                       DBG (KERN_INFO "Got LK_OUTPUT_ERROR and don't "
-                                       "know how to handle...\n");
-                       break;
                case LK_INPUT_ERROR:
-                       DBG (KERN_INFO "Got LK_INPUT_ERROR and don't "
-                                       "know how to handle...\n");
-                       break;
                case LK_KBD_LOCKED:
-                       DBG (KERN_INFO "Got LK_KBD_LOCKED and don't "
-                                       "know how to handle...\n");
-                       break;
                case LK_KBD_TEST_MODE_ACK:
-                       DBG (KERN_INFO "Got LK_KBD_TEST_MODE_ACK and don't "
-                                       "know how to handle...\n");
-                       break;
                case LK_PREFIX_KEY_DOWN:
-                       DBG (KERN_INFO "Got LK_PREFIX_KEY_DOWN and don't "
-                                       "know how to handle...\n");
-                       break;
                case LK_MODE_CHANGE_ACK:
-                       DBG (KERN_INFO "Got LK_MODE_CHANGE_ACK and ignored "
-                                       "it properly...\n");
-                       break;
                case LK_RESPONSE_RESERVED:
-                       DBG (KERN_INFO "Got LK_RESPONSE_RESERVED and don't "
-                                       "know how to handle...\n");
-                       break;
-               case 0x01:
-                       DBG (KERN_INFO "Got 0x01, scheduling re-initialization\n");
-                       lk->ignore_bytes = LK_NUM_IGNORE_BYTES;
-                       lk->id[LK_NUM_IGNORE_BYTES - lk->ignore_bytes--] = data;
-                       schedule_work (&lk->tq);
+                       DBG (KERN_INFO "Got %s and don't know how to handle...\n",
+                                       response_name (data));
                        break;
 
                default:
                        if (lk->keycode[data] != KEY_RESERVED) {
-                               input_regs (lk->dev, regs);
                                if (!test_bit (lk->keycode[data], lk->dev->key))
                                        input_report_key (lk->dev, lk->keycode[data], 1);
                                else
@@ -509,10 +521,10 @@ lkkbd_event (struct input_dev *dev, unsigned int type, unsigned int code,
 
        switch (type) {
                case EV_LED:
-                       CHECK_LED (LED_CAPSL, LK_LED_SHIFTLOCK);
-                       CHECK_LED (LED_COMPOSE, LK_LED_COMPOSE);
-                       CHECK_LED (LED_SCROLLL, LK_LED_SCROLLLOCK);
-                       CHECK_LED (LED_SLEEP, LK_LED_WAIT);
+                       CHECK_LED (lk, leds_on, leds_off, LED_CAPSL, LK_LED_SHIFTLOCK);
+                       CHECK_LED (lk, leds_on, leds_off, LED_COMPOSE, LK_LED_COMPOSE);
+                       CHECK_LED (lk, leds_on, leds_off, LED_SCROLLL, LK_LED_SCROLLLOCK);
+                       CHECK_LED (lk, leds_on, leds_off, LED_SLEEP, LK_LED_WAIT);
                        if (leds_on != 0) {
                                lk->serio->write (lk->serio, LK_CMD_LED_ON);
                                lk->serio->write (lk->serio, leds_on);
@@ -574,10 +586,10 @@ lkkbd_reinit (void *data)
        lk->serio->write (lk->serio, LK_CMD_SET_DEFAULTS);
 
        /* Set LEDs */
-       CHECK_LED (LED_CAPSL, LK_LED_SHIFTLOCK);
-       CHECK_LED (LED_COMPOSE, LK_LED_COMPOSE);
-       CHECK_LED (LED_SCROLLL, LK_LED_SCROLLLOCK);
-       CHECK_LED (LED_SLEEP, LK_LED_WAIT);
+       CHECK_LED (lk, leds_on, leds_off, LED_CAPSL, LK_LED_SHIFTLOCK);
+       CHECK_LED (lk, leds_on, leds_off, LED_COMPOSE, LK_LED_COMPOSE);
+       CHECK_LED (lk, leds_on, leds_off, LED_SCROLLL, LK_LED_SCROLLLOCK);
+       CHECK_LED (lk, leds_on, leds_off, LED_SLEEP, LK_LED_WAIT);
        if (leds_on != 0) {
                lk->serio->write (lk->serio, LK_CMD_LED_ON);
                lk->serio->write (lk->serio, leds_on);