All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550
@ 2022-11-16 10:09 Neil Armstrong
  2022-11-16 11:26 ` Krzysztof Kozlowski
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Neil Armstrong @ 2022-11-16 10:09 UTC (permalink / raw)
  To: Bjorn Andersson, Krzysztof Kozlowski, Daniel Lezcano, Andy Gross,
	Zhang Rui, Konrad Dybcio, Thara Gopinath, Rob Herring,
	Amit Kucheria, Rafael J. Wysocki
  Cc: Neil Armstrong, linux-kernel, linux-arm-msm, devicetree, linux-pm

The Qualcomm SM8550 platform has three instances of the tsens block,
add a compatible for these instances.

Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
To: Andy Gross <agross@kernel.org>
To: Bjorn Andersson <andersson@kernel.org>
To: Konrad Dybcio <konrad.dybcio@somainline.org>
To: Amit Kucheria <amitk@kernel.org>
To: Thara Gopinath <thara.gopinath@gmail.com>
To: "Rafael J. Wysocki" <rafael@kernel.org>
To: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Zhang Rui <rui.zhang@intel.com>
To: Rob Herring <robh+dt@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-pm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
---
 Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
index f0bd4b979e28..09dbd96d380e 100644
--- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
+++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
@@ -58,6 +58,7 @@ properties:
               - qcom,sm8250-tsens
               - qcom,sm8350-tsens
               - qcom,sm8450-tsens
+              - qcom,sm8550-tsens
           - const: qcom,tsens-v2
 
       - description: v2 of TSENS with combined interrupt

---
base-commit: 3c1f24109dfc4fb1a3730ed237e50183c6bb26b3
change-id: 20221114-narmstrong-sm8550-upstream-tsens-dfaec218c321

Best regards,
-- 
Neil Armstrong <neil.armstrong@linaro.org>

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550
  2022-11-16 10:09 [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550 Neil Armstrong
@ 2022-11-16 11:26 ` Krzysztof Kozlowski
  2022-11-17  8:06   ` Neil Armstrong
  2022-11-20  6:52 ` Amit Kucheria
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 7+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-16 11:26 UTC (permalink / raw)
  To: Neil Armstrong, Bjorn Andersson, Krzysztof Kozlowski,
	Daniel Lezcano, Andy Gross, Zhang Rui, Konrad Dybcio,
	Thara Gopinath, Rob Herring, Amit Kucheria, Rafael J. Wysocki
  Cc: linux-kernel, linux-arm-msm, devicetree, linux-pm

On 16/11/2022 11:09, Neil Armstrong wrote:
> The Qualcomm SM8550 platform has three instances of the tsens block,
> add a compatible for these instances.
> 
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
> To: Andy Gross <agross@kernel.org>
> To: Bjorn Andersson <andersson@kernel.org>
> To: Konrad Dybcio <konrad.dybcio@somainline.org>
> To: Amit Kucheria <amitk@kernel.org>
> To: Thara Gopinath <thara.gopinath@gmail.com>
> To: "Rafael J. Wysocki" <rafael@kernel.org>
> To: Daniel Lezcano <daniel.lezcano@linaro.org>
> To: Zhang Rui <rui.zhang@intel.com>
> To: Rob Herring <robh+dt@kernel.org>
> To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: linux-pm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> ---
>  Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> index f0bd4b979e28..09dbd96d380e 100644
> --- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> +++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> @@ -58,6 +58,7 @@ properties:
>                - qcom,sm8250-tsens
>                - qcom,sm8350-tsens
>                - qcom,sm8450-tsens
> +              - qcom,sm8550-tsens

alOf:if:then should be updated.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550
  2022-11-16 11:26 ` Krzysztof Kozlowski
@ 2022-11-17  8:06   ` Neil Armstrong
  2022-11-17 12:58     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 7+ messages in thread
From: Neil Armstrong @ 2022-11-17  8:06 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Bjorn Andersson, Krzysztof Kozlowski,
	Daniel Lezcano, Andy Gross, Zhang Rui, Konrad Dybcio,
	Thara Gopinath, Rob Herring, Amit Kucheria, Rafael J. Wysocki
  Cc: linux-kernel, linux-arm-msm, devicetree, linux-pm

Hi,

