Re: [PATCH v4 10/13] platform/x86: uniwill-wmi: use devm_blocking_notifier_chain_register()

Am 26.07.26 um 12:17 schrieb Eliav Farber:
> Replace the blocking_notifier_chain_register() +
> devm_add_action_or_reset() pattern with a single call to
> devm_blocking_notifier_chain_register(), removing the
> devm_uniwill_wmi_unregister_notifier() callback.
>=20

Reviewed-by: Armin Wolf <[email protected]>

> Signed-off-by: Eliav Farber <[email protected]>
> ---
>   drivers/platform/x86/uniwill/uniwill-wmi.c | 17 +++--------------
>   1 file changed, 3 insertions(+), 14 deletions(-)
>=20
> diff --git a/drivers/platform/x86/uniwill/uniwill-wmi.c b/drivers/platfo=
rm/x86/uniwill/uniwill-wmi.c
> index afcbfa4f7552..2ffa29a829ee 100644
> --- a/drivers/platform/x86/uniwill/uniwill-wmi.c
> +++ b/drivers/platform/x86/uniwill/uniwill-wmi.c
> @@ -26,22 +26,11 @@
>  =20
>   static BLOCKING_NOTIFIER_HEAD(uniwill_wmi_chain_head);
>  =20
> -static void devm_uniwill_wmi_unregister_notifier(void *data)
> -{
> -	struct notifier_block *nb =3D data;
> -
> -	blocking_notifier_chain_unregister(&uniwill_wmi_chain_head, nb);
> -}
> -
>   int devm_uniwill_wmi_register_notifier(struct device *dev, struct noti=
fier_block *nb)
>   {
> -	int ret;
> -
> -	ret =3D blocking_notifier_chain_register(&uniwill_wmi_chain_head, nb);
> -	if (ret < 0)
> -		return ret;
> -
> -	return devm_add_action_or_reset(dev, devm_uniwill_wmi_unregister_notif=
ier, nb);
> +	return devm_blocking_notifier_chain_register(dev,
> +						     &uniwill_wmi_chain_head,
> +						     nb);
>   }
>  =20
>   static void uniwill_wmi_notify(struct wmi_device *wdev, union acpi_obj=
ect *obj)