]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
soc: qcom: pmic_glink: fix OF node leak
authorJohan Hovold <johan@kernel.org>
Tue, 8 Jul 2025 08:57:17 +0000 (10:57 +0200)
committerBjorn Andersson <andersson@kernel.org>
Thu, 17 Jul 2025 03:58:57 +0000 (22:58 -0500)
Make sure to drop the OF node reference taken when registering the
auxiliary devices when the devices are later released.

Fixes: 58ef4ece1e41 ("soc: qcom: pmic_glink: Introduce base PMIC GLINK driver")
Cc: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20250708085717.15922-1-johan@kernel.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
drivers/soc/qcom/pmic_glink.c

index 0a6d325b195cd1d8cb6444b5011c10d5d675e4c6..c0a4be5df9267d1a2ea439712089f33c3a342300 100644 (file)
@@ -167,7 +167,10 @@ static int pmic_glink_rpmsg_callback(struct rpmsg_device *rpdev, void *data,
        return 0;
 }
 
-static void pmic_glink_aux_release(struct device *dev) {}
+static void pmic_glink_aux_release(struct device *dev)
+{
+       of_node_put(dev->of_node);
+}
 
 static int pmic_glink_add_aux_device(struct pmic_glink *pg,
                                     struct auxiliary_device *aux,
@@ -181,8 +184,10 @@ static int pmic_glink_add_aux_device(struct pmic_glink *pg,
        aux->dev.release = pmic_glink_aux_release;
        device_set_of_node_from_dev(&aux->dev, parent);
        ret = auxiliary_device_init(aux);
-       if (ret)
+       if (ret) {
+               of_node_put(aux->dev.of_node);
                return ret;
+       }
 
        ret = auxiliary_device_add(aux);
        if (ret)