On 16/11/2022 12:26, Krzysztof Kozlowski wrote:
> On 16/11/2022 11:09, Neil Armstrong wrote:
>> The Qualcomm SM8550 platform has three instances of the tsens block,
>> add a compatible for these instances.
>>
>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>> ---
>> To: Andy Gross <agross@kernel.org>
>> To: Bjorn Andersson <andersson@kernel.org>
>> To: Konrad Dybcio <konrad.dybcio@somainline.org>
>> To: Amit Kucheria <amitk@kernel.org>
>> To: Thara Gopinath <thara.gopinath@gmail.com>
>> To: "Rafael J. Wysocki" <rafael@kernel.org>
>> To: Daniel Lezcano <daniel.lezcano@linaro.org>
>> To: Zhang Rui <rui.zhang@intel.com>
>> To: Rob Herring <robh+dt@kernel.org>
>> To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
>> Cc: linux-arm-msm@vger.kernel.org
>> Cc: linux-pm@vger.kernel.org
>> Cc: devicetree@vger.kernel.org
>> Cc: linux-kernel@vger.kernel.org
>> ---
>>   Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
>> index f0bd4b979e28..09dbd96d380e 100644
>> --- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
>> +++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
>> @@ -58,6 +58,7 @@ properties:
>>                 - qcom,sm8250-tsens
>>                 - qcom,sm8350-tsens
>>                 - qcom,sm8450-tsens
>> +              - qcom,sm8550-tsens
> 
> alOf:if:then should be updated.

I thought the -v2 fallback would be enough since 8450 isn't present either in the alOf:if:then either.

Anyway, will fix this.

Thanks,
Neil

> 
> Best regards,
> Krzysztof
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550
  2022-11-17  8:06   ` Neil Armstrong
@ 2022-11-17 12:58     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-17 12:58 UTC (permalink / raw)
  To: neil.armstrong, Bjorn Andersson, Krzysztof Kozlowski,
	Daniel Lezcano, Andy Gross, Zhang Rui, Konrad Dybcio,
	Thara Gopinath, Rob Herring, Amit Kucheria, Rafael J. Wysocki
  Cc: linux-kernel, linux-arm-msm, devicetree, linux-pm

On 17/11/2022 09:06, Neil Armstrong wrote:
> Hi,
> 
> On 16/11/2022 12:26, Krzysztof Kozlowski wrote:
>> On 16/11/2022 11:09, Neil Armstrong wrote:
>>> The Qualcomm SM8550 platform has three instances of the tsens block,
>>> add a compatible for these instances.
>>>
>>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>>> ---
>>> To: Andy Gross <agross@kernel.org>
>>> To: Bjorn Andersson <andersson@kernel.org>
>>> To: Konrad Dybcio <konrad.dybcio@somainline.org>
>>> To: Amit Kucheria <amitk@kernel.org>
>>> To: Thara Gopinath <thara.gopinath@gmail.com>
>>> To: "Rafael J. Wysocki" <rafael@kernel.org>
>>> To: Daniel Lezcano <daniel.lezcano@linaro.org>
>>> To: Zhang Rui <rui.zhang@intel.com>
>>> To: Rob Herring <robh+dt@kernel.org>
>>> To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
>>> Cc: linux-arm-msm@vger.kernel.org
>>> Cc: linux-pm@vger.kernel.org
>>> Cc: devicetree@vger.kernel.org
>>> Cc: linux-kernel@vger.kernel.org
>>> ---
>>>   Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 +
>>>   1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
>>> index f0bd4b979e28..09dbd96d380e 100644
>>> --- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
>>> +++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
>>> @@ -58,6 +58,7 @@ properties:
>>>                 - qcom,sm8250-tsens
>>>                 - qcom,sm8350-tsens
>>>                 - qcom,sm8450-tsens
>>> +              - qcom,sm8550-tsens
>>
>> alOf:if:then should be updated.
> 
> I thought the -v2 fallback would be enough since 8450 isn't present either in the alOf:if:then either.
> 
> Anyway, will fix this.


No, you are right. The v2 fallback is already in several allOf:if:then,
so this should be fine.


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550
  2022-11-16 10:09 [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550 Neil Armstrong
  2022-11-16 11:26 ` Krzysztof Kozlowski
