[PATCH 0/2] Use devm_register_power_off_handler() where appropriate

[PATCH 0/2] Use devm_register_power_off_handler() where appropriate

[Dragan Simic]

This is a tiny series that simplifies the code a bit in a couple of MFD
drivers by using devm_register_power_off_handler(), which is a purpose-
specific wrapper for devm_register_sys_off_handler().

No intended functional changes are introduced.

Dragan Simic (2):
  mfd: axp20x: Use devm_register_power_off_handler()
  mfd: stpmic1: Use devm_register_power_off_handler()

 drivers/mfd/axp20x.c  | 5 +----
 drivers/mfd/stpmic1.c | 6 +-----
 2 files changed, 2 insertions(+), 9 deletions(-)

[PATCH 1/2] mfd: axp20x: Use devm_register_power_off_handler()

[Dragan Simic]

Simplify the code a bit by using devm_register_power_off_handler(), which is
a purpose-specific wrapper for devm_register_sys_off_handler().  No intended
functional changes are introduced.

Signed-off-by: Dragan Simic <dsimic@manjaro.org>
---
 drivers/mfd/axp20x.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/mfd/axp20x.c b/drivers/mfd/axp20x.c
index 4051551757f2..2005d4291332 100644
--- a/drivers/mfd/axp20x.c
+++ b/drivers/mfd/axp20x.c
@@ -1429,10 +1429,7 @@ int axp20x_device_probe(struct axp20x_dev *axp20x)
 	}
 
 	if (axp20x->variant != AXP288_ID)
-		devm_register_sys_off_handler(axp20x->dev,
-					      SYS_OFF_MODE_POWER_OFF,
-					      SYS_OFF_PRIO_DEFAULT,
-					      axp20x_power_off, axp20x);
+		devm_register_power_off_handler(axp20x->dev, axp20x_power_off, axp20x);
 
 	dev_info(axp20x->dev, "AXP20X driver loaded\n");
 

[PATCH 2/2] mfd: stpmic1: Use devm_register_power_off_handler()

[Dragan Simic]

Simplify the code a bit by using devm_register_power_off_handler(), which is
a purpose-specific wrapper for devm_register_sys_off_handler().  No intended
functional changes are introduced.

Signed-off-by: Dragan Simic <dsimic@manjaro.org>
---
 drivers/mfd/stpmic1.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/mfd/stpmic1.c b/drivers/mfd/stpmic1.c
index d8a603d95aa6..081827bc0596 100644
--- a/drivers/mfd/stpmic1.c
+++ b/drivers/mfd/stpmic1.c
@@ -170,11 +170,7 @@ static int stpmic1_probe(struct i2c_client *i2c)
 		return ret;
 	}
 
-	ret = devm_register_sys_off_handler(ddata->dev,
-					    SYS_OFF_MODE_POWER_OFF,
-					    SYS_OFF_PRIO_DEFAULT,
-					    stpmic1_power_off,
-					    ddata);
+	ret = devm_register_power_off_handler(ddata->dev, stpmic1_power_off, ddata);
 	if (ret) {
 		dev_err(ddata->dev, "failed to register sys-off handler: %d\n", ret);
 		return ret;

Re: [PATCH 0/2] Use devm_register_power_off_handler() where appropriate

[Lee Jones]

On Tue, 12 Nov 2024 13:46:46 +0100, Dragan Simic wrote:
> This is a tiny series that simplifies the code a bit in a couple of MFD
> drivers by using devm_register_power_off_handler(), which is a purpose-
> specific wrapper for devm_register_sys_off_handler().
> 
> No intended functional changes are introduced.
> 
> Dragan Simic (2):
>   mfd: axp20x: Use devm_register_power_off_handler()
>   mfd: stpmic1: Use devm_register_power_off_handler()
> 
> [...]

Applied, thanks!

[1/2] mfd: axp20x: Use devm_register_power_off_handler()
      commit: c8729f5dc9aacd4bad8ac93487c2f5ee1c409f2c
[2/2] mfd: stpmic1: Use devm_register_power_off_handler()
      commit: d893a7ed58430259cbc939cd001e24833df93a75

--
Lee Jones [李琼斯]