[PATCH v2 00/13] ARM: dts: imx: cleanup some CHECK_DTBS warnings

[PATCH v2 00/13] ARM: dts: imx: cleanup some CHECK_DTBS warnings

[Frank.Li]

cleanup more CHECK_DTBS warnings.
less than 90 line warns left, from 267 line.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Changes in v2:
- drop patch ARM: dts: imx6qdl-apalis: remove disabled node "panel-lvds"
- use /delete-property/ pinctrl-0; (sashikio)
- set touchscreen-size-x to 16384 (sashikio)
- Link to v1: https://patch.msgid.link/20261016-arm_dts_cleanup2-v1-0-3a0cab9fd784@nxp.com

---
Frank Li (13):
      ARM: dts: imx6q-dms-ba16: remove undocumented property fsl,receive-dpll-mode
      ARM: dts: imx50-amazon-yoshi: remove undocument property min-frequency
      ARM: dts: imx6dl-b155v2: add touchscreen size properties
      ARM: dts: imx6q-gw5400-a: fix SPI NOR flash vendor prefix
      ARM: dts: imx6dl-prtmvt: remove redundant panel compatible fallback
      ARM: dts: imx53-smd: change compatible string to dlg,da9053-aa
      ARM: dts: imx6qdl-savageboard: remove undocument avic,tm097tdh02
      ARM: dts: imx6ul-tx6ul: remove front compatible string karo,imx6ul-tx6ul-sgtl5000
      ARM: dts: imx6qdl-tx6-mb7: remove redundant pixelclk-active for simple panel
      ARM: dts: imx6q-h100: rename port to port@0 for mipi_csi
      ARM: dts: imx6qdl-sabrelite: fix typo vync-active
      ARM: dts: imx6q-novena: use interrupts replace irq-gpio
      ARM: dts: imx6sx-sdb-mqs: remove pinctrl-0 and pinctrl-names

 arch/arm/boot/dts/nxp/imx/imx50-amazon-yoshi.dts   | 1 -
 arch/arm/boot/dts/nxp/imx/imx53-smd.dts            | 2 +-
 arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts        | 2 ++
 arch/arm/boot/dts/nxp/imx/imx6dl-prtmvt.dts        | 2 +-
 arch/arm/boot/dts/nxp/imx/imx6q-dms-ba16.dts       | 1 -
 arch/arm/boot/dts/nxp/imx/imx6q-gw5400-a.dts       | 2 +-
 arch/arm/boot/dts/nxp/imx/imx6q-h100.dts           | 4 +++-
 arch/arm/boot/dts/nxp/imx/imx6q-novena.dts         | 3 ++-
 arch/arm/boot/dts/nxp/imx/imx6qdl-sabrelite.dtsi   | 2 +-
 arch/arm/boot/dts/nxp/imx/imx6qdl-savageboard.dtsi | 2 +-
 arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-mb7.dtsi     | 3 ---
 arch/arm/boot/dts/nxp/imx/imx6sx-sdb-mqs.dts       | 3 ++-
 arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi        | 3 +--
 13 files changed, 15 insertions(+), 15 deletions(-)
---
base-commit: 573f14ce666583abc3a87c855e27cd78252195e7
change-id: 20261016-arm_dts_cleanup2-efd86a2fa52d

Best regards,
--  
Frank Li <Frank.Li@nxp.com>

[PATCH v2 01/13] ARM: dts: imx6q-dms-ba16: remove undocumented property fsl,receive-dpll-mode

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

fsl,receive-dpll-mode is never documented and used in kernel tree. And also
not used by known user, such as uboot.

Remove it to fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6q-dms-ba16.dtb: sata@2200000 (fsl,imx6q-ahci): 'fsl,receive-dpll-mode' does not match any of the regexes: '^pinctrl-[0-9]+$'
        from schema $id: http://devicetree.org/schemas/ata/imx-sata.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6q-dms-ba16.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-dms-ba16.dts b/arch/arm/boot/dts/nxp/imx/imx6q-dms-ba16.dts
