* [PATCH 01/10] ARM: dts: imx35: rename emi to emi-bus to fix CHECK_DTBS warning
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-02 19:43 ` [PATCH 02/10] ARM: dts: imx35: rename i2c clock-names to ipg Frank Li
` (9 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
Rename emi to emi-bus to fix below CHECK_DTBS warning:
arch/arm/boot/dts/nxp/imx/imx31-bug.dtb: emi@b8000000 (simple-bus): $nodename:0: 'emi@b8000000' does not match '^([a-z][a-z0-9\\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
from schema $id: http://devicetree.org/schemas/simple-bus.yaml
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx31.dtsi | 2 +-
arch/arm/boot/dts/nxp/imx/imx35.dtsi | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx31.dtsi b/arch/arm/boot/dts/nxp/imx/imx31.dtsi
index 8541a666747a94f4bc64ca4b48c709ed64375a9d..c58f855ea85143e28ff33b14951c470e14c56a8e 100644
--- a/arch/arm/boot/dts/nxp/imx/imx31.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx31.dtsi
@@ -333,7 +333,7 @@ pwm: pwm@53fe0000 {
};
};
- emi@b8000000 { /* External Memory Interface */
+ emi-bus@b8000000 { /* External Memory Interface */
compatible = "simple-bus";
reg = <0xb8000000 0x5000>;
ranges;
diff --git a/arch/arm/boot/dts/nxp/imx/imx35.dtsi b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
index 111d7c0331f504723ef160235eca1381eab0524f..1c010a83d5df138562cd086c7b10d34431378add 100644
--- a/arch/arm/boot/dts/nxp/imx/imx35.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
@@ -356,7 +356,7 @@ usbmisc: usbmisc@53ff4600 {
};
};
- emi@80000000 { /* External Memory Interface */
+ emi-bus@80000000 { /* External Memory Interface */
compatible = "fsl,emi", "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 02/10] ARM: dts: imx35: rename i2c clock-names to ipg
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
2026-02-02 19:43 ` [PATCH 01/10] ARM: dts: imx35: rename emi to emi-bus to fix CHECK_DTBS warning Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-02 19:43 ` [PATCH 03/10] ARM: dts: imx35: remove simple-bus 'usbphy' Frank Li
` (8 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
Rename the i2c clock-names from "ipg_per" to "ipg" to match the binding
documentation.
Fix the following CHECK_DTBS warning:
i2c@43f80000 (fsl,imx35-i2c): clock-names:0: 'ipg' was expected
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx35.dtsi | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx35.dtsi b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
index 1c010a83d5df138562cd086c7b10d34431378add..aed7fe2fd6b93fe70a0abc7213fe0b874afe579a 100644
--- a/arch/arm/boot/dts/nxp/imx/imx35.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
@@ -79,7 +79,7 @@ i2c1: i2c@43f80000 {
compatible = "fsl,imx35-i2c", "fsl,imx1-i2c";
reg = <0x43f80000 0x4000>;
clocks = <&clks 51>;
- clock-names = "ipg_per";
+ clock-names = "ipg";
interrupts = <10>;
status = "disabled";
};
@@ -90,7 +90,7 @@ i2c3: i2c@43f84000 {
compatible = "fsl,imx35-i2c", "fsl,imx1-i2c";
reg = <0x43f84000 0x4000>;
clocks = <&clks 53>;
- clock-names = "ipg_per";
+ clock-names = "ipg";
interrupts = <3>;
status = "disabled";
};
@@ -119,7 +119,7 @@ i2c2: i2c@43f98000 {
compatible = "fsl,imx35-i2c", "fsl,imx1-i2c";
reg = <0x43f98000 0x4000>;
clocks = <&clks 52>;
- clock-names = "ipg_per";
+ clock-names = "ipg";
interrupts = <4>;
status = "disabled";
};
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 03/10] ARM: dts: imx35: remove simple-bus 'usbphy'
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
2026-02-02 19:43 ` [PATCH 01/10] ARM: dts: imx35: rename emi to emi-bus to fix CHECK_DTBS warning Frank Li
2026-02-02 19:43 ` [PATCH 02/10] ARM: dts: imx35: rename i2c clock-names to ipg Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-02 19:43 ` [PATCH 04/10] ARM: dts: imx51-ts4800: rename fpga@0 to fpga@0,0 Frank Li
` (7 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
Remove simple bus 'usbphy' and move chip nodes to up layers to fix below
CHECK_DTBS warnings.
arch/arm/boot/dts/nxp/imx/imx35-pdk.dtb: usbphy (simple-bus): usb-phy@1:reg:0: [1] is too short
from schema $id: http://devicetree.org/schemas/simple-bus.yaml
Remove property 'reg' because it is never used at driver.
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx35.dtsi | 22 +++++++---------------
1 file changed, 7 insertions(+), 15 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx35.dtsi b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
index aed7fe2fd6b93fe70a0abc7213fe0b874afe579a..ab7b6463998944899a5c49b566e3beca5c039a94 100644
--- a/arch/arm/boot/dts/nxp/imx/imx35.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
@@ -393,21 +393,13 @@ weim: memory-controller@b8002000 {
};
};
- usbphy {
- compatible = "simple-bus";
- #address-cells = <1>;
- #size-cells = <0>;
-
- usbphy0: usb-phy@0 {
- reg = <0>;
- compatible = "usb-nop-xceiv";
- #phy-cells = <0>;
- };
+ usbphy0: usb-phy0 {
+ compatible = "usb-nop-xceiv";
+ #phy-cells = <0>;
+ };
- usbphy1: usb-phy@1 {
- reg = <1>;
- compatible = "usb-nop-xceiv";
- #phy-cells = <0>;
- };
+ usbphy1: usb-phy1 {
+ compatible = "usb-nop-xceiv";
+ #phy-cells = <0>;
};
};
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 04/10] ARM: dts: imx51-ts4800: rename fpga@0 to fpga@0,0
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
` (2 preceding siblings ...)
2026-02-02 19:43 ` [PATCH 03/10] ARM: dts: imx35: remove simple-bus 'usbphy' Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-02 19:43 ` [PATCH 05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1 Frank Li
` (6 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
Change node name fpga@0 to fpga@0,0 to fix below CHECK_DTBS warnings:
memory-controller@83fda000 (fsl,imx51-weim): 'fpga@0' does not match any of the regexes: '^.*@[0-7],[0-9a-f]+$', '^pinctrl-[0-9]+$'
from schema $id: http://devicetree.org/schemas/memory-controllers/fsl/fsl,imx-weim.yam
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts b/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
index 079bd3d14999f96ef141964a10f492cb2920d9b5..5118a68dbbdc320a895decd7efa2ad5db6a0dd99 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
@@ -141,7 +141,7 @@ &weim {
pinctrl-0 = <&pinctrl_weim>;
status = "okay";
- fpga@0 {
+ fpga-bus@0,0 {
compatible = "simple-bus";
fsl,weim-cs-timing = <0x0061008F 0x00000002 0x1c022000
0x00000000 0x1c092480 0x00000000>;
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
` (3 preceding siblings ...)
2026-02-02 19:43 ` [PATCH 04/10] ARM: dts: imx51-ts4800: rename fpga@0 to fpga@0,0 Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-03 13:59 ` Daniel Baluta
2026-02-02 19:43 ` [PATCH 06/10] ARM: dts: imx53: drop fallback compatible "dlg,da9052" Frank Li
` (5 subsequent siblings)
10 siblings, 1 reply; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
Rename at45db321d@1 to flash@1 to fix below CHECK_DTBS warnings:
at45db321d@1 (atmel,at45db321d): $nodename:0: 'at45db321d@1' does not match '^(flash|.*sram|nand)(@.*)?$'
from schema $id: http://devicetree.org/schemas/mtd/atmel,dataflash.yaml
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx51-babbage.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts b/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
index 1b6ec55f906839f3efde267aaaee035e348f494e..b17264e06e6916057c426179e4207c4a28acf1ab 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
@@ -327,7 +327,7 @@ vgen3_reg: vgen3 {
};
};
- flash: at45db321d@1 {
+ flash: flash@1 {
#address-cells = <1>;
#size-cells = <1>;
compatible = "atmel,at45db321d", "atmel,at45", "atmel,dataflash";
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* Re: [PATCH 05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1
2026-02-02 19:43 ` [PATCH 05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1 Frank Li
@ 2026-02-03 13:59 ` Daniel Baluta
0 siblings, 0 replies; 15+ messages in thread
From: Daniel Baluta @ 2026-02-03 13:59 UTC (permalink / raw)
To: Frank Li
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King, devicetree,
imx, linux-arm-kernel, linux-kernel
On Mon, Feb 2, 2026 at 9:45 PM Frank Li <Frank.Li@nxp.com> wrote:
>
> Rename at45db321d@1 to flash@1 to fix below CHECK_DTBS warnings:
> at45db321d@1 (atmel,at45db321d): $nodename:0: 'at45db321d@1' does not match '^(flash|.*sram|nand)(@.*)?$'
> from schema $id: http://devicetree.org/schemas/mtd/atmel,dataflash.yaml
>
The commit message should say *why* the commit is needed and not
focus on fixing the warnings from a tool.
e.g:
ARM: dts: imx51-babbage: Use generic name for flash node
Prefer generic node name 'flash' over the existing specific one to reflect
the function of the device.
This fixes below CHECK_DTBS warning:
at45db321d@1 (atmel,at45db321d): $nodename:0: 'at45db321d@1' does not
match '^(flash|.*sram|nand)(@.*)?$'
from schema $id: http://devicetree.org/schemas/mtd/atmel,dataflash.yaml
<snip>
> - flash: at45db321d@1 {
> + flash: flash@1 {
> #address-cells = <1>;
> #size-cells = <1>;
> compatible = "atmel,at45db321d", "atmel,at45", "atmel,dataflash";
>
Thanks,
Daniel.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 06/10] ARM: dts: imx53: drop fallback compatible "dlg,da9052"
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
` (4 preceding siblings ...)
2026-02-02 19:43 ` [PATCH 05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1 Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-02 19:43 ` [PATCH 07/10] ARM: dts: imx6qdl-sr-som-ti: use fixed-clock instead of clock-frequency Frank Li
` (4 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
The "dlg,da9052" compatible string is not fully compatible with
"dlg,da9053-aa". Drop the fallback compatible to fix the following
CHECK_DTBS warning:
dialog@48 (dlg,da9053-aa): compatible: ['dlg,da9053-aa', 'dlg,da9052'] is too long
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx53-qsb.dts | 2 +-
arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx53-qsb.dts b/arch/arm/boot/dts/nxp/imx/imx53-qsb.dts
index 6831836bd726cb759d47821d104595e00c3bfb46..e7e15d2b82c4455d07e246eeae6830d905ae9bf1 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-qsb.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-qsb.dts
@@ -13,7 +13,7 @@ / {
&i2c1 {
pmic: dialog@48 {
- compatible = "dlg,da9053-aa", "dlg,da9052";
+ compatible = "dlg,da9053-aa";
reg = <0x48>;
interrupt-parent = <&gpio7>;
interrupts = <11 IRQ_TYPE_LEVEL_LOW>; /* low-level active IRQ at GPIO7_11 */
diff --git a/arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi b/arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi
index 6dc70a92d831b42b2c1ec8d8eba3839fb45f64f4..6eed683d92a83ec1e59943ce31bba5cf315f0246 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi
@@ -128,7 +128,7 @@ &i2c1 {
status = "okay";
pmic: dialog@48 {
- compatible = "dlg,da9053-aa", "dlg,da9052";
+ compatible = "dlg,da9053-aa";
reg = <0x48>;
interrupt-parent = <&gpio7>;
interrupts = <11 IRQ_TYPE_LEVEL_LOW>; /* low-level active IRQ at GPIO7_11 */
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 07/10] ARM: dts: imx6qdl-sr-som-ti: use fixed-clock instead of clock-frequency
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
` (5 preceding siblings ...)
2026-02-02 19:43 ` [PATCH 06/10] ARM: dts: imx53: drop fallback compatible "dlg,da9052" Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-02 19:43 ` [PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000 Frank Li
` (3 subsequent siblings)
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
Replace the clock-frequency property with a fixed-clock providing the
32kHz input to the gpio-gate-clock.
Fix the following CHECK_DTBS warning:
ti-wifi-clock (gpio-gate-clock): 'clock-frequency' does not match any of the regexes: '^pinctrl-[0-9]+$'
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi
index cd1e682f11add204f373b54b7ad90909fa6bfb5d..8192344d529462b7ee0a926e8650cdb316457961 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi
@@ -41,6 +41,12 @@
#include <dt-bindings/gpio/gpio.h>
/ {
+ clk32k: clock-32k {
+ compatible = "fixed-clock";
+ clock-frequency = <32768>;
+ #clock-cells = <0>;
+ };
+
nvcc_sd1: regulator-nvcc-sd1 {
compatible = "regulator-fixed";
regulator-always-on;
@@ -58,7 +64,7 @@ clk_ti_wifi: ti-wifi-clock {
*/
compatible = "gpio-gate-clock";
#clock-cells = <0>;
- clock-frequency = <32768>;
+ clocks = <&clk32k>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_microsom_ti_clk>;
enable-gpios = <&gpio5 5 GPIO_ACTIVE_HIGH>;
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
` (6 preceding siblings ...)
2026-02-02 19:43 ` [PATCH 07/10] ARM: dts: imx6qdl-sr-som-ti: use fixed-clock instead of clock-frequency Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-03 1:52 ` Peng Fan
2026-02-02 19:43 ` [PATCH 09/10] ARM: dts: imx53-smd: Add power supply node for fsl,sgtl5000 Frank Li
` (2 subsequent siblings)
10 siblings, 1 reply; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
According to the fsl,imx7ulp-pm.yaml binding, clock-controller@40410000 is
not a clock provider. Remove the #clock-cells property to fix the following
CHECK_DTBS warning:
clock-controller@40410000 (fsl,imx7ulp-smc1): '#clock-cells' does not match any of the regexes: '^pinctrl-[0-9]+$'
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
index 880b9a4f32b0846a773dbf9ad30715c84ac2fda6..6d8853d36d930aabb98fb57f49d6153ca6597561 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
@@ -291,7 +291,6 @@ pcc2: clock-controller@403f0000 {
smc1: clock-controller@40410000 {
compatible = "fsl,imx7ulp-smc1";
reg = <0x40410000 0x1000>;
- #clock-cells = <1>;
clocks = <&scg1 IMX7ULP_CLK_CORE_DIV>,
<&scg1 IMX7ULP_CLK_HSRUN_CORE_DIV>;
clock-names = "divcore", "hsrun_divcore";
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* Re: [PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000
2026-02-02 19:43 ` [PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000 Frank Li
@ 2026-02-03 1:52 ` Peng Fan
2026-02-03 15:37 ` Frank Li
0 siblings, 1 reply; 15+ messages in thread
From: Peng Fan @ 2026-02-03 1:52 UTC (permalink / raw)
To: Frank Li
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King, devicetree,
imx, linux-arm-kernel, linux-kernel
On Mon, Feb 02, 2026 at 02:43:25PM -0500, Frank Li wrote:
>According to the fsl,imx7ulp-pm.yaml binding, clock-controller@40410000 is
>not a clock provider. Remove the #clock-cells property to fix the following
>CHECK_DTBS warning:
> clock-controller@40410000 (fsl,imx7ulp-smc1): '#clock-cells' does not match any of the regexes: '^pinctrl-[0-9]+$'
>
>Signed-off-by: Frank Li <Frank.Li@nxp.com>
>---
> arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
>index 880b9a4f32b0846a773dbf9ad30715c84ac2fda6..6d8853d36d930aabb98fb57f49d6153ca6597561 100644
>--- a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
>+++ b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
>@@ -291,7 +291,6 @@ pcc2: clock-controller@403f0000 {
> smc1: clock-controller@40410000 {
> compatible = "fsl,imx7ulp-smc1";
> reg = <0x40410000 0x1000>;
>- #clock-cells = <1>;
Need to fix binding, this is a clock controller and needs #clock-cells to work
properly.
There is a driver for this compatible:
drivers/clk/imx/clk-imx7ulp.c
Regards,
Peng
> clocks = <&scg1 IMX7ULP_CLK_CORE_DIV>,
> <&scg1 IMX7ULP_CLK_HSRUN_CORE_DIV>;
> clock-names = "divcore", "hsrun_divcore";
>
>--
>2.34.1
>
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000
2026-02-03 1:52 ` Peng Fan
@ 2026-02-03 15:37 ` Frank Li
0 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-03 15:37 UTC (permalink / raw)
To: Peng Fan
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King, devicetree,
imx, linux-arm-kernel, linux-kernel
On Tue, Feb 03, 2026 at 09:52:39AM +0800, Peng Fan wrote:
> On Mon, Feb 02, 2026 at 02:43:25PM -0500, Frank Li wrote:
> >According to the fsl,imx7ulp-pm.yaml binding, clock-controller@40410000 is
> >not a clock provider. Remove the #clock-cells property to fix the following
> >CHECK_DTBS warning:
> > clock-controller@40410000 (fsl,imx7ulp-smc1): '#clock-cells' does not match any of the regexes: '^pinctrl-[0-9]+$'
> >
> >Signed-off-by: Frank Li <Frank.Li@nxp.com>
> >---
> > arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi | 1 -
> > 1 file changed, 1 deletion(-)
> >
> >diff --git a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
> >index 880b9a4f32b0846a773dbf9ad30715c84ac2fda6..6d8853d36d930aabb98fb57f49d6153ca6597561 100644
> >--- a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
> >+++ b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
> >@@ -291,7 +291,6 @@ pcc2: clock-controller@403f0000 {
> > smc1: clock-controller@40410000 {
> > compatible = "fsl,imx7ulp-smc1";
> > reg = <0x40410000 0x1000>;
> >- #clock-cells = <1>;
>
> Need to fix binding, this is a clock controller and needs #clock-cells to work
> properly.
Thanks, strange no one reference label "smc1"
Frank
>
> There is a driver for this compatible:
> drivers/clk/imx/clk-imx7ulp.c
>
> Regards,
> Peng
>
> > clocks = <&scg1 IMX7ULP_CLK_CORE_DIV>,
> > <&scg1 IMX7ULP_CLK_HSRUN_CORE_DIV>;
> > clock-names = "divcore", "hsrun_divcore";
> >
> >--
> >2.34.1
> >
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 09/10] ARM: dts: imx53-smd: Add power supply node for fsl,sgtl5000
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
` (7 preceding siblings ...)
2026-02-02 19:43 ` [PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000 Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-02 19:43 ` [PATCH 10/10] ARM: dts: imx7s-warp: Remove data-lanes and clock-lanes for ov2680 Frank Li
2026-02-24 19:57 ` (subset) [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
Add power supply, #sound-dai-cells and clock nodes for fsl,sgtl5000 to
fix below CHECK_DTB warnings:
arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: sgtl5000@a (fsl,sgtl5000): '#sound-dai-cells' is a required property
from schema $id: http://devicetree.org/schemas/sound/fsl,sgtl5000.yaml#
arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: sgtl5000@a (fsl,sgtl5000): 'clocks' is a required property
from schema $id: http://devicetree.org/schemas/sound/fsl,sgtl5000.yaml#
arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: sgtl5000@a (fsl,sgtl5000): 'VDDA-supply' is a required property
from schema $id: http://devicetree.org/schemas/sound/fsl,sgtl5000.yaml#
arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: sgtl5000@a (fsl,sgtl5000): 'VDDIO-supply' is a required property
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm/boot/dts/nxp/imx/imx53-smd.dts | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
diff --git a/arch/arm/boot/dts/nxp/imx/imx53-smd.dts b/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
index 386371c816f4d1f2566661e5dd7d02c498ed269b..a1e19f9709b2c7c7d02812f2b5c2afd382fb5d86 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
@@ -31,6 +31,20 @@ key-volume-down {
linux,code = <KEY_VOLUMEDOWN>;
};
};
+
+ reg_1v8: regulator-1v8 {
+ compatible = "regulator-fixed";
+ regulator-name = "1v8";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ };
+
+ reg_3v3: regulator-3v3 {
+ compatible = "regulator-fixed";
+ regulator-name = "3v3";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ };
};
&esdhc1 {
@@ -253,6 +267,10 @@ &i2c2 {
codec: sgtl5000@a {
compatible = "fsl,sgtl5000";
reg = <0x0a>;
+ #sound-dai-cells = <0>;
+ clocks = <&clks IMX5_CLK_DUMMY>;
+ VDDA-supply = <®_1v8>;
+ VDDIO-supply = <®_3v3>;
};
magnetometer: mag3110@e {
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 10/10] ARM: dts: imx7s-warp: Remove data-lanes and clock-lanes for ov2680
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
` (8 preceding siblings ...)
2026-02-02 19:43 ` [PATCH 09/10] ARM: dts: imx53-smd: Add power supply node for fsl,sgtl5000 Frank Li
@ 2026-02-02 19:43 ` Frank Li
2026-02-24 19:57 ` (subset) [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-02 19:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Frank Li
The ov2680 only support 1 lane. Needn't additional property to descript it.
Remove it to fix below DTB_CHECK warnings:
camera@36 (ovti,ov2680): port:endpoint: 'clock-lanes', 'data-lanes' do not match any of the regexes: '^pinctrl-[0-9]+$'
from schema $id: http://devicetree.org/schemas/media/i2c/ovti,ov2680.yaml
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Previous binding update discussion [1]
[1] https://lore.kernel.org/imx/aPiA3G9x_RsTL9Mv@valkosipuli.retiisi.eu/
---
arch/arm/boot/dts/nxp/imx/imx7s-warp.dts | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts b/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
index 92b6258059ee0150803d960ea076f6301ce1b209..25f38acc53501b0f0fd39446e0e016ec6fb6433d 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
@@ -199,8 +199,6 @@ ov2680: camera@36 {
port {
ov2680_to_mipi: endpoint {
remote-endpoint = <&mipi_from_sensor>;
- clock-lanes = <0>;
- data-lanes = <1>;
link-frequencies = /bits/ 64 <330000000>;
};
};
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* Re: (subset) [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings
2026-02-02 19:43 [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings Frank Li
` (9 preceding siblings ...)
2026-02-02 19:43 ` [PATCH 10/10] ARM: dts: imx7s-warp: Remove data-lanes and clock-lanes for ov2680 Frank Li
@ 2026-02-24 19:57 ` Frank Li
10 siblings, 0 replies; 15+ messages in thread
From: Frank Li @ 2026-02-24 19:57 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, Russell King, Frank Li
Cc: devicetree, imx, linux-arm-kernel, linux-kernel
On Mon, 02 Feb 2026 14:43:17 -0500, Frank Li wrote:
> Cleanup some CHECK_DTB warings.
>
>
Applied, thanks!
[01/10] ARM: dts: imx35: rename emi to emi-bus to fix CHECK_DTBS warning
[02/10] ARM: dts: imx35: rename i2c clock-names to ipg
[03/10] ARM: dts: imx35: remove simple-bus 'usbphy'
[04/10] ARM: dts: imx51-ts4800: rename fpga@0 to fpga@0,0
[05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1
[06/10] ARM: dts: imx53: drop fallback compatible "dlg,da9052"
[07/10] ARM: dts: imx6qdl-sr-som-ti: use fixed-clock instead of clock-frequency
[09/10] ARM: dts: imx53-smd: Add power supply node for fsl,sgtl5000
[10/10] ARM: dts: imx7s-warp: Remove data-lanes and clock-lanes for ov2680
Best regards,
--
Frank Li <Frank.Li@nxp.com>
^ permalink raw reply [flat|nested] 15+ messages in thread