PCI: mvebu: Use devm_add_action_or_reset() instead of devm_add_action()

Replace devm_add_action() with devm_add_action_or_reset() to avoid
explicitly dropping the 'port->clk' reference in error path.

Signed-off-by: Salah Triki <salah.triki@gmail.com>
[mani: reworded commit subject and description]
Signed-off-by: Manivannan Sadhasivam <mani@kernel.org>
Link: https://patch.msgid.link/aHsgYALHfQbrgq0t@pc

diff --git a/drivers/pci/controller/pci-mvebu.c b/drivers/pci/controller/pci-mvebu.c
index a4a2bac4f4b279c818ec6e68e8497294ea56e2cc..755651f33811d7daf277247e29a5741cedeec8d0 100644 (file)
--- a/drivers/pci/controller/pci-mvebu.c
+++ b/drivers/pci/controller/pci-mvebu.c
@@ -1353,11 +1353,9 @@ static int mvebu_pcie_parse_port(struct mvebu_pcie *pcie,
                goto skip;
        }
 
-       ret = devm_add_action(dev, mvebu_pcie_port_clk_put, port);
-       if (ret < 0) {
-               clk_put(port->clk);
+       ret = devm_add_action_or_reset(dev, mvebu_pcie_port_clk_put, port);
+       if (ret < 0)
                goto err;
-       }
 
        return 1;