index 484a60892229d..766cd50390bc0 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6q-dms-ba16.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6q-dms-ba16.dts
@@ -115,7 +115,6 @@ &sata {
 	fsl,transmit-atten-16ths = <12>;
 	fsl,transmit-boost-mdB = <3330>;
 	fsl,transmit-level-mV = <1133>;
-	fsl,receive-dpll-mode = <1>;
 	status = "okay";
 };
 

-- 
2.43.0

[PATCH v2 02/13] ARM: dts: imx50-amazon-yoshi: remove undocument property min-frequency

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Remove undocumented and unused property min-frequency, which also not used
by known other user, such as uboot.

Fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx50-amazon-yoshi.dtb: mmc@50020000 (fsl,imx50-esdhc): Unevaluated properties are not allowed ('min-frequency' was unexpected)
        from schema $id: http://devicetree.org/schemas/mmc/fsl-imx-esdhc.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx50-amazon-yoshi.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx50-amazon-yoshi.dts b/arch/arm/boot/dts/nxp/imx/imx50-amazon-yoshi.dts
index 417972ad6bb3e..0b6d0ba65f957 100644
--- a/arch/arm/boot/dts/nxp/imx/imx50-amazon-yoshi.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx50-amazon-yoshi.dts
@@ -21,7 +21,6 @@ &esdhc3 {
 	pinctrl-0 = <&pinctrl_sd3>;
 	bus-width = <8>;
 	non-removable;
-	min-frequency = <400000>;
 	max-frequency = <40000000>;
 	disable-wp;
 	status = "okay";

-- 
2.43.0

[PATCH v2 03/13] ARM: dts: imx6dl-b155v2: add touchscreen size properties

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

The eeti,exc3000 schema requires touchscreen-size-x and touchscreen-size-y
properties. Add them for the EXC80H84 touchscreen on the B155v2 board,
matching the 1366x768 resolution of the AUO G156XTN01 panel.

Fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dtb: touchscreen@2a (eeti,exc80h84): 'touchscreen-size-x' is a required property
        from schema $id: http://devicetree.org/schemas/input/touchscreen/eeti,exc3000.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts
index 7edc788bcb8f3..dd7b892cf0722 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts
@@ -25,6 +25,8 @@ touchscreen@2a {
 		pinctrl-0 = <&pinctrl_q7_gpio0>;
 		interrupt-parent = <&gpio5>;
 		interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
+		touchscreen-size-x = <16384>;
+		touchscreen-size-y = <16384>;
 		touchscreen-inverted-x;
 		touchscreen-inverted-y;
 		reset-gpios = <&tca6424a 21 GPIO_ACTIVE_HIGH>;

-- 
2.43.0

[PATCH v2 04/13] ARM: dts: imx6q-gw5400-a: fix SPI NOR flash vendor prefix

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

The W25Q256 is a Winbond part. Replace the incorrect 'sst,w25q256'
compatible string with 'winbond,w25q256' to match the jedec,spi-nor
schema, which accepts '(winbond,)?w25q256' but not 'sst,w25q256'.

Fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6q-gw5400-a.dtb: flash@0 (sst,w25q256): compatible: 'oneOf' conditional failed, one must be fixed:
        ['sst,w25q256', 'jedec,spi-nor'] is too long

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6q-gw5400-a.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-gw5400-a.dts b/arch/arm/boot/dts/nxp/imx/imx6q-gw5400-a.dts
index bf8fde9cb38d9..89c52b2e7c24f 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6q-gw5400-a.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6q-gw5400-a.dts
@@ -128,7 +128,7 @@ &ecspi1 {
 	status = "okay";
 
 	flash: flash@0 {
-		compatible = "sst,w25q256", "jedec,spi-nor";
+		compatible = "winbond,w25q256", "jedec,spi-nor";
 		spi-max-frequency = <30000000>;
 		reg = <0>;
 	};

-- 
2.43.0

[PATCH v2 05/13] ARM: dts: imx6dl-prtmvt: remove redundant panel compatible fallback

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

The panel-simple schema allows only a single compatible string. Drop the
spurious 'lg,lb070wv8' fallback from the kyo,tcg070wvlq panel node.

Fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6dl-prtmvt.dtb: panel (kyo,tcg070wvlq): compatible: ['kyo,tcg070wvlq', 'lg,lb070wv8'] is too long
        from schema $id: http://devicetree.org/schemas/display/panel/panel-simple.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6dl-prtmvt.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-prtmvt.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-prtmvt.dts
index dcd5a4099c602..4896cd3deaa98 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-prtmvt.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-prtmvt.dts
@@ -178,7 +178,7 @@ led-2 {
 	};
 
 	panel {
-		compatible = "kyo,tcg070wvlq", "lg,lb070wv8";
+		compatible = "kyo,tcg070wvlq";
 		backlight = <&backlight>;
 		power-supply = <&reg_3v3>;
 

-- 
2.43.0

[PATCH v2 06/13] ARM: dts: imx53-smd: change compatible string to dlg,da9053-aa

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

All imx53-smd use the same PMIC with imx53-qsb boards. It should be
dlg,da9053-aa. dlg,da9053 is not supported by drivers and wrong fallback
to dlg,da9052.

Fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: dialog@48 (dlg,da9053): compatible: ['dlg,da9053', 'dlg,da9052'] is too long
        from schema $id: http://devicetree.org/schemas/trivial-devices.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx53-smd.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx53-smd.dts b/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
index 29d38484cd386..aed551566683c 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
@@ -323,7 +323,7 @@ ov5642_to_ipu_csi0: endpoint {
 	};
 
 	pmic: dialog@48 {
-		compatible = "dlg,da9053", "dlg,da9052";
+		compatible = "dlg,da9053-aa";
 		reg = <0x48>;
 		interrupt-parent = <&gpio7>;
 		interrupts = <11 IRQ_TYPE_LEVEL_LOW>;

-- 
2.43.0

[PATCH v2 07/13] ARM: dts: imx6qdl-savageboard: remove undocument avic,tm097tdh02

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Remove undocument avic,tm097tdh02 compatible string, which is not supported
by simple-panel driver. Use fallback compatible string hannstar,hsd100pxn1.

Fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6dl-savageboard.dtb: panel (avic,tm097tdh02): compatible: ['avic,tm097tdh02', 'hannstar,hsd100pxn1'] is too long
        from schema $id: http://devicetree.org/schemas/display/panel/panel-simple.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6qdl-savageboard.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-savageboard.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-savageboard.dtsi
index 2daf2b6af8849..8b476b39aaaf3 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-savageboard.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-savageboard.dtsi
@@ -67,7 +67,7 @@ key-power {
 	};
 
 	panel {
-		compatible = "avic,tm097tdh02", "hannstar,hsd100pxn1";
+		compatible = "hannstar,hsd100pxn1";
 		backlight = <&panel_bl>;
 		power-supply = <&reg_3p3v>;
 

-- 
2.43.0

[PATCH v2 08/13] ARM: dts: imx6ul-tx6ul: remove front compatible string karo,imx6ul-tx6ul-sgtl5000

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

simple-audio-card needn't front compatible string
karo,imx6ul-tx6ul-sgtl5000. Remove it to fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul-0010.dtb: /sound: failed to match any schema with compatible: ['karo,imx6ul-tx6ul-sgtl5000', 'simple-audio-card']

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi b/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
index 2f6916c0359c5..c678f8fc06c28 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
@@ -192,8 +192,7 @@ &gpio1 10 GPIO_ACTIVE_HIGH
 	};
 
 	sound {
-		compatible = "karo,imx6ul-tx6ul-sgtl5000",
-			     "simple-audio-card";
+		compatible = "simple-audio-card";
 		simple-audio-card,name = "imx6ul-tx6ul-sgtl5000-audio";
 		simple-audio-card,format = "i2s";
 		simple-audio-card,bitclock-master = <&codec_dai>;

-- 
2.43.0

[PATCH v2 09/13] ARM: dts: imx6qdl-tx6-mb7: remove redundant pixelclk-active for simple panel

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

The information pixelclk-active already set in panel-simple.c according to
compatible string. Remove redundant pixelclk-active to fix below CHECK_DTBS
warnings:
  arch/arm/boot/dts/nxp/imx/imx6q-tx6q-10x0-mb7.dtb: lcd-panel (edt,et057090dhu): Unevaluated properties are not allowed ('pixelclk-active' was unexpected)
        from schema $id: http://devicetree.org/schemas/display/panel/panel-simple.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-mb7.dtsi | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-mb7.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-mb7.dtsi
index 8232f4ea27526..b770d884c6f3e 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-mb7.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-tx6-mb7.dtsi
@@ -17,19 +17,16 @@ backlight1 {
 	lcd-panel {
 		compatible = "edt,et057090dhu";
 		power-supply = <&reg_lcd1_pwr>;
-		pixelclk-active = <0>;
 	};
 
 	lvds0-panel {
 		compatible = "edt,etml1010g0dka";
 		power-supply = <&reg_lcd1_pwr>;
-		pixelclk-active = <0>;
 	};
 
 	lvds1-panel {
 		compatible = "edt,etml1010g0dka";
 		power-supply = <&reg_lcd1_pwr>;
-		pixelclk-active = <0>;
 	};
 };
 

-- 
2.43.0

[PATCH v2 10/13] ARM: dts: imx6q-h100: rename port to port@0 for mipi_csi

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

mipi_csi port@0 is input port. There are not 'port' at binding doc. Rename
it to port@0 and add reg = <0> to fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6q-h100.dtb: mipi@21dc000 (fsl,imx6-mipi-csi2): 'port' does not match any of the regexes: '^pinctrl-[0-9]+$', '^port@[1-4]$'
        from schema $id: http://devicetree.org/schemas/media/fsl,imx6-mipi-csi2.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6q-h100.dts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-h100.dts b/arch/arm/boot/dts/nxp/imx/imx6q-h100.dts
index 4c8ea43815597..a5180c655c9a6 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6q-h100.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6q-h100.dts
@@ -335,7 +335,9 @@ MX6QDL_PAD_KEY_ROW1__SD2_VSELECT	0x1b0b0
 &mipi_csi {
 	status = "okay";
 
-	port {
+	port@0 {
+		reg = <0>;
+
 		mipi_csi2_in: endpoint {
 			remote-endpoint = <&tc358743_out>;
 			data-lanes = <1 2 3 4>;

-- 
2.43.0

[PATCH v2 11/13] ARM: dts: imx6qdl-sabrelite: fix typo vync-active

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Fix typo vync-active, which should be vsync-active. Fix below CHECK_DTBS
warnings:
  arch/arm/boot/dts/nxp/imx/imx6dl-sabrelite.dtb: endpoint: vync-active: b'\x00\x00\x00\x01' is not of type 'object', 'integer', 'array', 'boolean', 'null'
        from schema $id: http://devicetree.org/schemas/dt-core.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6qdl-sabrelite.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-sabrelite.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-sabrelite.dtsi
index 3b7d01065e87b..308db8af73719 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-sabrelite.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-sabrelite.dtsi
@@ -234,7 +234,7 @@ &ipu1_csi0_from_ipu1_csi0_mux {
 	bus-width = <8>;
 	data-shift = <12>; /* Lines 19:12 used */
 	hsync-active = <1>;
-	vync-active = <1>;
+	vsync-active = <1>;
 };
 
 &ipu1_csi0_mux_from_parallel_sensor {

-- 
2.43.0

[PATCH v2 12/13] ARM: dts: imx6q-novena: use interrupts replace irq-gpio

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

irq-gpio is not existing property, which should use interrupts and
interrupt-parent.

Fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6q-novena.dtb: stmpe811@44 (st,stmpe811): 'irq-gpio' does not match any of the regexes: '^pinctrl-[0-9]+$'
	from schema $id: http://devicetree.org/schemas/mfd/st,stmpe.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6q-novena.dts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-novena.dts b/arch/arm/boot/dts/nxp/imx/imx6q-novena.dts
index 9938a63b5110c..fce04448d932e 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6q-novena.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6q-novena.dts
@@ -280,7 +280,8 @@ sbs_battery: bq20z75@b {
 	touch: stmpe811@44 {
 		compatible = "st,stmpe811";
 		reg = <0x44>;
-		irq-gpio = <&gpio5 13 GPIO_ACTIVE_HIGH>;
+		interrupts = <13 IRQ_TYPE_EDGE_FALLING>;
+		interrupt-parent = <&gpio5>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_stmpe_novena>;
 		vio-supply = <&reg_3p3v>;

-- 
2.43.0

[PATCH v2 13/13] ARM: dts: imx6sx-sdb-mqs: remove pinctrl-0 and pinctrl-names

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Use /delete-property/ pinctrl-0 to remove pinctrl property to avoid pin
conflict.

Remove empty property pinctrl-0 to fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/imx/imx6sx-sdb-mqs.dtb: sai@21d4000 (fsl,imx6sx-sai): pinctrl-0: True is not of type 'array'
        from schema $id: http://devicetree.org/schemas/pinctrl/pinctrl-consumer.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6sx-sdb-mqs.dts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6sx-sdb-mqs.dts b/arch/arm/boot/dts/nxp/imx/imx6sx-sdb-mqs.dts
index a4ab2d3e960c4..a943a31c4c9c6 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6sx-sdb-mqs.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6sx-sdb-mqs.dts
@@ -33,7 +33,8 @@ &mqs {
 };
 
 &sai1 {
-	pinctrl-0 = <>;
+	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "okay";
 };
 

-- 
2.43.0

Re: [PATCH v2 03/13] ARM: dts: imx6dl-b155v2: add touchscreen size properties

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] ARM: dts: imx6dl-b155v2: commit message contradicts 16384 touchscreen size
--

commit 9dc060e080fedd6ab87ca8276cfb3bb59a468536
Author: Frank Li <Frank.Li@nxp.com>

ARM: dts: imx6dl-b155v2: add touchscreen size properties

This commit adds touchscreen-size-x and touchscreen-size-y properties to the
touchscreen@2a node in arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts to satisfy
the eeti,exc3000 binding schema.

>     ARM: dts: imx6dl-b155v2: add touchscreen size properties
>     
>     The eeti,exc3000 schema requires touchscreen-size-x and touchscreen-size-y
>     properties. Add them for the EXC80H84 touchscreen on the B155v2 board,
>     matching the 1366x768 resolution of the AUO G156XTN01 panel.
>     
>     Fix below CHECK_DTBS warnings:
>       arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dtb: touchscreen@2a (eeti,exc80h84): 'touchscreen-size-x' is a required property
>             from schema $id: http://devicetree.org/schemas/input/touchscreen/eeti,exc3000.yaml
>     
>     Signed-off-by: Frank Li <Frank.Li@nxp.com>
> 
> diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts b/arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts
> index 7edc788bcb8f3..dd7b892cf0722 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts
> +++ b/arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts
> @@ -25,6 +25,8 @@ touchscreen@2a {
>  		pinctrl-0 = <&pinctrl_q7_gpio0>;
>  		interrupt-parent = <&gpio5>;
>  		interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> +		touchscreen-size-x = <16384>;
> +		touchscreen-size-y = <16384>;
>  		touchscreen-inverted-x;
>  		touchscreen-inverted-y;
>  		reset-gpios = <&tca6424a 21 GPIO_ACTIVE_HIGH>;

[Severity: Medium]
The commit message states that these properties are added "matching the
1366x768 resolution of the AUO G156XTN01 panel", but the touchscreen@2a node in
arch/arm/boot/dts/nxp/imx/imx6dl-b155v2.dts sets both touchscreen-size-x and
touchscreen-size-y to 16384.

In drivers/input/touchscreen/exc3000.c, exc3000_info defines the native
coordinate range for EETI_EXC80H84 as 16K:

drivers/input/touchscreen/exc3000.c:exc3000_info[] = {
	...
	[EETI_EXC80H84] = {
		.name = "EETI EXC80H84 Touch Screen",
		.max_xy = SZ_16K - 1,
	},
	...
};

Should the commit message be updated to reflect that the properties are set to
16384 to match the EXC80H84 controller's 16K coordinate range rather than the
1366x768 panel resolution?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261019-arm_dts_cleanup2-v2-0-dcdcbe6389c1@nxp.com?part=3