]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
phy: ti-pipe3: fix device leak at unbind
authorJohan Hovold <johan@kernel.org>
Thu, 24 Jul 2025 13:12:06 +0000 (15:12 +0200)
committerVinod Koul <vkoul@kernel.org>
Tue, 12 Aug 2025 16:00:00 +0000 (21:30 +0530)
Make sure to drop the reference to the control device taken by
of_find_device_by_node() during probe when the driver is unbound.

Fixes: 918ee0d21ba4 ("usb: phy: omap-usb3: Don't use omap_get_control_dev()")
Cc: stable@vger.kernel.org # 3.13
Cc: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Johan Hovold <johan@kernel.org>
Link: https://lore.kernel.org/r/20250724131206.2211-4-johan@kernel.org
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/ti/phy-ti-pipe3.c

index da2cbacb982c6bebad3e412f87ff02778da61753..ae764d6524c99a6df3236f45e845e97e692bd17f 100644 (file)
@@ -667,12 +667,20 @@ static int ti_pipe3_get_clk(struct ti_pipe3 *phy)
        return 0;
 }
 
+static void ti_pipe3_put_device(void *_dev)
+{
+       struct device *dev = _dev;
+
+       put_device(dev);
+}
+
 static int ti_pipe3_get_sysctrl(struct ti_pipe3 *phy)
 {
        struct device *dev = phy->dev;
        struct device_node *node = dev->of_node;
        struct device_node *control_node;
        struct platform_device *control_pdev;
+       int ret;
 
        phy->phy_power_syscon = syscon_regmap_lookup_by_phandle(node,
                                                        "syscon-phy-power");
@@ -704,6 +712,11 @@ static int ti_pipe3_get_sysctrl(struct ti_pipe3 *phy)
                }
 
                phy->control_dev = &control_pdev->dev;
+
+               ret = devm_add_action_or_reset(dev, ti_pipe3_put_device,
+                                              phy->control_dev);
+               if (ret)
+                       return ret;
        }
 
        if (phy->mode == PIPE3_MODE_PCIE) {