<div dir="ltr"><br><br>On Tue, Mar 16, 2021 at 3:21 AM 'w00385741 <<a href="mailto:weiyongjun1@huawei.com">weiyongjun1@huawei.com</a>> wrote:<br>><br>> -               if (ret)<br>> +               if (ret) {<br>> +                       mutex_unlock(&priv->lock);<br>>                         return ret;<br>> +               }<br>>         }<br>>         mutex_unlock(&priv->lock);<br>><br><br>I think you can let it go and check ret after unlocking:<div><br><span style="font-family:monospace"><span style="color:rgb(0,0,0)">                /* Disable clock in case of PWM is off */
</span><br>                clk_disable_unprepare(priv->plat->clk_ptp_ref);
<br>                ret = pm_runtime_force_suspend(dev);
<br>        }
<br>        mutex_unlock(&priv->lock);
<br>        if (ret)
<br>                return ret;
<br>
<br>        priv->speed = SPEED_UNKNOWN;
<br>        return 0;<br>
<br></span></div></div>