[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 12 Dec 2019 07:17:42 +0000
From: rkambl@...eaurora.org
To: Doug Anderson <dianders@...omium.org>
Cc: Andy Gross <agross@...nel.org>,
Bjorn Andersson <bjorn.andersson@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
linux-arm-msm <linux-arm-msm@...r.kernel.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@...r.kernel.org>, LKML <linux-kernel@...r.kernel.org>,
sivaa@...eaurora.org, sanm@...eaurora.org,
Amit Kucheria <amit.kucheria@...aro.org>,
Matthias Kaehlcke <mka@...omium.org>
Subject: Re: [PATCH v2 1/1] arm64: dts: qcom: sc7180: Add device node support
for TSENS in SC7180
On 2019-12-12 01:03, Doug Anderson wrote:
> Hi,
>
> On Thu, Nov 28, 2019 at 1:55 AM Rajeshwari <rkambl@...eaurora.org>
> wrote:
>>
>> Add TSENS node and user thermal zone for TSENS sensors in SC7180.
>>
>> Signed-off-by: Rajeshwari <rkambl@...eaurora.org>
>> ---
>> arch/arm64/boot/dts/qcom/sc7180.dtsi | 527
>> +++++++++++++++++++++++++++++++++++
>> 1 file changed, 527 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> b/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> index 666e9b9..6656ffc 100644
>> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> @@ -911,6 +911,26 @@
>> status = "disabled";
>> };
>>
>> + tsens0: thermal-sensor@...3000 {
>> + compatible =
>> "qcom,sc7180-tsens","qcom,tsens-v2";
>> + reg = <0 0x0c263000 0 0x1ff>, /* TM */
>> + <0 0x0c222000 0 0x1ff>; /* SROT */
>> + #qcom,sensors = <15>;
>> + interrupts = <GIC_SPI 506
>> IRQ_TYPE_LEVEL_HIGH>;
>> + interrupt-names = "uplow";
>
> Can you also send a patch to match Amit's ("arm64: dts: sdm845:
> thermal: Add critical interrupt support") [1]. If I'm reading things
> correctly it should be 508 for tsens0 and 509 for tsens1 just like on
> sdm845.
>
> [1]
> https://lore.kernel.org/r/c536e9cdb448bbad3441f6580fa57f1f921fb580.1573499020.git.amit.kucheria@linaro.org
yes, i will send the patch for critical interrupt support.
>
>
> Thanks!
>
> -Rajeshwari
Powered by blists - more mailing lists