usb: mtu3: Handle return value of clk_prepare_enable
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Fri, 9 Jun 2017 12:03:31 +0000 (17:33 +0530)
committerFelipe Balbi <felipe.balbi@linux.intel.com>
Tue, 13 Jun 2017 10:21:06 +0000 (13:21 +0300)
clk_prepare_enable() can fail here and we must check its return value.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
drivers/usb/mtu3/mtu3_plat.c

index 42550c7db3e790c5f5263fdf51e656fa0ecd64ac..0d3ebb353e0828ce5f28e4ce0446956e1c11eba3 100644 (file)
@@ -458,6 +458,7 @@ static int __maybe_unused mtu3_resume(struct device *dev)
 {
        struct platform_device *pdev = to_platform_device(dev);
        struct ssusb_mtk *ssusb = platform_get_drvdata(pdev);
+       int ret;
 
        dev_dbg(dev, "%s\n", __func__);
 
@@ -465,12 +466,28 @@ static int __maybe_unused mtu3_resume(struct device *dev)
                return 0;
 
        ssusb_wakeup_disable(ssusb);
-       clk_prepare_enable(ssusb->sys_clk);
-       clk_prepare_enable(ssusb->ref_clk);
-       ssusb_phy_power_on(ssusb);
+       ret = clk_prepare_enable(ssusb->sys_clk);
+       if (ret)
+               goto err_sys_clk;
+
+       ret = clk_prepare_enable(ssusb->ref_clk);
+       if (ret)
+               goto err_ref_clk;
+
+       ret = ssusb_phy_power_on(ssusb);
+       if (ret)
+               goto err_power_on;
+
        ssusb_host_enable(ssusb);
 
        return 0;
+
+err_power_on:
+       clk_disable_unprepare(ssusb->ref_clk);
+err_ref_clk:
+       clk_disable_unprepare(ssusb->sys_clk);
+err_sys_clk:
+       return ret;
 }
 
 static const struct dev_pm_ops mtu3_pm_ops = {