Linux Power Management development
 help / color / mirror / Atom feed
* [PATCH 0/2] arm64: dts: qcom: sc8180x: Fix CPUfreq compatible
@ 2024-05-25 17:48 Bjorn Andersson
  2024-05-25 17:48 ` [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add sc8180x compatible Bjorn Andersson
  2024-05-25 17:48 ` [PATCH 2/2] arm64: dts: qcom: sc8180x: Correct cpufreq compatible Bjorn Andersson
  0 siblings, 2 replies; 7+ messages in thread
From: Bjorn Andersson @ 2024-05-25 17:48 UTC (permalink / raw)
  To: Rafael J. Wysocki, Viresh Kumar, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Manivannan Sadhasivam, Bjorn Andersson,
	Konrad Dybcio
  Cc: linux-pm, devicetree, linux-kernel, linux-arm-msm,
	Bjorn Andersson

Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com>
---
Bjorn Andersson (2):
      dt-bindings: cpufreq: cpufreq-qcom-hw: Add sc8180x compatible
      arm64: dts: qcom: sc8180x: Correct cpufreq compatible

 Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml | 1 +
 arch/arm64/boot/dts/qcom/sc8180x.dtsi                          | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)
---
base-commit: 3689b0ef08b70e4e03b82ebd37730a03a672853a
change-id: 20240525-sc8180x-cpufreq-compatible-633c45b93886

Best regards,
-- 
Bjorn Andersson <quic_bjorande@quicinc.com>


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