@ 2022-11-20  6:52 ` Amit Kucheria
  2022-12-06 17:24 ` Daniel Lezcano
  2022-12-09 15:26 ` [thermal: thermal/next] " thermal-bot for Neil Armstrong
  3 siblings, 0 replies; 7+ messages in thread
From: Amit Kucheria @ 2022-11-20  6:52 UTC (permalink / raw)
  To: Neil Armstrong
  Cc: Bjorn Andersson, Krzysztof Kozlowski, Daniel Lezcano, Andy Gross,
	Zhang Rui, Konrad Dybcio, Thara Gopinath, Rob Herring,
	Rafael J. Wysocki, linux-kernel, linux-arm-msm, devicetree,
	linux-pm

On Wed, Nov 16, 2022 at 3:40 PM Neil Armstrong
<neil.armstrong@linaro.org> wrote:
>
> The Qualcomm SM8550 platform has three instances of the tsens block,
> add a compatible for these instances.
>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>

Acked-by: Amit Kucheria <amitk@kernel.org>

> ---
> To: Andy Gross <agross@kernel.org>
> To: Bjorn Andersson <andersson@kernel.org>
> To: Konrad Dybcio <konrad.dybcio@somainline.org>
> To: Amit Kucheria <amitk@kernel.org>
> To: Thara Gopinath <thara.gopinath@gmail.com>
> To: "Rafael J. Wysocki" <rafael@kernel.org>
> To: Daniel Lezcano <daniel.lezcano@linaro.org>
> To: Zhang Rui <rui.zhang@intel.com>
> To: Rob Herring <robh+dt@kernel.org>
> To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: linux-pm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> ---
>  Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> index f0bd4b979e28..09dbd96d380e 100644
> --- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> +++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
> @@ -58,6 +58,7 @@ properties:
>                - qcom,sm8250-tsens
>                - qcom,sm8350-tsens
>                - qcom,sm8450-tsens
> +              - qcom,sm8550-tsens
>            - const: qcom,tsens-v2
>
>        - description: v2 of TSENS with combined interrupt
>
> ---
> base-commit: 3c1f24109dfc4fb1a3730ed237e50183c6bb26b3
> change-id: 20221114-narmstrong-sm8550-upstream-tsens-dfaec218c321
>
> Best regards,
> --
> Neil Armstrong <neil.armstrong@linaro.org>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550
  2022-11-16 10:09 [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550 Neil Armstrong
  2022-11-16 11:26 ` Krzysztof Kozlowski
  2022-11-20  6:52 ` Amit Kucheria
@ 2022-12-06 17:24 ` Daniel Lezcano
  2022-12-09 15:26 ` [thermal: thermal/next] " thermal-bot for Neil Armstrong
  3 siblings, 0 replies; 7+ messages in thread
From: Daniel Lezcano @ 2022-12-06 17:24 UTC (permalink / raw)
  To: Neil Armstrong, Bjorn Andersson, Krzysztof Kozlowski, Andy Gross,
	Zhang Rui, Konrad Dybcio, Thara Gopinath, Rob Herring,
	Amit Kucheria, Rafael J. Wysocki
  Cc: linux-kernel, linux-arm-msm, devicetree, linux-pm

On 16/11/2022 11:09, Neil Armstrong wrote:
> The Qualcomm SM8550 platform has three instances of the tsens block,
> add a compatible for these instances.
> 
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---

Applied, thanks

-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [thermal: thermal/next] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550
  2022-11-16 10:09 [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550 Neil Armstrong
                   ` (2 preceding siblings ...)
  2022-12-06 17:24 ` Daniel Lezcano
@ 2022-12-09 15:26 ` thermal-bot for Neil Armstrong
  3 siblings, 0 replies; 7+ messages in thread
From: thermal-bot for Neil Armstrong @ 2022-12-09 15:26 UTC (permalink / raw)
  To: linux-pm
  Cc: Neil Armstrong, Krzysztof Kozlowski, Amit Kucheria,
	Daniel Lezcano, rui.zhang, daniel.lezcano

The following commit has been merged into the thermal/next branch of thermal:

Commit-ID:     2d709c4f0520bafba4a797b116d3504fc70af99c
Gitweb:        https://git.kernel.org/pub/scm/linux/kernel/git/thermal/linux.git//2d709c4f0520bafba4a797b116d3504fc70af99c
Author:        Neil Armstrong <neil.armstrong@linaro.org>
AuthorDate:    Wed, 16 Nov 2022 11:09:50 +01:00
Committer:     Daniel Lezcano <daniel.lezcano@kernel.org>
CommitterDate: Thu, 08 Dec 2022 14:30:45 +01:00

dt-bindings: thermal: qcom-tsens: Add compatible for sm8550

The Qualcomm SM8550 platform has three instances of the tsens block,
add a compatible for these instances.

Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Amit Kucheria <amitk@kernel.org>
Link: https://lore.kernel.org/r/20221114-narmstrong-sm8550-upstream-tsens-v1-0-0e169822830f@linaro.org
Signed-off-by: Daniel Lezcano <daniel.lezcano@kernel.org>
---
 Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
index 5bcfddc..7e04804 100644
--- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
+++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml
@@ -58,6 +58,7 @@ properties:
               - qcom,sm8250-tsens
               - qcom,sm8350-tsens
               - qcom,sm8450-tsens
+              - qcom,sm8550-tsens
           - const: qcom,tsens-v2
 
       - description: v2 of TSENS with combined interrupt

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2022-12-09 15:26 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-16 10:09 [PATCH] dt-bindings: thermal: qcom-tsens: Add compatible for sm8550 Neil Armstrong
2022-11-16 11:26 ` Krzysztof Kozlowski
2022-11-17  8:06   ` Neil Armstrong
2022-11-17 12:58     ` Krzysztof Kozlowski
2022-11-20  6:52 ` Amit Kucheria
2022-12-06 17:24 ` Daniel Lezcano
2022-12-09 15:26 ` [thermal: thermal/next] " thermal-bot for Neil Armstrong

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.