2021-05-19 13:43:00

by Satya Priya

Subject: [RESEND PATCH V4 1/8] arm64: dts: qcom: sm8350: Add label for thermal-zones node

Add label "thermal_zones" for thermal-zones node.

Signed-off-by: satya priya <[email protected]>
Reviewed-by: Matthias Kaehlcke <[email protected]>
---
Changes in RESEND V4:
- No Changes.

arch/arm64/boot/dts/qcom/sm8350.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi
index ed0b51b..47c6c0b 100644
--- a/arch/arm64/boot/dts/qcom/sm8350.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi
@@ -1317,7 +1317,7 @@
};
};

- thermal-zones {
+ thermal_zones: thermal-zones {
cpu0-thermal {
polling-delay-passive = <250>;
polling-delay = <1000>;
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation

2021-05-19 17:59:33

by Vinod Koul

Subject: Re: [RESEND PATCH V4 1/8] arm64: dts: qcom: sm8350: Add label for thermal-zones node

On 18-05-21, 11:50, satya priya wrote:
> Add label "thermal_zones" for thermal-zones node.
>
> Signed-off-by: satya priya <[email protected]>
> Reviewed-by: Matthias Kaehlcke <[email protected]>
> ---
> Changes in RESEND V4:
> - No Changes.
>
> arch/arm64/boot/dts/qcom/sm8350.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi
> index ed0b51b..47c6c0b 100644
> --- a/arch/arm64/boot/dts/qcom/sm8350.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi
> @@ -1317,7 +1317,7 @@
> };
> };
>
> - thermal-zones {
> + thermal_zones: thermal-zones {

is this label used anywhere in this series? If not lets drop it

> cpu0-thermal {
> polling-delay-passive = <250>;
> polling-delay = <1000>;
> --
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation

--
~Vinod

2021-05-20 06:26:07

by Satya Priya

Subject: Re: [RESEND PATCH V4 1/8] arm64: dts: qcom: sm8350: Add label for thermal-zones node

On 2021-05-18 17:13, Vinod Koul wrote:
> On 18-05-21, 11:50, satya priya wrote:
>> Add label "thermal_zones" for thermal-zones node.
>>
>> Signed-off-by: satya priya <[email protected]>
>> Reviewed-by: Matthias Kaehlcke <[email protected]>
>> ---
>> Changes in RESEND V4:
>> - No Changes.
>>
>> arch/arm64/boot/dts/qcom/sm8350.dtsi | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi
>> b/arch/arm64/boot/dts/qcom/sm8350.dtsi
>> index ed0b51b..47c6c0b 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8350.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi
>> @@ -1317,7 +1317,7 @@
>> };
>> };
>>
>> - thermal-zones {
>> + thermal_zones: thermal-zones {
>
> is this label used anywhere in this series? If not lets drop it
>

yes, it is used in pm8350c.dtsi, pmk8350.dtsi and pmr735a.dtsi files
which are included in sm8350 board dts.

>> cpu0-thermal {
>> polling-delay-passive = <250>;
>> polling-delay = <1000>;
>> --
>> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
>> member
>> of Code Aurora Forum, hosted by The Linux Foundation