platform/x86: intel: hid: Always call BTNL ACPI method
authorHans de Goede <hdegoede@redhat.com>
Sat, 15 Jul 2023 18:15:16 +0000 (20:15 +0200)
committerHans de Goede <hdegoede@redhat.com>
Tue, 25 Jul 2023 12:35:39 +0000 (14:35 +0200)
commite3ab18de2b09361d6f0e4aafb9cfd6d002ce43a1
tree0a3e3ea2dd92bf6039fdbb961ab21084f8672b8d
parent33c9ab5b493a0e922b06c12fed4fdcb862212cda
platform/x86: intel: hid: Always call BTNL ACPI method

On a HP Elite Dragonfly G2 the 0xcc and 0xcd events for SW_TABLET_MODE
are only send after the BTNL ACPI method has been called.

Likely more devices need this, so make the BTNL ACPI method unconditional
instead of only doing it on devices with a 5 button array.

Note this also makes the intel_button_array_enable() call in probe()
unconditional, that function does its own priv->array check. This makes
the intel_button_array_enable() call in probe() consistent with the calls
done on suspend/resume which also rely on the priv->array check inside
the function.

Reported-by: Maxim Mikityanskiy <maxtram95@gmail.com>
Closes: https://lore.kernel.org/platform-driver-x86/20230712175023.31651-1-maxtram95@gmail.com/
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20230715181516.5173-1-hdegoede@redhat.com
drivers/platform/x86/intel/hid.c