* [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema
@ 2024-05-09 10:48 Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 2/5] ARM: dts: imx: correct choice of panel native mode Krzysztof Kozlowski
` (5 more replies)
0 siblings, 6 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-09 10:48 UTC (permalink / raw
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, linux-kernel
Cc: Krzysztof Kozlowski
DT schema expects panel timings node to follow certain pattern,
dtbs_check warnings:
imx6dl-gw54xx.dtb: display-timings: 'hsd100pxn1' does not match any of the regexes: '^timing', 'pinctrl-[0-9]+'
Linux drivers do not care about node name, so this should not have
effect on Linux.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx51-babbage.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx53-tx53-x03x.dts | 14 +++++++-------
arch/arm/boot/dts/nxp/imx/imx53-tx53-x13x.dts | 6 +++---
.../boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts | 2 +-
.../boot/dts/nxp/imx/imx6dl-aristainetos_4.dts | 2 +-
.../boot/dts/nxp/imx/imx6dl-aristainetos_7.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx6qdl-gw52xx.dtsi | 2 +-
arch/arm/boot/dts/nxp/imx/imx6qdl-gw53xx.dtsi | 2 +-
arch/arm/boot/dts/nxp/imx/imx6qdl-gw54xx.dtsi | 2 +-
arch/arm/boot/dts/nxp/imx/imx6qdl-gw560x.dtsi | 2 +-
arch/arm/boot/dts/nxp/imx/imx6qdl-gw5903.dtsi | 2 +-
arch/arm/boot/dts/nxp/imx/imx6qdl-gw5904.dtsi | 2 +-
arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi | 2 +-
arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi | 16 ++++++++--------
arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi | 16 ++++++++--------
arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi | 14 +++++++-------
19 files changed, 47 insertions(+), 47 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts b/arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts
index b61d55ca1467..f72e109342bc 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts
@@ -25,7 +25,7 @@ disp1 {
pinctrl-0 = <&pinctrl_ipu_disp1>;
display-timings {
- lw700 {
+ timing0: timing-lw700 {
native-mode;
clock-frequency = <33000033>;
hactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts b/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
index 16ff543f3fbf..f4a47e8348b2 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
@@ -89,7 +89,7 @@ display2: disp2 {
status = "disabled";
display-timings {
native-mode = <&timing1>;
- timing1: claawvga {
+ timing1: timing-claawvga {
clock-frequency = <27000000>;
hactive = <800>;
vactive = <480>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts b/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
index 2bd0761c7e90..87a34cbdf233 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
@@ -58,7 +58,7 @@ display1: disp1 {
pinctrl-0 = <&pinctrl_lcd>;
display-timings {
- 800x480p60 {
+ timing0: timing-800x480p60 {
native-mode;
clock-frequency = <30066000>;
hactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts b/arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts
index 1353d985969c..f0f92ee7ba95 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts
@@ -17,7 +17,7 @@ display1: disp1 {
pinctrl-0 = <&pinctrl_ipu_disp1>;
display-timings {
- 800x480p60 {
+ timing0: timing-800x480p60 {
native-mode;
clock-frequency = <31500000>;
hactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx53-tx53-x03x.dts b/arch/arm/boot/dts/nxp/imx/imx53-tx53-x03x.dts
index a7f77527269d..a02d77bb5672 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-tx53-x03x.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-tx53-x03x.dts
@@ -67,7 +67,7 @@ display0_in: endpoint {
};
display-timings {
- VGA {
+ timing-vga {
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
@@ -83,7 +83,7 @@ VGA {
pixelclk-active = <0>;
};
- ETV570 {
+ timing-etc570 {
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
@@ -99,7 +99,7 @@ ETV570 {
pixelclk-active = <0>;
};
- ET0350 {
+ timing-et0350 {
clock-frequency = <6413760>;
hactive = <320>;
vactive = <240>;
@@ -115,7 +115,7 @@ ET0350 {
pixelclk-active = <0>;
};
- ET0430 {
+ timing-et0430 {
clock-frequency = <9009000>;
hactive = <480>;
vactive = <272>;
@@ -131,7 +131,7 @@ ET0430 {
pixelclk-active = <1>;
};
- ET0500 {
+ timing-et0500 {
clock-frequency = <33264000>;
hactive = <800>;
vactive = <480>;
@@ -147,7 +147,7 @@ ET0500 {
pixelclk-active = <0>;
};
- ET0700 { /* same as ET0500 */
+ timing-et0700 { /* same as ET0500 */
clock-frequency = <33264000>;
hactive = <800>;
vactive = <480>;
@@ -163,7 +163,7 @@ ET0700 { /* same as ET0500 */
pixelclk-active = <0>;
};
- ETQ570 {
+ timing-etq570 {
clock-frequency = <6596040>;
hactive = <320>;
vactive = <240>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx53-tx53-x13x.dts b/arch/arm/boot/dts/nxp/imx/imx53-tx53-x13x.dts
index 6cdf2082c742..e10c179dbdb3 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-tx53-x13x.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-tx53-x13x.dts
@@ -191,7 +191,7 @@ lvds0: lvds-channel@0 {
display-timings {
native-mode = <&lvds0_timing0>;
- lvds0_timing0: hsd100pxn1 {
+ lvds0_timing0: timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
@@ -207,7 +207,7 @@ lvds0_timing0: hsd100pxn1 {
pixelclk-active = <1>;
};
- lvds0_timing1: nl12880bc20 {
+ lvds0_timing1: timing-nl12880bc20 {
clock-frequency = <71000000>;
hactive = <1280>;
vactive = <800>;
@@ -233,7 +233,7 @@ lvds1: lvds-channel@1 {
display-timings {
native-mode = <&lvds1_timing0>;
- lvds1_timing0: hsd100pxn1 {
+ lvds1_timing0: timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
index dfa6f64d43cc..c75606fc4abd 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
@@ -85,7 +85,7 @@ lcd_panel: display@0 {
power-on-delay = <10>;
display-timings {
- 480x800p57 {
+ timing0: timing-480x800p57 {
native-mode;
clock-frequency = <27000027>;
hactive = <480>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts
index a5ac79346854..0d86927819c2 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts
@@ -36,7 +36,7 @@ display0: disp0 {
status = "okay";
display-timings {
- 480x800p60 {
+ timing0: timing-480x800p60 {
native-mode;
clock-frequency = <30000000>;
hactive = <480>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts
index 5a25bdbbeb68..66271daf97d9 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts
@@ -25,7 +25,7 @@ display0: disp0 {
status = "okay";
display-timings {
- 800x480p60 {
+ timing0: timing-800x480p60 {
native-mode;
clock-frequency = <33246000>;
hactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw52xx.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw52xx.dtsi
index 48ffb3ee01bd..082a2e3a391f 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw52xx.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw52xx.dtsi
@@ -485,7 +485,7 @@ lvds-channel@0 {
display-timings {
native-mode = <&timing0>;
- timing0: hsd100pxn1 {
+ timing0: timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw53xx.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw53xx.dtsi
index 1eae438fbdae..8ec442038ea0 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw53xx.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw53xx.dtsi
@@ -482,7 +482,7 @@ lvds-channel@0 {
display-timings {
native-mode = <&timing0>;
- timing0: hsd100pxn1 {
+ timing0: timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw54xx.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw54xx.dtsi
index c2ec8572c8a5..9df9f79affae 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw54xx.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw54xx.dtsi
@@ -529,7 +529,7 @@ lvds-channel@0 {
display-timings {
native-mode = <&timing0>;
- timing0: hsd100pxn1 {
+ timing0: timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw560x.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw560x.dtsi
index 7cee983da669..7693f92195d5 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw560x.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw560x.dtsi
@@ -584,7 +584,7 @@ lvds-channel@0 {
display-timings {
native-mode = <&timing0>;
- timing0: hsd100pxn1 {
+ timing0: timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw5903.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw5903.dtsi
index fbc704c064b6..9d0836df0fed 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw5903.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw5903.dtsi
@@ -486,7 +486,7 @@ lvds-channel@0 {
display-timings {
native-mode = <&timing0>;
- timing0: g101evn010 {
+ timing0: timing-g101evn010 {
clock-frequency = <68930000>;
hactive = <1280>;
vactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw5904.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw5904.dtsi
index 070506279186..f4cb9e1d34a9 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-gw5904.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-gw5904.dtsi
@@ -551,7 +551,7 @@ lvds-channel@0 {
display-timings {
native-mode = <&timing0>;
- timing0: hsd100pxn1 {
+ timing0: timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi
index 6656e2e762a1..0a3deaf92eea 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi
@@ -786,7 +786,7 @@ lvds-channel@0 {
display-timings {
native-mode = <&timing0>;
- timing0: hsd100pxn1 {
+ timing0: timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi
index 79f2354886b7..7ae495bdc48f 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi
@@ -110,7 +110,7 @@ lcd_out: endpoint {
};
display-timings {
- VGA {
+ timing-vga {
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
@@ -126,7 +126,7 @@ VGA {
pixelclk-active = <0>;
};
- ETV570 {
+ timing-etv570 {
u-boot,panel-name = "edt,et057090dhu";
clock-frequency = <25200000>;
hactive = <640>;
@@ -143,7 +143,7 @@ ETV570 {
pixelclk-active = <0>;
};
- ET0350 {
+ timing-et0350 {
u-boot,panel-name = "edt,et0350g0dh6";
clock-frequency = <6413760>;
hactive = <320>;
@@ -160,7 +160,7 @@ ET0350 {
pixelclk-active = <0>;
};
- ET0430 {
+ timing-et0430 {
u-boot,panel-name = "edt,et0430g0dh6";
clock-frequency = <9009000>;
hactive = <480>;
@@ -177,7 +177,7 @@ ET0430 {
pixelclk-active = <1>;
};
- ET0500 {
+ timing-et0500 {
clock-frequency = <33264000>;
hactive = <800>;
vactive = <480>;
@@ -193,7 +193,7 @@ ET0500 {
pixelclk-active = <0>;
};
- ET0700 { /* same as ET0500 */
+ timing-et0700 { /* same as ET0500 */
u-boot,panel-name = "edt,etm0700g0dh6";
clock-frequency = <33264000>;
hactive = <800>;
@@ -210,7 +210,7 @@ ET0700 { /* same as ET0500 */
pixelclk-active = <0>;
};
- ETQ570 {
+ timing-etq570 {
clock-frequency = <6596040>;
hactive = <320>;
vactive = <240>;
@@ -226,7 +226,7 @@ ETQ570 {
pixelclk-active = <0>;
};
- CoMTFT { /* same as ET0700 but with inverted pixel clock */
+ timing-comtft { /* same as ET0700 but with inverted pixel clock */
u-boot,panel-name = "edt,etm0700g0edh6";
clock-frequency = <33264000>;
hactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi
index 2ca2eb37e14f..dfbdbb87aec4 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi
@@ -127,7 +127,7 @@ lvds0_out: endpoint {
};
display-timings {
- hsd100pxn1 {
+ timing-hsd100pxn1 {
u-boot,panel-name = "hannstar,hsd100pxn1";
clock-frequency = <65000000>;
hactive = <1024>;
@@ -142,7 +142,7 @@ hsd100pxn1 {
pixelclk-active = <1>;
};
- VGA {
+ timing-vga {
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
@@ -158,7 +158,7 @@ VGA {
pixelclk-active = <0>;
};
- nl12880bc20 {
+ timing-nl12880bc20 {
u-boot,panel-name = "nlt,nl12880bc20-spwg-24";
clock-frequency = <71000000>;
hactive = <1280>;
@@ -175,7 +175,7 @@ nl12880bc20 {
pixelclk-active = <1>;
};
- ET0700 {
+ timing-et0700 {
u-boot,panel-name = "edt,etm0700g0dh6";
clock-frequency = <33264000>;
hactive = <800>;
@@ -192,7 +192,7 @@ ET0700 {
pixelclk-active = <0>;
};
- ETV570 {
+ timing-etv570 {
u-boot,panel-name = "edt,et057090dhu";
clock-frequency = <25200000>;
hactive = <640>;
@@ -224,7 +224,7 @@ lvds1_out: endpoint {
};
display-timings {
- hsd100pxn1 {
+ timing-hsd100pxn1 {
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
@@ -238,7 +238,7 @@ hsd100pxn1 {
pixelclk-active = <1>;
};
- VGA {
+ timing-vga {
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
@@ -254,7 +254,7 @@ VGA {
pixelclk-active = <0>;
};
- nl12880bc20 {
+ timing-nl12880bc20 {
clock-frequency = <71000000>;
hactive = <1280>;
vactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi b/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
index 1db146ac1c17..864173e30709 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
@@ -405,7 +405,7 @@ display: disp0 {
status = "okay";
display-timings {
- VGA {
+ timing-vga {
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
@@ -421,7 +421,7 @@ VGA {
pixelclk-active = <1>;
};
- ETV570 {
+ timing-etv570 {
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
@@ -437,7 +437,7 @@ ETV570 {
pixelclk-active = <1>;
};
- ET0350 {
+ timing-et0350 {
clock-frequency = <6413760>;
hactive = <320>;
vactive = <240>;
@@ -453,7 +453,7 @@ ET0350 {
pixelclk-active = <1>;
};
- ET0430 {
+ timing-et0430 {
clock-frequency = <9009000>;
hactive = <480>;
vactive = <272>;
@@ -469,7 +469,7 @@ ET0430 {
pixelclk-active = <0>;
};
- ET0500 {
+ timing-et0500 {
clock-frequency = <33264000>;
hactive = <800>;
vactive = <480>;
@@ -485,7 +485,7 @@ ET0500 {
pixelclk-active = <1>;
};
- ET0700 { /* same as ET0500 */
+ timing-et0700 { /* same as ET0500 */
clock-frequency = <33264000>;
hactive = <800>;
vactive = <480>;
@@ -501,7 +501,7 @@ ET0700 { /* same as ET0500 */
pixelclk-active = <1>;
};
- ETQ570 {
+ timing-etq570 {
clock-frequency = <6596040>;
hactive = <320>;
vactive = <240>;
--
2.43.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/5] ARM: dts: imx: correct choice of panel native mode
2024-05-09 10:48 [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
@ 2024-05-09 10:48 ` Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 3/5] ARM: dts: imx6dl-aristainetos2_4: drop redundant 'power-on-delay' property Krzysztof Kozlowski
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-09 10:48 UTC (permalink / raw
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, linux-kernel
Cc: Krzysztof Kozlowski
Bindings and Linux driver expect native-mode to be a phandle to one of
the timings node, not a boolean property. Correct the DTS to fix
dtbs_check warnings like:
imx53-m53evk.dtb: display-timings: timing-800x480p60: 'native-mode' does not match any of the regexes: 'pinctrl-[0-9]+'
This should not have actual effect for Linux kernel (no real bug
affecting choice of native-mode), because the first timing node is
chosen in absence of proper native-mode property.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts | 2 +-
6 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts b/arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts
index f72e109342bc..de6b7607510a 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-apf51dev.dts
@@ -25,8 +25,8 @@ disp1 {
pinctrl-0 = <&pinctrl_ipu_disp1>;
display-timings {
+ native-mode = <&timing0>;
timing0: timing-lw700 {
- native-mode;
clock-frequency = <33000033>;
hactive = <800>;
vactive = <480>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts b/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
index 87a34cbdf233..079bd3d14999 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
@@ -58,8 +58,8 @@ display1: disp1 {
pinctrl-0 = <&pinctrl_lcd>;
display-timings {
+ native-mode = <&timing0>;
timing0: timing-800x480p60 {
- native-mode;
clock-frequency = <30066000>;
hactive = <800>;
vactive = <480>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts b/arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts
index f0f92ee7ba95..ba0c62994f75 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-m53evk.dts
@@ -17,8 +17,8 @@ display1: disp1 {
pinctrl-0 = <&pinctrl_ipu_disp1>;
display-timings {
+ native-mode = <&timing0>;
timing0: timing-800x480p60 {
- native-mode;
clock-frequency = <31500000>;
hactive = <800>;
vactive = <480>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
index c75606fc4abd..ec806b8d503a 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
@@ -85,8 +85,8 @@ lcd_panel: display@0 {
power-on-delay = <10>;
display-timings {
+ native-mode = <&timing0>;
timing0: timing-480x800p57 {
- native-mode;
clock-frequency = <27000027>;
hactive = <480>;
vactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts
index 0d86927819c2..9ec038f1d0ff 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_4.dts
@@ -36,8 +36,8 @@ display0: disp0 {
status = "okay";
display-timings {
+ native-mode = <&timing0>;
timing0: timing-480x800p60 {
- native-mode;
clock-frequency = <30000000>;
hactive = <480>;
vactive = <800>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts
index 66271daf97d9..b3129832f471 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos_7.dts
@@ -25,8 +25,8 @@ display0: disp0 {
status = "okay";
display-timings {
+ native-mode = <&timing0>;
timing0: timing-800x480p60 {
- native-mode;
clock-frequency = <33246000>;
hactive = <800>;
vactive = <480>;
--
2.43.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/5] ARM: dts: imx6dl-aristainetos2_4: drop redundant 'power-on-delay' property
2024-05-09 10:48 [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 2/5] ARM: dts: imx: correct choice of panel native mode Krzysztof Kozlowski
@ 2024-05-09 10:48 ` Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 4/5] ARM: dts: imx: drop redundant 'u-boot,panel-name' property Krzysztof Kozlowski
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-09 10:48 UTC (permalink / raw
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, linux-kernel
Cc: Krzysztof Kozlowski
LG4573 panel bindings do not allow 'power-on-delay' property. Linux
driver does not use it, either. Reported by dtbs_check:
imx6dl-aristainetos2_4.dtb: display@0: Unevaluated properties are not allowed ('power-on-delay' was unexpected)
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
index ec806b8d503a..c9b2ea2b24b2 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-aristainetos2_4.dts
@@ -82,7 +82,6 @@ lcd_panel: display@0 {
compatible = "lg,lg4573";
spi-max-frequency = <10000000>;
reg = <0>;
- power-on-delay = <10>;
display-timings {
native-mode = <&timing0>;
--
2.43.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 4/5] ARM: dts: imx: drop redundant 'u-boot,panel-name' property
2024-05-09 10:48 [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 2/5] ARM: dts: imx: correct choice of panel native mode Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 3/5] ARM: dts: imx6dl-aristainetos2_4: drop redundant 'power-on-delay' property Krzysztof Kozlowski
@ 2024-05-09 10:48 ` Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 5/5] ARM: dts: imx28-tx28: drop redundant 'panel-name' property Krzysztof Kozlowski
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-09 10:48 UTC (permalink / raw
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, linux-kernel
Cc: Krzysztof Kozlowski
Panel timing bindings do not allow 'u-boot,panel-name' and there seems
to be no users of it: neither Linux kernel drivers, nor U-boot as of
v2024.07-rc2. Reported by dtbs_check:
imx6qp-tx6qp-8037.dtb: display-timings: timing-et0700: 'u-boot,panel-name' does not match any of the regexes: 'pinctrl-[0-9]+'
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi | 5 -----
arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi | 4 ----
2 files changed, 9 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi
index 7ae495bdc48f..ded241a39906 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lcd.dtsi
@@ -127,7 +127,6 @@ timing-vga {
};
timing-etv570 {
- u-boot,panel-name = "edt,et057090dhu";
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
@@ -144,7 +143,6 @@ timing-etv570 {
};
timing-et0350 {
- u-boot,panel-name = "edt,et0350g0dh6";
clock-frequency = <6413760>;
hactive = <320>;
vactive = <240>;
@@ -161,7 +159,6 @@ timing-et0350 {
};
timing-et0430 {
- u-boot,panel-name = "edt,et0430g0dh6";
clock-frequency = <9009000>;
hactive = <480>;
vactive = <272>;
@@ -194,7 +191,6 @@ timing-et0500 {
};
timing-et0700 { /* same as ET0500 */
- u-boot,panel-name = "edt,etm0700g0dh6";
clock-frequency = <33264000>;
hactive = <800>;
vactive = <480>;
@@ -227,7 +223,6 @@ timing-etq570 {
};
timing-comtft { /* same as ET0700 but with inverted pixel clock */
- u-boot,panel-name = "edt,etm0700g0edh6";
clock-frequency = <33264000>;
hactive = <800>;
vactive = <480>;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi
index dfbdbb87aec4..4eb53d5677a6 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-lvds.dtsi
@@ -128,7 +128,6 @@ lvds0_out: endpoint {
display-timings {
timing-hsd100pxn1 {
- u-boot,panel-name = "hannstar,hsd100pxn1";
clock-frequency = <65000000>;
hactive = <1024>;
vactive = <768>;
@@ -159,7 +158,6 @@ timing-vga {
};
timing-nl12880bc20 {
- u-boot,panel-name = "nlt,nl12880bc20-spwg-24";
clock-frequency = <71000000>;
hactive = <1280>;
vactive = <800>;
@@ -176,7 +174,6 @@ timing-nl12880bc20 {
};
timing-et0700 {
- u-boot,panel-name = "edt,etm0700g0dh6";
clock-frequency = <33264000>;
hactive = <800>;
vactive = <480>;
@@ -193,7 +190,6 @@ timing-et0700 {
};
timing-etv570 {
- u-boot,panel-name = "edt,et057090dhu";
clock-frequency = <25200000>;
hactive = <640>;
vactive = <480>;
--
2.43.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 5/5] ARM: dts: imx28-tx28: drop redundant 'panel-name' property
2024-05-09 10:48 [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
` (2 preceding siblings ...)
2024-05-09 10:48 ` [PATCH 4/5] ARM: dts: imx: drop redundant 'u-boot,panel-name' property Krzysztof Kozlowski
@ 2024-05-09 10:48 ` Krzysztof Kozlowski
2024-06-12 11:34 ` [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
2024-06-15 7:45 ` Shawn Guo
5 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-09 10:48 UTC (permalink / raw
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, linux-kernel
Cc: Krzysztof Kozlowski
Panel timing bindings do not allow 'panel-name' and there seems to be no
users of it: neither Linux kernel drivers, nor U-boot as of
v2024.07-rc2.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts | 6 ------
1 file changed, 6 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts b/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
index 5485fe118dc4..d38183edf0fd 100644
--- a/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
+++ b/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
@@ -323,7 +323,6 @@ display0: display0 {
display-timings {
native-mode = <&timing5>;
timing0: timing0 {
- panel-name = "VGA";
clock-frequency = <25175000>;
hactive = <640>;
vactive = <480>;
@@ -340,7 +339,6 @@ timing0: timing0 {
};
timing1: timing1 {
- panel-name = "ETV570";
clock-frequency = <25175000>;
hactive = <640>;
vactive = <480>;
@@ -357,7 +355,6 @@ timing1: timing1 {
};
timing2: timing2 {
- panel-name = "ET0350";
clock-frequency = <6500000>;
hactive = <320>;
vactive = <240>;
@@ -374,7 +371,6 @@ timing2: timing2 {
};
timing3: timing3 {
- panel-name = "ET0430";
clock-frequency = <9000000>;
hactive = <480>;
vactive = <272>;
@@ -391,7 +387,6 @@ timing3: timing3 {
};
timing4: timing4 {
- panel-name = "ET0500", "ET0700";
clock-frequency = <33260000>;
hactive = <800>;
vactive = <480>;
@@ -408,7 +403,6 @@ timing4: timing4 {
};
timing5: timing5 {
- panel-name = "ETQ570";
clock-frequency = <6400000>;
hactive = <320>;
vactive = <240>;
--
2.43.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema
2024-05-09 10:48 [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
` (3 preceding siblings ...)
2024-05-09 10:48 ` [PATCH 5/5] ARM: dts: imx28-tx28: drop redundant 'panel-name' property Krzysztof Kozlowski
@ 2024-06-12 11:34 ` Krzysztof Kozlowski
2024-06-15 7:45 ` Shawn Guo
5 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2024-06-12 11:34 UTC (permalink / raw
To: Shawn Guo
Cc: Rob Herring, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
devicetree, imx, linux-arm-kernel, linux-kernel, Conor Dooley,
Krzysztof Kozlowski
On 09/05/2024 12:48, Krzysztof Kozlowski wrote:
> DT schema expects panel timings node to follow certain pattern,
> dtbs_check warnings:
>
> imx6dl-gw54xx.dtb: display-timings: 'hsd100pxn1' does not match any of the regexes: '^timing', 'pinctrl-[0-9]+'
>
> Linux drivers do not care about node name, so this should not have
> effect on Linux.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Hi Shawn,
Any comments here? Can you pick the series?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema
2024-05-09 10:48 [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
` (4 preceding siblings ...)
2024-06-12 11:34 ` [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
@ 2024-06-15 7:45 ` Shawn Guo
5 siblings, 0 replies; 7+ messages in thread
From: Shawn Guo @ 2024-06-15 7:45 UTC (permalink / raw
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, linux-kernel
On Thu, May 09, 2024 at 12:48:34PM +0200, Krzysztof Kozlowski wrote:
> DT schema expects panel timings node to follow certain pattern,
> dtbs_check warnings:
>
> imx6dl-gw54xx.dtb: display-timings: 'hsd100pxn1' does not match any of the regexes: '^timing', 'pinctrl-[0-9]+'
>
> Linux drivers do not care about node name, so this should not have
> effect on Linux.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Applied all, thanks!
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-06-15 7:46 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-09 10:48 [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 2/5] ARM: dts: imx: correct choice of panel native mode Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 3/5] ARM: dts: imx6dl-aristainetos2_4: drop redundant 'power-on-delay' property Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 4/5] ARM: dts: imx: drop redundant 'u-boot,panel-name' property Krzysztof Kozlowski
2024-05-09 10:48 ` [PATCH 5/5] ARM: dts: imx28-tx28: drop redundant 'panel-name' property Krzysztof Kozlowski
2024-06-12 11:34 ` [PATCH 1/5] ARM: dts: imx: align panel timings node name with dtschema Krzysztof Kozlowski
2024-06-15 7:45 ` Shawn Guo
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 read-only IMAP folder(s) and NNTP newsgroup(s).