Input: i8042 - another attempt to fix AUX delivery checks
authorDmitry Torokhov <dtor@insightbb.com>
Thu, 8 Mar 2007 04:20:55 +0000 (23:20 -0500)
committerDmitry Torokhov <dtor@insightbb.com>
Thu, 8 Mar 2007 04:20:55 +0000 (23:20 -0500)
Do not assume that AUX_LOOP command is broken unless it
completes successfully but returns wrong (unexpected) data.

Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
drivers/input/serio/i8042.c

index ec195a36e8f66ea9f170698d5b6dbb34f6140649..db9cca3b65e068f4e22002ebab7d448a510d8eb0 100644 (file)
@@ -553,7 +553,8 @@ static int __devinit i8042_check_aux(void)
  */
 
        param = 0x5a;
-       if (i8042_command(&param, I8042_CMD_AUX_LOOP) || param != 0x5a) {
+       retval = i8042_command(&param, I8042_CMD_AUX_LOOP);
+       if (retval || param != 0x5a) {
 
 /*
  * External connection test - filters out AT-soldered PS/2 i8042's
@@ -567,7 +568,12 @@ static int __devinit i8042_check_aux(void)
                    (param && param != 0xfa && param != 0xff))
                        return -1;
 
-               aux_loop_broken = 1;
+/*
+ * If AUX_LOOP completed without error but returned unexpected data
+ * mark it as broken
+ */
+               if (!retval)
+                       aux_loop_broken = 1;
        }
 
 /*