[v6] Add support for binding ACPI platform profile to multiple drivers

[PATCH v6 14/22] ACPI: platform_profile: Notify change events on register and unregister

Posted by Mario Limonciello 1 year, 3 months ago

As multiple platform profile handlers may come and go, send a notification
to userspace each time that a platform profile handler is registered or
unregistered.

Tested-by: Mark Pearson <mpearson-lenovo@squebb.ca>
Tested-by: Matthew Schwartz <matthew.schwartz@linux.dev>
Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
---
 drivers/acpi/platform_profile.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/acpi/platform_profile.c b/drivers/acpi/platform_profile.c
index c5487d3928c1b..c4d451782ff18 100644
--- a/drivers/acpi/platform_profile.c
+++ b/drivers/acpi/platform_profile.c
@@ -380,6 +380,8 @@ int platform_profile_register(struct platform_profile_handler *pprof)
 		goto cleanup_ida;
 	}
 
+	sysfs_notify(acpi_kobj, NULL, "platform_profile");
+
 	cur_profile = pprof;
 
 	err = sysfs_update_group(acpi_kobj, &platform_profile_group);
@@ -408,6 +410,8 @@ int platform_profile_remove(struct platform_profile_handler *pprof)
 	device_unregister(pprof->class_dev);
 	ida_free(&platform_profile_ida, id);
 
+	sysfs_notify(acpi_kobj, NULL, "platform_profile");
+
 	cur_profile = NULL;
 
 	sysfs_update_group(acpi_kobj, &platform_profile_group);
-- 
2.43.0

Re: [PATCH v6 14/22] ACPI: platform_profile: Notify change events on register and unregister

Posted by Armin Wolf 1 year, 2 months ago

Am 09.11.24 um 05:41 schrieb Mario Limonciello:

> As multiple platform profile handlers may come and go, send a notification
> to userspace each time that a platform profile handler is registered or
> unregistered.

Reviewed-by: Armin Wolf <W_Armin@gmx.de>

>
> Tested-by: Mark Pearson <mpearson-lenovo@squebb.ca>
> Tested-by: Matthew Schwartz <matthew.schwartz@linux.dev>
> Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca>
> Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
> ---
>   drivers/acpi/platform_profile.c | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/drivers/acpi/platform_profile.c b/drivers/acpi/platform_profile.c
> index c5487d3928c1b..c4d451782ff18 100644
> --- a/drivers/acpi/platform_profile.c
> +++ b/drivers/acpi/platform_profile.c
> @@ -380,6 +380,8 @@ int platform_profile_register(struct platform_profile_handler *pprof)
>   		goto cleanup_ida;
>   	}
>
> +	sysfs_notify(acpi_kobj, NULL, "platform_profile");
> +
>   	cur_profile = pprof;
>
>   	err = sysfs_update_group(acpi_kobj, &platform_profile_group);
> @@ -408,6 +410,8 @@ int platform_profile_remove(struct platform_profile_handler *pprof)
>   	device_unregister(pprof->class_dev);
>   	ida_free(&platform_profile_ida, id);
>
> +	sysfs_notify(acpi_kobj, NULL, "platform_profile");
> +
>   	cur_profile = NULL;
>
>   	sysfs_update_group(acpi_kobj, &platform_profile_group);