| Message ID | 20221225115844.55126-1-krzysztof.kozlowski@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | [1/4] ARM: dts: qcom: align OPP table node name with DT schema | expand |
On 25.12.2022 12:58, Krzysztof Kozlowski wrote: > Bindings expect OPP tables to start with "opp-table". > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad > arch/arm/boot/dts/qcom-sdx55.dtsi | 2 +- > arch/arm/boot/dts/qcom-sdx65.dtsi | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/qcom-sdx55.dtsi b/arch/arm/boot/dts/qcom-sdx55.dtsi > index 5408ff715fbf..c64088c12c89 100644 > --- a/arch/arm/boot/dts/qcom-sdx55.dtsi > +++ b/arch/arm/boot/dts/qcom-sdx55.dtsi > @@ -62,7 +62,7 @@ cpu0: cpu@0 { > }; > }; > > - cpu_opp_table: cpu-opp-table { > + cpu_opp_table: opp-table-cpu { > compatible = "operating-points-v2"; > opp-shared; > > diff --git a/arch/arm/boot/dts/qcom-sdx65.dtsi b/arch/arm/boot/dts/qcom-sdx65.dtsi > index d3c661d7650d..aa6439d8763a 100644 > --- a/arch/arm/boot/dts/qcom-sdx65.dtsi > +++ b/arch/arm/boot/dts/qcom-sdx65.dtsi > @@ -61,7 +61,7 @@ cpu0: cpu@0 { > }; > }; > > - cpu_opp_table: cpu-opp-table { > + cpu_opp_table: opp-table-cpu { > compatible = "operating-points-v2"; > opp-shared; >
On 25.12.2022 12:58, Krzysztof Kozlowski wrote: > Neither qcom,sm8250-lpass-wsa-macro bindings nor the driver use > "clock-frequency" property. > > sm8250-hdk.dtb: codec@3240000: Unevaluated properties are not allowed ('clock-frequency' was unexpected) > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad > arch/arm64/boot/dts/qcom/sm8250.dtsi | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi > index 434e2c4cce79..a58f51083715 100644 > --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi > +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi > @@ -2215,7 +2215,6 @@ wsamacro: codec@3240000 { > clock-names = "mclk", "npl", "macro", "dcodec", "va", "fsgen"; > > #clock-cells = <0>; > - clock-frequency = <9600000>; > clock-output-names = "mclk"; > #sound-dai-cells = <1>; >
On 25.12.2022 12:58, Krzysztof Kozlowski wrote: > Neither qcom,sm8250-lpass-tx-macro bindings nor the driver use > "clock-frequency" and address/size cells properties. > > sm8250-mtp.dtb: txmacro@3220000: Unevaluated properties are not allowed ('clock-frequency', '#address-cells', '#size-cells' were unexpected) > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad > arch/arm64/boot/dts/qcom/sm8250.dtsi | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi > index a58f51083715..ddd750591e8c 100644 > --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi > +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi > @@ -2329,10 +2329,7 @@ txmacro: txmacro@3220000 { > clock-names = "mclk", "npl", "macro", "dcodec", "fsgen"; > > #clock-cells = <0>; > - clock-frequency = <9600000>; > clock-output-names = "mclk"; > - #address-cells = <2>; > - #size-cells = <2>; > #sound-dai-cells = <1>; > }; >
diff --git a/arch/arm/boot/dts/qcom-sdx55.dtsi b/arch/arm/boot/dts/qcom-sdx55.dtsi index 5408ff715fbf..c64088c12c89 100644 --- a/arch/arm/boot/dts/qcom-sdx55.dtsi +++ b/arch/arm/boot/dts/qcom-sdx55.dtsi @@ -62,7 +62,7 @@ cpu0: cpu@0 { }; }; - cpu_opp_table: cpu-opp-table { + cpu_opp_table: opp-table-cpu { compatible = "operating-points-v2"; opp-shared; diff --git a/arch/arm/boot/dts/qcom-sdx65.dtsi b/arch/arm/boot/dts/qcom-sdx65.dtsi index d3c661d7650d..aa6439d8763a 100644 --- a/arch/arm/boot/dts/qcom-sdx65.dtsi +++ b/arch/arm/boot/dts/qcom-sdx65.dtsi @@ -61,7 +61,7 @@ cpu0: cpu@0 { }; }; - cpu_opp_table: cpu-opp-table { + cpu_opp_table: opp-table-cpu { compatible = "operating-points-v2"; opp-shared;
Bindings expect OPP tables to start with "opp-table". Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- arch/arm/boot/dts/qcom-sdx55.dtsi | 2 +- arch/arm/boot/dts/qcom-sdx65.dtsi | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)