* [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add sc8180x compatible
  2024-05-25 17:48 [PATCH 0/2] arm64: dts: qcom: sc8180x: Fix CPUfreq compatible Bjorn Andersson
@ 2024-05-25 17:48 ` Bjorn Andersson
  2024-05-26 18:23   ` Krzysztof Kozlowski
  2024-05-25 17:48 ` [PATCH 2/2] arm64: dts: qcom: sc8180x: Correct cpufreq compatible Bjorn Andersson
  1 sibling, 1 reply; 7+ messages in thread
From: Bjorn Andersson @ 2024-05-25 17:48 UTC (permalink / raw)
  To: Rafael J. Wysocki, Viresh Kumar, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Manivannan Sadhasivam, Bjorn Andersson,
	Konrad Dybcio
  Cc: linux-pm, devicetree, linux-kernel, linux-arm-msm,
	Bjorn Andersson

From: Bjorn Andersson <quic_bjorande@quicinc.com>

The SC8180X platform has a EPSS block, add a compatible for the SC8180X
platform.

Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com>
---
 Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml b/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
index 1e9797f96410..24c02ee33266 100644
--- a/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
+++ b/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
@@ -36,6 +36,7 @@ properties:
               - qcom,qdu1000-cpufreq-epss
               - qcom,sa8775p-cpufreq-epss
               - qcom,sc7280-cpufreq-epss
+              - qcom,sc8180x-cpufreq-epss
               - qcom,sc8280xp-cpufreq-epss
               - qcom,sdx75-cpufreq-epss
               - qcom,sm4450-cpufreq-epss

-- 
2.43.0


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

* [PATCH 2/2] arm64: dts: qcom: sc8180x: Correct cpufreq compatible
  2024-05-25 17:48 [PATCH 0/2] arm64: dts: qcom: sc8180x: Fix CPUfreq compatible Bjorn Andersson
  2024-05-25 17:48 ` [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add sc8180x compatible Bjorn Andersson
@ 2024-05-25 17:48 ` Bjorn Andersson
  2024-05-25 20:11   ` Dmitry Baryshkov
  1 sibling, 1 reply; 7+ messages in thread
From: Bjorn Andersson @ 2024-05-25 17:48 UTC (permalink / raw)
  To: Rafael J. Wysocki, Viresh Kumar, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Manivannan Sadhasivam, Bjorn Andersson,
	Konrad Dybcio
  Cc: linux-pm, devicetree, linux-kernel, linux-arm-msm,
	Bjorn Andersson

From: Bjorn Andersson <quic_bjorande@quicinc.com>

The Qualcomm EPSS CPUfreq binding requires a platform-specific
compatible, add this.

Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com>
---
 arch/arm64/boot/dts/qcom/sc8180x.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8180x.dtsi b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
index 067712310560..1f103cb54f99 100644
--- a/arch/arm64/boot/dts/qcom/sc8180x.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8180x.dtsi
@@ -3733,7 +3733,7 @@ lmh@18358800 {
 		};
 
 		cpufreq_hw: cpufreq@18323000 {
-			compatible = "qcom,cpufreq-hw";
+			compatible = "qcom,sc8180x-cpufreq-epss", "qcom,cpufreq-epss";
 			reg = <0 0x18323000 0 0x1400>, <0 0x18325800 0 0x1400>;
 			reg-names = "freq-domain0", "freq-domain1";
 

-- 
2.43.0


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

* Re: [PATCH 2/2] arm64: dts: qcom: sc8180x: Correct cpufreq compatible
  2024-05-25 17:48 ` [PATCH 2/2] arm64: dts: qcom: sc8180x: Correct cpufreq compatible Bjorn Andersson
@ 2024-05-25 20:11   ` Dmitry Baryshkov
  0 siblings, 0 replies; 7+ messages in thread
From: Dmitry Baryshkov @ 2024-05-25 20:11 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Rafael J. Wysocki, Viresh Kumar, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Manivannan Sadhasivam, Konrad Dybcio, linux-pm,
	devicetree, linux-kernel, linux-arm-msm, Bjorn Andersson

On Sat, May 25, 2024 at 10:48:51AM -0700, Bjorn Andersson wrote:
> From: Bjorn Andersson <quic_bjorande@quicinc.com>
> 
> The Qualcomm EPSS CPUfreq binding requires a platform-specific
> compatible, add this.
> 
> Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com>
> ---
>  arch/arm64/boot/dts/qcom/sc8180x.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

-- 
With best wishes
Dmitry

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

* Re: [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add sc8180x compatible
  2024-05-25 17:48 ` [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add sc8180x compatible Bjorn Andersson
@ 2024-05-26 18:23   ` Krzysztof Kozlowski
  2024-05-26 20:35     ` Bjorn Andersson
  0 siblings, 1 reply; 7+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-26 18:23 UTC (permalink / raw)
  To: Bjorn Andersson, Rafael J. Wysocki, Viresh Kumar, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
	Konrad Dybcio
  Cc: linux-pm, devicetree, linux-kernel, linux-arm-msm,
	Bjorn Andersson

On 25/05/2024 19:48, Bjorn Andersson wrote:
> From: Bjorn Andersson <quic_bjorande@quicinc.com>
> 
> The SC8180X platform has a EPSS block, add a compatible for the SC8180X
> platform.
> 
> Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com>
> ---
>  Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml b/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
> index 1e9797f96410..24c02ee33266 100644
> --- a/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
> +++ b/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
> @@ -36,6 +36,7 @@ properties:
>                - qcom,qdu1000-cpufreq-epss
>                - qcom,sa8775p-cpufreq-epss
>                - qcom,sc7280-cpufreq-epss
> +              - qcom,sc8180x-cpufreq-epss

I think you forgot to update the allOf:if:then: entry.

Best regards,
Krzysztof


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

* Re: [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add sc8180x compatible
  2024-05-26 18:23   ` Krzysztof Kozlowski
@ 2024-05-26 20:35     ` Bjorn Andersson
  0 siblings, 0 replies; 7+ messages in thread
From: Bjorn Andersson @ 2024-05-26 20:35 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Rafael J. Wysocki, Viresh Kumar, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Manivannan Sadhasivam, Konrad Dybcio, linux-pm,
	devicetree, linux-kernel, linux-arm-msm, Bjorn Andersson

On Sun, May 26, 2024 at 08:23:24PM GMT, Krzysztof Kozlowski wrote:
> On 25/05/2024 19:48, Bjorn Andersson wrote:
> > From: Bjorn Andersson <quic_bjorande@quicinc.com>
> > 
> > The SC8180X platform has a EPSS block, add a compatible for the SC8180X
> > platform.
> > 
> > Signed-off-by: Bjorn Andersson <quic_bjorande@quicinc.com>
> > ---
> >  Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml b/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
> > index 1e9797f96410..24c02ee33266 100644
> > --- a/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
> > +++ b/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
> > @@ -36,6 +36,7 @@ properties:
> >                - qcom,qdu1000-cpufreq-epss
> >                - qcom,sa8775p-cpufreq-epss
> >                - qcom,sc7280-cpufreq-epss
> > +              - qcom,sc8180x-cpufreq-epss
> 
> I think you forgot to update the allOf:if:then: entry.
> 

Indeed, failed to spot that.

Thanks,
Bjorn

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

* [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add SC8180X compatible
  2024-11-16 11:31 [PATCH 0/2] Fix SC8180X cpufreq dt-bindings error Konrad Dybcio
@ 2024-11-16 11:31 ` Konrad Dybcio
  0 siblings, 0 replies; 7+ messages in thread
From: Konrad Dybcio @ 2024-11-16 11:31 UTC (permalink / raw)
  To: Rafael J. Wysocki, Viresh Kumar, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Manivannan Sadhasivam, Bjorn Andersson,
	Konrad Dybcio, Vinod Koul
  Cc: Marijn Suijten, linux-pm, devicetree, linux-kernel, linux-arm-msm,
	Konrad Dybcio

From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Add a SoC-specific compatible for SC8180X. It has 2 domains and uses
the older, non-EPSS hardware.

Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
---
 Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml b/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
index 729f68c1f81d32e54e170879c4f7dd91c0d7c80a..e937eb7355e7fb74c0aeafb4cb9f5ed80d96e6c3 100644
--- a/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
+++ b/Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml
@@ -23,6 +23,7 @@ properties:
           - enum:
               - qcom,qcm2290-cpufreq-hw
               - qcom,sc7180-cpufreq-hw
+              - qcom,sc8180x-cpufreq-hw
               - qcom,sdm670-cpufreq-hw
               - qcom,sdm845-cpufreq-hw
               - qcom,sm6115-cpufreq-hw
@@ -135,6 +136,7 @@ allOf:
               - qcom,qdu1000-cpufreq-epss
               - qcom,sa8255p-cpufreq-epss
               - qcom,sc7180-cpufreq-hw
+              - qcom,sc8180x-cpufreq-hw
               - qcom,sc8280xp-cpufreq-epss
               - qcom,sdm670-cpufreq-hw
               - qcom,sdm845-cpufreq-hw

-- 
2.47.0


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

end of thread, other threads:[~2024-11-16 11:31 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-25 17:48 [PATCH 0/2] arm64: dts: qcom: sc8180x: Fix CPUfreq compatible Bjorn Andersson
2024-05-25 17:48 ` [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add sc8180x compatible Bjorn Andersson
2024-05-26 18:23   ` Krzysztof Kozlowski
2024-05-26 20:35     ` Bjorn Andersson
2024-05-25 17:48 ` [PATCH 2/2] arm64: dts: qcom: sc8180x: Correct cpufreq compatible Bjorn Andersson
2024-05-25 20:11   ` Dmitry Baryshkov
  -- strict thread matches above, loose matches on Subject: below --
2024-11-16 11:31 [PATCH 0/2] Fix SC8180X cpufreq dt-bindings error Konrad Dybcio
2024-11-16 11:31 ` [PATCH 1/2] dt-bindings: cpufreq: cpufreq-qcom-hw: Add SC8180X compatible Konrad Dybcio

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox