Re: [PATCH 09/18] platform/x86: dell-pc: Use devm_platform_profile_register()

Mario Limonciello
2025-01-09T16:36:10Z

On 1/9/2025 09:06, Kurt Borja wrote:
> Replace platform_profile_register() with it's device managed version.
>
> Signed-off-by: Kurt Borja <kuurtb@gmail.com>
Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
> ---
> drivers/platform/x86/dell/dell-pc.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/platform/x86/dell/dell-pc.c b/drivers/platform/x86/dell/dell-pc.c
> index 1b6e60a63491..dda6f13295e8 100644
> --- a/drivers/platform/x86/dell/dell-pc.c
> +++ b/drivers/platform/x86/dell/dell-pc.c
> @@ -281,7 +281,7 @@ static int thermal_init(void)
> thermal_handler->ops = &dell_pc_platform_profile_ops;
>
> /* Clean up if failed */
> - ret = platform_profile_register(thermal_handler, NULL);
> + ret = devm_platform_profile_register(thermal_handler, NULL);
> if (ret)
> goto cleanup_thermal_handler;
>
> @@ -298,8 +298,6 @@ static int thermal_init(void)
>
> static void thermal_cleanup(void)
> {
> - if (thermal_handler)
> - platform_profile_remove(thermal_handler);
> platform_device_unregister(platform_device);
> }
>