2023-05-15 10:33:02

by Varadarajan Narayanan

[permalink] [raw]
Subject: [PATCH v3 0/4] Enable IPQ9574 TSENS support

This patch set enables tsens in IPQ9574

Depends on
https://lore.kernel.org/linux-arm-msm/[email protected]/
[v3]:
Fix make DT_CHECKER_FLAGS=-m dt_binding_check and make dtbs_check errors

[v2]:
Drop the driver change (https://lore.kernel.org/lkml/b45d33d38a334aabbd52c83b0d6028af1f4c74c8.1682682753.git.quic_varada@quicinc.com/)
since the tsens device is compatible with 8074's tsens
and use 8074's compatible itself

Rename clusterX nodes as cpussX

[v1]:
Fix DT node names

[v0]:
Initial patch introducing TSENS support

Praveenkumar I (1):
dt-bindings: thermal: tsens: Add ipq9574 compatible

Varadarajan Narayanan (3):
dt-bindings: thermal: tsens: Fix "make dtbs_check" error
arm64: dts: qcom: ipq9574: add tsens node
arm64: dts: qcom: ipq9574: add thermal zone nodes

.../devicetree/bindings/thermal/qcom-tsens.yaml | 15 +-
arch/arm64/boot/dts/qcom/ipq9574.dtsi | 217 +++++++++++++++++++++
2 files changed, 230 insertions(+), 2 deletions(-)

--
2.7.4



2023-05-15 10:34:31

by Varadarajan Narayanan

[permalink] [raw]
Subject: [PATCH v3 1/4] dt-bindings: thermal: tsens: Fix "make dtbs_check" error

While verifying make dtbs_check for ipq9574, qcm2290-tsens and
sm6375-tsens threw the following errors.
['qcom,qcm2290-tsens', 'qcom,tsens-v2'] is too long
...
['qcom,sm6375-tsens', 'qcom,tsens-v2'] is too long

Fix them by adding their entries.

Signed-off-by: Varadarajan Narayanan <[email protected]>
---
[v3]:
Fix the following 'make dtbs_check' error
['qcom,qcm2290-tsens', 'qcom,tsens-v2'] is too long
['qcom,sm6375-tsens', 'qcom,tsens-v2'] is too long
---
Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 2 ++
1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
index d1ec963..d9aa54c 100644
--- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
+++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
@@ -48,6 +48,7 @@ properties:
- qcom,msm8953-tsens
- qcom,msm8996-tsens
- qcom,msm8998-tsens
+ - qcom,qcm2290-tsens
- qcom,sc7180-tsens
- qcom,sc7280-tsens
- qcom,sc8180x-tsens
@@ -56,6 +57,7 @@ properties:
- qcom,sdm845-tsens
- qcom,sm6115-tsens
- qcom,sm6350-tsens
+ - qcom,sm6375-tsens
- qcom,sm8150-tsens
- qcom,sm8250-tsens
- qcom,sm8350-tsens
--
2.7.4


2023-05-15 16:17:12

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v3 1/4] dt-bindings: thermal: tsens: Fix "make dtbs_check" error

On 15/05/2023 12:13, Varadarajan Narayanan wrote:
> While verifying make dtbs_check for ipq9574, qcm2290-tsens and
> sm6375-tsens threw the following errors.
> ['qcom,qcm2290-tsens', 'qcom,tsens-v2'] is too long
> ...
> ['qcom,sm6375-tsens', 'qcom,tsens-v2'] is too long

Subject: Fix error can be anything, so this should be specific, e.g.
Document missing QCM2290


https://lore.kernel.org/all/[email protected]/

Best regards,
Krzysztof


2023-05-16 01:16:47

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH v3 1/4] dt-bindings: thermal: tsens: Fix "make dtbs_check" error



On 15.05.2023 12:13, Varadarajan Narayanan wrote:
> While verifying make dtbs_check for ipq9574, qcm2290-tsens and
> sm6375-tsens threw the following errors.
> ['qcom,qcm2290-tsens', 'qcom,tsens-v2'] is too long
> ...
> ['qcom,sm6375-tsens', 'qcom,tsens-v2'] is too long
>
> Fix them by adding their entries.
>
> Signed-off-by: Varadarajan Narayanan <[email protected]>
> ---
https://lore.kernel.org/linux-arm-msm/[email protected]/
https://lore.kernel.org/linux-arm-msm/[email protected]/

These never got picked up..

I'll bump them.

Konrad
> [v3]:
> Fix the following 'make dtbs_check' error
> ['qcom,qcm2290-tsens', 'qcom,tsens-v2'] is too long
> ['qcom,sm6375-tsens', 'qcom,tsens-v2'] is too long
> ---
> Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> index d1ec963..d9aa54c 100644
> --- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> +++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> @@ -48,6 +48,7 @@ properties:
> - qcom,msm8953-tsens
> - qcom,msm8996-tsens
> - qcom,msm8998-tsens
> + - qcom,qcm2290-tsens
> - qcom,sc7180-tsens
> - qcom,sc7280-tsens
> - qcom,sc8180x-tsens
> @@ -56,6 +57,7 @@ properties:
> - qcom,sdm845-tsens
> - qcom,sm6115-tsens
> - qcom,sm6350-tsens
> + - qcom,sm6375-tsens
> - qcom,sm8150-tsens
> - qcom,sm8250-tsens
> - qcom,sm8350-tsens