usb: host: xhci-plat: Improve clock handling in probe()

It is always better to acquire all the clock resources first and
then do the clock operations.

This patch acquires all the optional clocks first and then calls
corresponding prepare_enable().

There is no functional change.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/r/20230121145853.4792-8-biju.das.jz@bp.renesas.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/host/xhci-plat.c b/drivers/usb/host/xhci-plat.c
index 5fb55bf19493150f55eb02c4727a99a3999462f8..11b3a0d6722db808b9b6014c5bd89aa3da430ce4 100644 (file)
--- a/drivers/usb/host/xhci-plat.c
+++ b/drivers/usb/host/xhci-plat.c
@@ -257,16 +257,16 @@ static int xhci_plat_probe(struct platform_device *pdev)
                goto put_hcd;
        }
 
-       ret = clk_prepare_enable(xhci->reg_clk);
-       if (ret)
-               goto put_hcd;
-
        xhci->clk = devm_clk_get_optional(&pdev->dev, NULL);
        if (IS_ERR(xhci->clk)) {
                ret = PTR_ERR(xhci->clk);
-               goto disable_reg_clk;
+               goto put_hcd;
        }
 
+       ret = clk_prepare_enable(xhci->reg_clk);
+       if (ret)
+               goto put_hcd;
+
        ret = clk_prepare_enable(xhci->clk);
        if (ret)
                goto disable_reg_clk;