]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
phy: tegra: xusb: fix device and OF node leak at probe
authorJohan Hovold <johan@kernel.org>
Thu, 24 Jul 2025 13:12:04 +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 references taken to the PMC OF node and device by
of_parse_phandle() and of_find_device_by_node() during probe.

Note the holding a reference to the PMC device does not prevent the
PMC regmap from going away (e.g. if the PMC driver is unbound) so there
is no need to keep the reference.

Fixes: 2d1021487273 ("phy: tegra: xusb: Add wake/sleepwalk for Tegra210")
Cc: stable@vger.kernel.org # 5.14
Cc: JC Kuo <jckuo@nvidia.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
Link: https://lore.kernel.org/r/20250724131206.2211-2-johan@kernel.org
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/tegra/xusb-tegra210.c

index ebc8a7e21a318160b162113eea8a6c97b7ed7966..3409924498e9cf5e76d1bd2a203d4f195241ee11 100644 (file)
@@ -3164,18 +3164,22 @@ tegra210_xusb_padctl_probe(struct device *dev,
        }
 
        pdev = of_find_device_by_node(np);
+       of_node_put(np);
        if (!pdev) {
                dev_warn(dev, "PMC device is not available\n");
                goto out;
        }
 
-       if (!platform_get_drvdata(pdev))
+       if (!platform_get_drvdata(pdev)) {
+               put_device(&pdev->dev);
                return ERR_PTR(-EPROBE_DEFER);
+       }
 
        padctl->regmap = dev_get_regmap(&pdev->dev, "usb_sleepwalk");
        if (!padctl->regmap)
                dev_info(dev, "failed to find PMC regmap\n");
 
+       put_device(&pdev->dev);
 out:
        return &padctl->base;
 }