[PATCH v2 0/3] SDM670 CPU frequency scaling: dtschema fixes

[PATCH v2 0/3] SDM670 CPU frequency scaling: dtschema fixes

[Richard Acayan]

Changes since v1 (20230815223108.306018-5-mailingradian@gmail.com):
 - add compatible to allOf area (2/3)
 - change subject line (Acked-by tag retained) (1/3)
 - add fixes tag (2/3)

This adds appropriate compatible strings to pass bindings checks.

Patch 1/3 is preserved from a previous series, as it was not applied to
linux-next yet and I wasn't notified that it was applied anywhere else:
https://lore.kernel.org/linux-arm-msm/20230724214209.208699-7-mailingradian@gmail.com/

Richard Acayan (3):
  dt-bindings: interconnect: OSM L3: add SDM670 compatible
  dt-bindings: cpufreq: cpufreq-qcom-hw: add SDM670 compatible
  arm64: dts: qcom: sdm670: add specific cpufreq compatible

 Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml  | 2 ++
 Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
 arch/arm64/boot/dts/qcom/sdm670.dtsi                            | 2 +-
 3 files changed, 4 insertions(+), 1 deletion(-)

-- 
2.41.0

[PATCH v2 1/3] dt-bindings: interconnect: OSM L3: add SDM670 compatible

[Richard Acayan]

Add the compatible for the OSM L3 interconnect used in the Snapdragon
670.

Signed-off-by: Richard Acayan <mailingradian@gmail.com>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
index 9d0a98d77ae9..21dae0b92819 100644
--- a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
+++ b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
@@ -21,6 +21,7 @@ properties:
           - enum:
               - qcom,sc7180-osm-l3
               - qcom,sc8180x-osm-l3
+              - qcom,sdm670-osm-l3
               - qcom,sdm845-osm-l3
               - qcom,sm6350-osm-l3
               - qcom,sm8150-osm-l3
-- 
2.41.0

[PATCH v2 2/3] dt-bindings: cpufreq: cpufreq-qcom-hw: add SDM670 compatible

[Richard Acayan]

The bindings for Qualcomm CPU frequency have a compatible for each SoC.
Add the compatible for SDM670.

Fixes: 0c665213d126 ("arm64: dts: qcom: sdm670: add cpu frequency scaling")
Signed-off-by: Richard Acayan <mailingradian@gmail.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 a6b3bb8fdf33..3ce288c5e591 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,sdm670-cpufreq-hw
               - qcom,sdm845-cpufreq-hw
               - qcom,sm6115-cpufreq-hw
               - qcom,sm6350-cpufreq-hw
@@ -125,6 +126,7 @@ allOf:
               - qcom,qdu1000-cpufreq-epss
               - qcom,sc7180-cpufreq-hw
               - qcom,sc8280xp-cpufreq-epss
+              - qcom,sdm670-cpufreq-hw
               - qcom,sdm845-cpufreq-hw
               - qcom,sm6115-cpufreq-hw
               - qcom,sm6350-cpufreq-hw
-- 
2.41.0

Re: [PATCH v2 2/3] dt-bindings: cpufreq: cpufreq-qcom-hw: add SDM670 compatible

[Krzysztof Kozlowski]

On 17/08/2023 01:04, Richard Acayan wrote:
> The bindings for Qualcomm CPU frequency have a compatible for each SoC.
> Add the compatible for SDM670.
> 
> Fixes: 0c665213d126 ("arm64: dts: qcom: sdm670: add cpu frequency scaling")
> Signed-off-by: Richard Acayan <mailingradian@gmail.com>
> ---


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

Best regards,
Krzysztof

Re: [PATCH v2 2/3] dt-bindings: cpufreq: cpufreq-qcom-hw: add SDM670 compatible

[Viresh Kumar]

On 18-08-23, 11:40, Krzysztof Kozlowski wrote:
> On 17/08/2023 01:04, Richard Acayan wrote:
> > The bindings for Qualcomm CPU frequency have a compatible for each SoC.
> > Add the compatible for SDM670.
> > 
> > Fixes: 0c665213d126 ("arm64: dts: qcom: sdm670: add cpu frequency scaling")
> > Signed-off-by: Richard Acayan <mailingradian@gmail.com>
> > ---
> 
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Applied. Thanks.

