[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b3228ab0-63c0-4cb4-9671-87601a7bed9b@quicinc.com>
Date: Fri, 28 Mar 2025 16:11:44 +0530
From: Jagadeesh Kona <quic_jkona@...cinc.com>
To: Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
Bjorn Andersson
<andersson@...nel.org>,
Michael Turquette <mturquette@...libre.com>,
"Stephen
Boyd" <sboyd@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski
<krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Konrad Dybcio
<konradybcio@...nel.org>,
Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>,
Dmitry Baryshkov <lumag@...nel.org>
CC: Ajit Pandey <quic_ajipan@...cinc.com>,
Imran Shaik
<quic_imrashai@...cinc.com>,
Taniya Das <quic_tdas@...cinc.com>,
"Satya Priya
Kakitapalli" <quic_skakitap@...cinc.com>,
<linux-arm-msm@...r.kernel.org>, <linux-clk@...r.kernel.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
Krzysztof Kozlowski
<krzysztof.kozlowski@...aro.org>
Subject: Re: [PATCH v3 05/18] clk: qcom: common: Handle runtime power
management in qcom_cc_really_probe
On 3/27/2025 9:28 PM, Bryan O'Donoghue wrote:
> On 27/03/2025 09:52, Jagadeesh Kona wrote:
>> - return ret;
>> + goto put_rpm;
>> +
>> + ret = qcom_cc_icc_register(dev, desc);
>> +
>> +put_rpm:
>> + if (desc->use_rpm)
>> + pm_runtime_put(dev);
>> - return qcom_cc_icc_register(dev, desc);
>> + return ret;
>> }
>> EXPORT_SYMBOL_GPL(qcom_cc_really_probe);
>
> Doesn't look right you're missing the put if register goes wrong
>
The intention is to call pm_runtime_put() regardless of the return value
from qcom_cc_icc_register(), as it is the final API call. Therefore, the
return type is not checked, and pm_runtime_put() is called in both success
and failure cases before returning the final return code.
Thanks,
Jagadeesh
> ret = qcom_cc_icc_register(dev, desc);
>
> if (ret)
> goto put_rpm;
>
> return 0;
>
> put_rpm:
> if (desc->us_rpm)
> pm_runtime_put();
>
> return ret;
Powered by blists - more mailing lists