[v1] Device managed platform_profile_register()

[PATCH 0/2] Device managed platform_profile_register()

Kurt Borja posted 2 patches 1 year, 1 month ago

Diff against v2
Download series mbox

There is a newer version of this series

drivers/acpi/platform_profile.c           | 27 +++++++++++++++++++++++
drivers/platform/x86/dell/alienware-wmi.c | 10 +--------
include/linux/platform_profile.h          |  2 +-
3 files changed, 29 insertions(+), 10 deletions(-)

Expand all Fold all

[PATCH 0/2] Device managed platform_profile_register()

Posted by Kurt Borja 1 year, 1 month ago

Hi :)

Now that the platform profile is tied to a device, this would be a nice
thing to have. Let me know what you think!

I can also replace existing users of platform_profile_register() where
it makes sense, if that's ok.

This is meant to be merged on the pdx86 tree.

~ Kurt

Kurt Borja (2):
  ACPI: platform_profile: Add devm_platform_profile_register()
  alienware-wmi: Use devm_platform_profile_register()

 drivers/acpi/platform_profile.c           | 27 +++++++++++++++++++++++
 drivers/platform/x86/dell/alienware-wmi.c | 10 +--------
 include/linux/platform_profile.h          |  2 +-
 3 files changed, 29 insertions(+), 10 deletions(-)


base-commit: 9e0894d07072e50b83ef077ce91e504bdb7484a3
-- 
2.47.1

Re: [PATCH 0/2] Device managed platform_profile_register()

Posted by Armin Wolf 1 year, 1 month ago

Am 21.12.24 um 08:08 schrieb Kurt Borja:

> Hi :)
>
> Now that the platform profile is tied to a device, this would be a nice
> thing to have. Let me know what you think!

Hi,

i like this patch series, having a devres version of platform_profile_register()
will be very helpful in simplifying the error handling of such drivers.

> I can also replace existing users of platform_profile_register() where
> it makes sense, if that's ok.

Sure. However keep in mind that this is entirely optional :).

Thanks,
Armin Wolf

>
> This is meant to be merged on the pdx86 tree.
>
> ~ Kurt
>
> Kurt Borja (2):
>    ACPI: platform_profile: Add devm_platform_profile_register()
>    alienware-wmi: Use devm_platform_profile_register()
>
>   drivers/acpi/platform_profile.c           | 27 +++++++++++++++++++++++
>   drivers/platform/x86/dell/alienware-wmi.c | 10 +--------
>   include/linux/platform_profile.h          |  2 +-
>   3 files changed, 29 insertions(+), 10 deletions(-)
>
>
> base-commit: 9e0894d07072e50b83ef077ce91e504bdb7484a3