-- 
viresh

[PATCH v2 3/3] arm64: dts: qcom: sdm670: add specific cpufreq compatible

[Richard Acayan]

The bindings for the CPU frequency scaling driver require a specific
compatible for the SoC. Add the compatible.

Fixes: 0c665213d126 ("arm64: dts: qcom: sdm670: add cpu frequency scaling")
Signed-off-by: Richard Acayan <mailingradian@gmail.com>
---
 arch/arm64/boot/dts/qcom/sdm670.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sdm670.dtsi b/arch/arm64/boot/dts/qcom/sdm670.dtsi
index 84cd2e39266f..dae426b08e86 100644
--- a/arch/arm64/boot/dts/qcom/sdm670.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm670.dtsi
@@ -1531,7 +1531,7 @@ osm_l3: interconnect@17d41000 {
 		};
 
 		cpufreq_hw: cpufreq@17d43000 {
-			compatible = "qcom,cpufreq-hw";
+			compatible = "qcom,sdm670-cpufreq-hw", "qcom,cpufreq-hw";
 			reg = <0 0x17d43000 0 0x1400>, <0 0x17d45800 0 0x1400>;
 			reg-names = "freq-domain0", "freq-domain1";
 
-- 
2.41.0

Re: [PATCH v2 3/3] arm64: dts: qcom: sdm670: add specific cpufreq compatible

[Konrad Dybcio]

On 17.08.2023 01:04, Richard Acayan wrote:
> The bindings for the CPU frequency scaling driver require a specific
> compatible for the SoC. Add the compatible.
> 
> Fixes: 0c665213d126 ("arm64: dts: qcom: sdm670: add cpu frequency scaling")
> Signed-off-by: Richard Acayan <mailingradian@gmail.com>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad

Re: [PATCH v2 0/3] SDM670 CPU frequency scaling: dtschema fixes

[Richard Acayan]

On Wed, Aug 16, 2023 at 07:04:14PM -0400, Richard Acayan wrote:
> Changes since v1 (20230815223108.306018-5-mailingradian@gmail.com):
>  - add compatible to allOf area (2/3)
>  - change subject line (Acked-by tag retained) (1/3)
>  - add fixes tag (2/3)
>
> This adds appropriate compatible strings to pass bindings checks.
>
> Patch 1/3 is preserved from a previous series, as it was not applied to
> linux-next yet and I wasn't notified that it was applied anywhere else:
> https://lore.kernel.org/linux-arm-msm/20230724214209.208699-7-mailingradian@gmail.com/
>
> Richard Acayan (3):
>   dt-bindings: interconnect: OSM L3: add SDM670 compatible
>   dt-bindings: cpufreq: cpufreq-qcom-hw: add SDM670 compatible
>   arm64: dts: qcom: sdm670: add specific cpufreq compatible

Hi, just a reminder that this series is here, and patches 2-3 are stale.

Would it help to send them separately?

>
>  Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.yaml  | 2 ++
>  Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
>  arch/arm64/boot/dts/qcom/sdm670.dtsi                            | 2 +-
>  3 files changed, 4 insertions(+), 1 deletion(-)
>
> -- 
> 2.41.0
>

Re: (subset) [PATCH v2 0/3] SDM670 CPU frequency scaling: dtschema fixes

[Bjorn Andersson]

On Wed, 16 Aug 2023 19:04:14 -0400, Richard Acayan wrote:
> Changes since v1 (20230815223108.306018-5-mailingradian@gmail.com):
>  - add compatible to allOf area (2/3)
>  - change subject line (Acked-by tag retained) (1/3)
>  - add fixes tag (2/3)
> 
> This adds appropriate compatible strings to pass bindings checks.
> 
> [...]

Applied, thanks!

[3/3] arm64: dts: qcom: sdm670: add specific cpufreq compatible
      commit: 032ff6a3b39addd54427844affaf21e1e80fabc2

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>