Re: [PATCH 4/4] dell-wmi: Rework code for generating sparse keymap and processing WMI events

From: Andy Lutomirski
Date: Mon May 23 2016 - 13:08:24 EST


On Sun, May 22, 2016 at 4:36 AM, Pali RohÃr <pali.rohar@xxxxxxxxx> wrote:
> This patch unify procedure for generating sparse keymap and unify also big
> switch code for processing WMI events of different types. After this patch
> dell-wmi driver does not differ between "old" and "new" hotkey type.
>
> It construct sparse keymap table with all WMI codes. It is because on some
> laptops (e.g. Dell Latitude E6440) ACPI/firmware send both event types (old
> and new).
>
> Each WMI code in sparse keymap table is prefixed by 16bit event type, so it
> does not change functionality on laptops with "old" hotkey support (those
> without scancodes in DMI).

This seems like a nice cleanup. I'll try to test it soon.

--Andy