HID: amd_sfh: if no sensors are enabled, clean up
authorMario Limonciello <mario.limonciello@amd.com>
Fri, 3 Feb 2023 22:08:49 +0000 (16:08 -0600)
committerBenjamin Tissoires <benjamin.tissoires@redhat.com>
Mon, 6 Feb 2023 10:00:36 +0000 (11:00 +0100)
commit7bcfdab3f0c6672ca52be3cb65a0550d8b99554b
tree0a40184f4bbca1bd9f4a9d3f9af4d66b18bd4b13
parent690eb7dec72ae52d1d710d14a451844b4d0f4f19
HID: amd_sfh: if no sensors are enabled, clean up

It was reported that commit b300667b33b2 ("HID: amd_sfh: Disable the
interrupt for all command") had caused increased resume time on HP Envy
x360.

Before this commit 3 sensors were reported, but they were not actually
functional.  After this commit the sensors are no longer reported, but
also the resume time increased.

To avoid this problem explicitly look for the number of disabled sensors.
If all the sensors are disabled, clean everything up.

Fixes: b300667b33b2 ("HID: amd_sfh: Disable the interrupt for all command")
Link: https://gitlab.freedesktop.org/drm/amd/-/issues/2115
Reported-by: Xaver Hugl <xaver.hugl@gmail.com>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
Acked-by: Basavaraj Natikar <Basavaraj.Natikar@amd.com>
Link: https://lore.kernel.org/r/20230203220850.13924-1-mario.limonciello@amd.com
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
drivers/hid/amd-sfh-hid/amd_sfh_client.c
drivers/hid/amd-sfh-hid/amd_sfh_hid.h