hwmon: (dell-smm) Unify i8k_ioctl() and i8k_ioctl_unlocked()
authorArmin Wolf <W_Armin@gmx.de>
Sat, 11 Dec 2021 15:54:22 +0000 (16:54 +0100)
committerGuenter Roeck <linux@roeck-us.net>
Sun, 26 Dec 2021 23:02:06 +0000 (15:02 -0800)
commit87b93329fdd64bbb87db72d06ca084a06d183d6d
treeac76aba824db853b9065e5e7b09de6d6a9ed01be
parent02405387746915b93b283c18780b6cef90394ea1
hwmon: (dell-smm) Unify i8k_ioctl() and i8k_ioctl_unlocked()

The only purpose of i8k_ioctl() is to call i8k_ioctl_unlocked()
with i8k_mutex held. Judging from the hwmon code, this mutex
only needs to be held when setting the fan speed/mode, so
the operation of I8K_SET_FAN is guaranteed to be atomic.
Unify both functions and reduce the locking of i8k_mutex
to I8K_SET_FAN.

Tested on a Dell Inspiron 3505.

Signed-off-by: Armin Wolf <W_Armin@gmx.de>
Reviewed-by: Pali Rohár <pali@kernel.org>
Link: https://lore.kernel.org/r/20211211155422.16830-3-W_Armin@gmx.de
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/dell-smm-hwmon.c