[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <46b8a66b-5cb7-2ad1-d633-f393696fb238@quicinc.com>
Date: Tue, 19 Sep 2023 12:56:20 +0530
From: Sricharan Ramabadhran <quic_srichara@...cinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
CC: <krzysztof.kozlowski@...aro.org>, <agross@...nel.org>,
<andersson@...nel.org>, <konrad.dybcio@...aro.org>,
<srinivas.kandagatla@...aro.org>, <robh+dt@...nel.org>,
<krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>,
<thara.gopinath@...il.com>, <rafael@...nel.org>,
<daniel.lezcano@...aro.org>, <linux-arm-msm@...r.kernel.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<linux-pm@...r.kernel.org>
Subject: Re: [PATCH V2 3/4] thermal/drivers/tsens: Add support for IPQ5018
tsens
<..>
>> +static int init_ipq5018(struct tsens_priv *priv)
>> +{
>> + int ret;
>> + u32 mask;
>> +
>> + init_common(priv);
>
> Please take care of init_common()'s return code. Don't continue init
> if init_common() fails.
ok
>
>> + if (!priv->tm_map)
>> + return -ENODEV;
>
> Why?
>
ok, redundant, will remove.
Regards,
Sricharan
Powered by blists - more mailing lists