]> git.baikalelectronics.ru Git - kernel.git/commitdiff
platform/x86: dell-wmi: add new dmi mapping for keycode 0xffff
authorY Paritcher <y.linux@paritcher.com>
Wed, 10 Jun 2020 17:56:58 +0000 (13:56 -0400)
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 9 Jul 2020 19:47:27 +0000 (22:47 +0300)
This keycode is used by Dell as a no-op for keys that should have
no function.

This keycode is never triggered by a keypress in practice, rather
it is included from the 0xB2 DMI table at startup.

This prevents the following messages from being logged at startup on a
Dell Inspiron 5593:

    dell_wmi: firmware scancode 0x48 maps to unrecognized keycode 0xffff
    dell_wmi: firmware scancode 0x50 maps to unrecognized keycode 0xffff

as per this code comment:

   Log if we find an entry in the DMI table that we don't
   understand.  If this happens, we should figure out what
   the entry means and add it to bios_to_linux_keycode.

Signed-off-by: Y Paritcher <y.linux@paritcher.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Pali Rohár <pali@kernel.org>
Reviewed-by: Mario Limonciello <mario.limonciello@dell.com>
drivers/platform/x86/dell-wmi.c

index e3bc2601e631efe7153c88b66dfd28c21eab0440..bbdb3e8608927d795c729b6f31995cb19981b297 100644 (file)
@@ -506,7 +506,7 @@ static void handle_dmi_entry(const struct dmi_header *dm, void *opaque)
                u16 keycode = (bios_entry->keycode <
                               ARRAY_SIZE(bios_to_linux_keycode)) ?
                        bios_to_linux_keycode[bios_entry->keycode] :
-                       KEY_RESERVED;
+                       (bios_entry->keycode == 0xffff ? KEY_UNKNOWN : KEY_RESERVED);
 
                /*
                 * Log if we find an entry in the DMI table that we don't