* [PATCH] ARM: dts: exynos: align OPP table names with DT schema
@ 2023-01-20 6:47 Krzysztof Kozlowski
2023-01-23 10:01 ` Krzysztof Kozlowski
0 siblings, 1 reply; 2+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-20 6:47 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Alim Akhtar, devicetree,
linux-arm-kernel, linux-samsung-soc, linux-kernel
Cc: Krzysztof Kozlowski
DT schema expects names of operating points tables to match certain
pattern:
exynos5422-odroidxu3-lite.dtb: opp-table0: $nodename:0: 'opp-table0' does not match '^opp-table(-[a-z0-9]+)?$'
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/exynos3250.dtsi | 10 +++---
arch/arm/boot/dts/exynos4210.dtsi | 12 +++----
arch/arm/boot/dts/exynos4412.dtsi | 14 ++++----
arch/arm/boot/dts/exynos5250.dtsi | 2 +-
arch/arm/boot/dts/exynos5420.dtsi | 4 +--
arch/arm/boot/dts/exynos5422-odroid-core.dtsi | 32 +++++++++----------
6 files changed, 37 insertions(+), 37 deletions(-)
diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi
index a2d6ee7fff08..9b25449d5311 100644
--- a/arch/arm/boot/dts/exynos3250.dtsi
+++ b/arch/arm/boot/dts/exynos3250.dtsi
@@ -780,7 +780,7 @@ bus_dmc: bus-dmc {
status = "disabled";
};
- bus_dmc_opp_table: opp-table1 {
+ bus_dmc_opp_table: opp-table-1 {
compatible = "operating-points-v2";
opp-50000000 {
@@ -869,7 +869,7 @@ bus_mfc: bus-mfc {
status = "disabled";
};
- bus_leftbus_opp_table: opp-table2 {
+ bus_leftbus_opp_table: opp-table-2 {
compatible = "operating-points-v2";
opp-50000000 {
@@ -894,7 +894,7 @@ opp-200000000 {
};
};
- bus_mcuisp_opp_table: opp-table3 {
+ bus_mcuisp_opp_table: opp-table-3 {
compatible = "operating-points-v2";
opp-50000000 {
@@ -914,7 +914,7 @@ opp-400000000 {
};
};
- bus_isp_opp_table: opp-table4 {
+ bus_isp_opp_table: opp-table-4 {
compatible = "operating-points-v2";
opp-50000000 {
@@ -934,7 +934,7 @@ opp-300000000 {
};
};
- bus_peril_opp_table: opp-table5 {
+ bus_peril_opp_table: opp-table-5 {
compatible = "operating-points-v2";
opp-50000000 {
diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
index 2c25cc37934e..d11cbe03770d 100644
--- a/arch/arm/boot/dts/exynos4210.dtsi
+++ b/arch/arm/boot/dts/exynos4210.dtsi
@@ -287,7 +287,7 @@ bus_mfc: bus-mfc {
status = "disabled";
};
- bus_dmc_opp_table: opp-table1 {
+ bus_dmc_opp_table: opp-table-1 {
compatible = "operating-points-v2";
opp-shared;
@@ -306,7 +306,7 @@ opp-400000000 {
};
};
- bus_acp_opp_table: opp-table2 {
+ bus_acp_opp_table: opp-table-2 {
compatible = "operating-points-v2";
opp-shared;
@@ -321,7 +321,7 @@ opp-200000000 {
};
};
- bus_peri_opp_table: opp-table3 {
+ bus_peri_opp_table: opp-table-3 {
compatible = "operating-points-v2";
opp-shared;
@@ -333,7 +333,7 @@ opp-100000000 {
};
};
- bus_fsys_opp_table: opp-table4 {
+ bus_fsys_opp_table: opp-table-4 {
compatible = "operating-points-v2";
opp-shared;
@@ -345,7 +345,7 @@ opp-134000000 {
};
};
- bus_display_opp_table: opp-table5 {
+ bus_display_opp_table: opp-table-5 {
compatible = "operating-points-v2";
opp-shared;
@@ -360,7 +360,7 @@ opp-160000000 {
};
};
- bus_leftbus_opp_table: opp-table6 {
+ bus_leftbus_opp_table: opp-table-6 {
compatible = "operating-points-v2";
opp-shared;
diff --git a/arch/arm/boot/dts/exynos4412.dtsi b/arch/arm/boot/dts/exynos4412.dtsi
index aa0b61b59970..c02865ff0761 100644
--- a/arch/arm/boot/dts/exynos4412.dtsi
+++ b/arch/arm/boot/dts/exynos4412.dtsi
@@ -93,7 +93,7 @@ cpu3: cpu@a03 {
};
};
- cpu0_opp_table: opp-table0 {
+ cpu0_opp_table: opp-table-0 {
compatible = "operating-points-v2";
opp-shared;
@@ -420,7 +420,7 @@ bus_c2c: bus-c2c {
status = "disabled";
};
- bus_dmc_opp_table: opp-table1 {
+ bus_dmc_opp_table: opp-table-1 {
compatible = "operating-points-v2";
opp-100000000 {
@@ -446,7 +446,7 @@ opp-400000000 {
};
};
- bus_acp_opp_table: opp-table2 {
+ bus_acp_opp_table: opp-table-2 {
compatible = "operating-points-v2";
opp-100000000 {
@@ -515,7 +515,7 @@ bus_mfc: bus-mfc {
status = "disabled";
};
- bus_leftbus_opp_table: opp-table3 {
+ bus_leftbus_opp_table: opp-table-3 {
compatible = "operating-points-v2";
opp-100000000 {
@@ -537,7 +537,7 @@ opp-200000000 {
};
};
- bus_display_opp_table: opp-table4 {
+ bus_display_opp_table: opp-table-4 {
compatible = "operating-points-v2";
opp-160000000 {
@@ -548,7 +548,7 @@ opp-200000000 {
};
};
- bus_fsys_opp_table: opp-table5 {
+ bus_fsys_opp_table: opp-table-5 {
compatible = "operating-points-v2";
opp-100000000 {
@@ -559,7 +559,7 @@ opp-134000000 {
};
};
- bus_peri_opp_table: opp-table6 {
+ bus_peri_opp_table: opp-table-6 {
compatible = "operating-points-v2";
opp-50000000 {
diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
index 4708dcd575a7..f82f82fc803f 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -81,7 +81,7 @@ cpu1: cpu@1 {
};
};
- cpu0_opp_table: opp-table0 {
+ cpu0_opp_table: opp-table-0 {
compatible = "operating-points-v2";
opp-shared;
diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index 9f2523a873d9..12168f3fa732 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -42,7 +42,7 @@ aliases {
* by exynos5420-cpus.dtsi or exynos5422-cpus.dtsi.
*/
- cluster_a15_opp_table: opp-table0 {
+ cluster_a15_opp_table: opp-table-0 {
compatible = "operating-points-v2";
opp-shared;
@@ -108,7 +108,7 @@ opp-700000000 {
};
};
- cluster_a7_opp_table: opp-table1 {
+ cluster_a7_opp_table: opp-table-1 {
compatible = "operating-points-v2";
opp-shared;
diff --git a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
index 35818c4cd852..06103dd37265 100644
--- a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
+++ b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
@@ -35,7 +35,7 @@ oscclk {
};
};
- bus_wcore_opp_table: opp-table2 {
+ bus_wcore_opp_table: opp-table-2 {
compatible = "operating-points-v2";
/* derived from 532MHz MPLL */
@@ -61,7 +61,7 @@ opp04 {
};
};
- bus_noc_opp_table: opp-table3 {
+ bus_noc_opp_table: opp-table-3 {
compatible = "operating-points-v2";
/* derived from 666MHz CPLL */
@@ -79,7 +79,7 @@ opp03 {
};
};
- bus_fsys_apb_opp_table: opp-table4 {
+ bus_fsys_apb_opp_table: opp-table-4 {
compatible = "operating-points-v2";
/* derived from 666MHz CPLL */
@@ -91,7 +91,7 @@ opp01 {
};
};
- bus_fsys2_opp_table: opp-table5 {
+ bus_fsys2_opp_table: opp-table-5 {
compatible = "operating-points-v2";
/* derived from 600MHz DPLL */
@@ -106,7 +106,7 @@ opp02 {
};
};
- bus_mfc_opp_table: opp-table6 {
+ bus_mfc_opp_table: opp-table-6 {
compatible = "operating-points-v2";
/* derived from 666MHz CPLL */
@@ -127,7 +127,7 @@ opp04 {
};
};
- bus_gen_opp_table: opp-table7 {
+ bus_gen_opp_table: opp-table-7 {
compatible = "operating-points-v2";
/* derived from 532MHz MPLL */
@@ -145,7 +145,7 @@ opp03 {
};
};
- bus_peri_opp_table: opp-table8 {
+ bus_peri_opp_table: opp-table-8 {
compatible = "operating-points-v2";
/* derived from 666MHz CPLL */
@@ -154,7 +154,7 @@ opp00 {
};
};
- bus_g2d_opp_table: opp-table9 {
+ bus_g2d_opp_table: opp-table-9 {
compatible = "operating-points-v2";
/* derived from 666MHz CPLL */
@@ -175,7 +175,7 @@ opp04 {
};
};
- bus_g2d_acp_opp_table: opp-table10 {
+ bus_g2d_acp_opp_table: opp-table-10 {
compatible = "operating-points-v2";
/* derived from 532MHz MPLL */
@@ -193,7 +193,7 @@ opp03 {
};
};
- bus_jpeg_opp_table: opp-table11 {
+ bus_jpeg_opp_table: opp-table-11 {
compatible = "operating-points-v2";
/* derived from 600MHz DPLL */
@@ -211,7 +211,7 @@ opp03 {
};
};
- bus_jpeg_apb_opp_table: opp-table12 {
+ bus_jpeg_apb_opp_table: opp-table-12 {
compatible = "operating-points-v2";
/* derived from 666MHz CPLL */
@@ -229,7 +229,7 @@ opp03 {
};
};
- bus_disp1_fimd_opp_table: opp-table13 {
+ bus_disp1_fimd_opp_table: opp-table-13 {
compatible = "operating-points-v2";
/* derived from 600MHz DPLL */
@@ -241,7 +241,7 @@ opp01 {
};
};
- bus_disp1_opp_table: opp-table14 {
+ bus_disp1_opp_table: opp-table-14 {
compatible = "operating-points-v2";
/* derived from 600MHz DPLL */
@@ -256,7 +256,7 @@ opp02 {
};
};
- bus_gscl_opp_table: opp-table15 {
+ bus_gscl_opp_table: opp-table-15 {
compatible = "operating-points-v2";
/* derived from 600MHz DPLL */
@@ -271,7 +271,7 @@ opp02 {
};
};
- bus_mscl_opp_table: opp-table16 {
+ bus_mscl_opp_table: opp-table-16 {
compatible = "operating-points-v2";
/* derived from 666MHz CPLL */
@@ -292,7 +292,7 @@ opp04 {
};
};
- dmc_opp_table: opp-table17 {
+ dmc_opp_table: opp-table-17 {
compatible = "operating-points-v2";
opp00 {
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ARM: dts: exynos: align OPP table names with DT schema
2023-01-20 6:47 [PATCH] ARM: dts: exynos: align OPP table names with DT schema Krzysztof Kozlowski
@ 2023-01-23 10:01 ` Krzysztof Kozlowski
0 siblings, 0 replies; 2+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-23 10:01 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel, Rob Herring, devicetree,
Alim Akhtar, linux-samsung-soc, Krzysztof Kozlowski,
Krzysztof Kozlowski
On Fri, 20 Jan 2023 07:47:24 +0100, Krzysztof Kozlowski wrote:
> DT schema expects names of operating points tables to match certain
> pattern:
>
> exynos5422-odroidxu3-lite.dtb: opp-table0: $nodename:0: 'opp-table0' does not match '^opp-table(-[a-z0-9]+)?$'
>
>
Applied, thanks!
[1/1] ARM: dts: exynos: align OPP table names with DT schema
https://git.kernel.org/krzk/linux/c/e1c7def07c5c1d1cd6d28aefa8f11e31032c90fe
Best regards,
--
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-01-23 10:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-20 6:47 [PATCH] ARM: dts: exynos: align OPP table names with DT schema Krzysztof Kozlowski
2023-01-23 10:01 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).