[PATCH v2] power: supply: max17042: fix OF node reference imbalance

Johan Hovold posted 1 patch 2 months, 2 weeks ago
drivers/power/supply/max17042_battery.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
[PATCH v2] power: supply: max17042: fix OF node reference imbalance
Posted by Johan Hovold 2 months, 2 weeks ago
The driver reuses the OF node of the parent multi-function device but
fails to take another reference to balance the one dropped by the
platform bus code when unbinding the MFD and deregistering the child
devices.

Fix this by using the intended helper for reusing OF nodes.

Fixes: 0cd4f1f77ad4 ("power: supply: max17042: add platform driver variant")
Cc: stable@vger.kernel.org	# 6.14
Cc: Dzmitry Sankouski <dsankouski@gmail.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
---

Changes in v2:
 - add missing driver name to patch summary prefix


 drivers/power/supply/max17042_battery.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply/max17042_battery.c
index acea176101fa..914f18ce79b3 100644
--- a/drivers/power/supply/max17042_battery.c
+++ b/drivers/power/supply/max17042_battery.c
@@ -1165,7 +1165,8 @@ static int max17042_platform_probe(struct platform_device *pdev)
 	if (!i2c)
 		return -EINVAL;
 
-	dev->of_node = dev->parent->of_node;
+	device_set_of_node_from_dev(dev, dev->parent);
+
 	id = platform_get_device_id(pdev);
 	irq = platform_get_irq(pdev, 0);
 
-- 
2.52.0
Re: [PATCH v2] power: supply: max17042: fix OF node reference imbalance
Posted by Sebastian Reichel 2 weeks, 3 days ago
On Tue, 07 Apr 2026 14:33:38 +0200, Johan Hovold wrote:
> The driver reuses the OF node of the parent multi-function device but
> fails to take another reference to balance the one dropped by the
> platform bus code when unbinding the MFD and deregistering the child
> devices.
> 
> Fix this by using the intended helper for reusing OF nodes.
> 
> [...]

Applied, thanks!

[1/1] power: supply: max17042: fix OF node reference imbalance
      commit: 68d234144b7dffd1f50b07ba74d0d6e833ef43a4

Best regards,
-- 
Sebastian Reichel <sebastian.reichel@collabora.com>
Re: [PATCH v2] power: supply: max17042: fix OF node reference imbalance
Posted by Johan Hovold 1 month, 1 week ago
Hi Sebastian,

On Tue, Apr 07, 2026 at 02:33:38PM +0200, Johan Hovold wrote:
> The driver reuses the OF node of the parent multi-function device but
> fails to take another reference to balance the one dropped by the
> platform bus code when unbinding the MFD and deregistering the child
> devices.
> 
> Fix this by using the intended helper for reusing OF nodes.
> 
> Fixes: 0cd4f1f77ad4 ("power: supply: max17042: add platform driver variant")
> Cc: stable@vger.kernel.org	# 6.14
> Cc: Dzmitry Sankouski <dsankouski@gmail.com>
> Signed-off-by: Johan Hovold <johan@kernel.org>
> ---
> 
> Changes in v2:
>  - add missing driver name to patch summary prefix

Can this one be picked up now?

Johan