Re: [PATCH v6 3/3] hwmon: (applesmc) Convert to hwmon_device_register_with_info

[sashiko-bot@kernel.org]

> The legacy hwmon_device_register() function is deprecated and triggers
> warnings in dmesg. Convert the driver to the modern
> hwmon_device_register_with_info() API.
> 
> This conversion does the following:
> - Dynamically allocates standard HWMON temp, fan, and pwm channels.
> - Configures HWMON ops callbacks (.is_visible, .read, .read_string, .write).
> - Standardizes attribute naming to match the HWMON ABI:
>   - fanX_output -> fanX_target (HWMON_F_TARGET)
>   - fanX_manual -> pwmX_enable (HWMON_PWM_ENABLE)
> - Dynamically registers non-standard fanX_safe attributes under the HWMON
>   class directory via extra_groups.
> - Cleans up legacy sysfs nodes, groups, and unused show/store static functions
>   to avoid unused symbol compiler warnings.
> - Avoids recursive mutex deadlocks when writing to pwmX_enable by locklessly
>   resolving the entry and invoking the underlying raw SMC read/write calls.
> - Avoids UAF race condition on module exit by using unmanaged registration and
>   explicitly calling hwmon_device_unregister() as the first step of applesmc_exit(),
>   guaranteeing that HWMON nodes are destroyed before static structures are freed.
> 
> Signed-off-by: Shih-Yuan Lee <fourdollars@debian.org>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260711093323.14529-1-fourdollars@debian.org?part=3