Commit a2a94e73 authored by Paul Bolle's avatar Paul Bolle Committed by Dmitry Torokhov

Input: i8042 - downgrade selftest error message to dbg()

On a "really fragile" laptop I noticed a single
    i8042.c: i8042 controller selftest failed. (0x1 != 0x55)

error in the log. But there's no reason to print this message at
KERN_ERR level each time that loop fails, especially since the message
telling about the overall selftest failure is printed at KERN_INFO level
(on X86).

Add an actual error message for non-X86 systems, where a selftest
failure is (apparently) more serious. Remove a space in an another error
message.
Signed-off-by: default avatarPaul Bolle <pebolle@tiscali.nl>
Signed-off-by: default avatarDmitry Torokhov <dtor@mail.ru>
parent 70874867
...@@ -869,15 +869,15 @@ static int i8042_controller_selftest(void) ...@@ -869,15 +869,15 @@ static int i8042_controller_selftest(void)
do { do {
if (i8042_command(&param, I8042_CMD_CTL_TEST)) { if (i8042_command(&param, I8042_CMD_CTL_TEST)) {
pr_err("i8042 controller self test timeout\n"); pr_err("i8042 controller selftest timeout\n");
return -ENODEV; return -ENODEV;
} }
if (param == I8042_RET_CTL_TEST) if (param == I8042_RET_CTL_TEST)
return 0; return 0;
pr_err("i8042 controller selftest failed. (%#x != %#x)\n", dbg("i8042 controller selftest: %#x != %#x\n",
param, I8042_RET_CTL_TEST); param, I8042_RET_CTL_TEST);
msleep(50); msleep(50);
} while (i++ < 5); } while (i++ < 5);
...@@ -891,6 +891,7 @@ static int i8042_controller_selftest(void) ...@@ -891,6 +891,7 @@ static int i8042_controller_selftest(void)
pr_info("giving up on controller selftest, continuing anyway...\n"); pr_info("giving up on controller selftest, continuing anyway...\n");
return 0; return 0;
#else #else
pr_err("i8042 controller selftest failed\n");
return -EIO; return -EIO;
#endif #endif
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment