* [PATCH v2 01/12] arm64: dts: imx8-ss-img: remove undocument slot for jpeg
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 02/12] arm64: dts: fsl-ls1043a: move "fsl,ls1043a-qdma" ahead "fsl,ls1021a-qdma" Frank Li
` (11 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Remove undocument 'slot' property. It is default 0 at driver. We can remove
it safely.
Fix below warning:
arch/arm64/boot/dts/freescale/imx8dx-colibri-iris.dtb: jpegdec@58400000:
'slot' does not match any of the regexes: 'pinctrl-[0-9]+'
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Previous discuss at thread
- https://lore.kernel.org/imx/e3d64b07-98fc-4d43-a1ef-561cbac58552@nxp.com/
- https://lore.kernel.org/all/bbb1875b-7980-46aa-80b4-dbaf2a2d5755@linaro.org/
Slot should be managed by driver self. So remove "slot" in dts file.
---
arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi b/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
index 77d2928997b4b..d39242c1b9f79 100644
--- a/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
@@ -26,7 +26,6 @@ jpegdec: jpegdec@58400000 {
assigned-clock-rates = <200000000>, <200000000>;
power-domains = <&pd IMX_SC_R_MJPEG_DEC_MP>,
<&pd IMX_SC_R_MJPEG_DEC_S0>;
- slot = <0>;
};
jpegenc: jpegenc@58450000 {
@@ -39,7 +38,6 @@ jpegenc: jpegenc@58450000 {
assigned-clock-rates = <200000000>, <200000000>;
power-domains = <&pd IMX_SC_R_MJPEG_ENC_MP>,
<&pd IMX_SC_R_MJPEG_ENC_S0>;
- slot = <0>;
};
img_jpeg_dec_lpcg: clock-controller@585d0000 {
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 02/12] arm64: dts: fsl-ls1043a: move "fsl,ls1043a-qdma" ahead "fsl,ls1021a-qdma"
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
2024-08-19 17:01 ` [PATCH v2 01/12] arm64: dts: imx8-ss-img: remove undocument slot for jpeg Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 03/12] arm64: dts: imx: rename gpio hog as <gpio name>-hog Frank Li
` (10 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Move fsl,ls1043a-qdma ahead fallback string fsl,ls1021a-qdma.
Fix warning:
dma-controller@8380000: compatible: 'oneOf' conditional failed, one must be fixed:
['fsl,ls1021a-qdma', 'fsl,ls1043a-qdma'] is too long
Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
index fc38a0e543ab8..c0e3e8fa1e794 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
@@ -955,7 +955,7 @@ pcie3: pcie@3600000 {
};
qdma: dma-controller@8380000 {
- compatible = "fsl,ls1021a-qdma", "fsl,ls1043a-qdma";
+ compatible = "fsl,ls1043a-qdma", "fsl,ls1021a-qdma";
reg = <0x0 0x8380000 0x0 0x1000>, /* Controller regs */
<0x0 0x8390000 0x0 0x10000>, /* Status regs */
<0x0 0x83a0000 0x0 0x40000>; /* Block regs */
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 03/12] arm64: dts: imx: rename gpio hog as <gpio name>-hog
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
2024-08-19 17:01 ` [PATCH v2 01/12] arm64: dts: imx8-ss-img: remove undocument slot for jpeg Frank Li
2024-08-19 17:01 ` [PATCH v2 02/12] arm64: dts: fsl-ls1043a: move "fsl,ls1043a-qdma" ahead "fsl,ls1021a-qdma" Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 04/12] arm64: dts: fsl-ls1012a-frdm: move clock-sc16is7xx under root node Frank Li
` (9 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Rename admin_led_lower to admin-led-lower-hog.
Rename gpio rs485_en as rs485-en-hog.
Rename gpio uart4_rs485_en to uart4-rs485-en-hog.
Fix below warning:
arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: gpio@76:
'admin_led_lower' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtb:
gpio@30230000: 'rs485_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtb:
gpio@30220000: 'uart4_rs485_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts | 2 +-
arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso | 2 +-
arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso | 2 +-
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso | 2 +-
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso | 4 ++--
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso | 4 ++--
arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso | 2 +-
arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso | 4 ++--
arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso | 4 ++--
9 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
index d4867d6cf47cd..57f2082143c8a 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
@@ -220,7 +220,7 @@ sfpgpio: gpio@76 {
#gpio-cells = <2>;
gpio-controller;
- admin_led_lower {
+ admin-led-lower-hog {
gpio-hog;
gpios = <13 GPIO_ACTIVE_HIGH>;
output-low;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
index f1f38b739ef76..78f4e8d5814da 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
@@ -18,7 +18,7 @@ &gpio3 {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_gpio3_hog>;
- uart4_rs485_en {
+ uart4-rs485-en-hog {
gpio-hog;
gpios = <20 GPIO_ACTIVE_HIGH>;
output-low;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
index 1d8951e1a47e8..66288948bdd39 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
@@ -19,7 +19,7 @@ &gpio3 {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_gpio3_hog>;
- uart4_rs485_en {
+ uart4-rs485-en-hog {
gpio-hog;
gpios = <20 GPIO_ACTIVE_HIGH>;
output-high;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
index 47d3c0c49e8a0..bb2056746f8c9 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
@@ -16,7 +16,7 @@
/plugin/;
&gpio4 {
- rs485_en {
+ rs485-en-hog {
gpio-hog;
gpios = <0 GPIO_ACTIVE_HIGH>;
output-low;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
index 7fcd8c851159a..45ac8bdce8699 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
@@ -19,14 +19,14 @@
/plugin/;
&gpio4 {
- rs485_en {
+ rs485-en-hog {
gpio-hog;
gpios = <0 GPIO_ACTIVE_HIGH>;
output-high;
line-name = "rs485_en";
};
- rs485_hd {
+ rs485-hd-hog {
gpio-hog;
gpios = <2 GPIO_ACTIVE_HIGH>;
output-low;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
index b19e38fc27bae..30aa620d7004d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
@@ -19,14 +19,14 @@
/plugin/;
&gpio4 {
- rs485_en {
+ rs485-en-hog {
gpio-hog;
gpios = <0 GPIO_ACTIVE_HIGH>;
output-high;
line-name = "rs485_en";
};
- rs485_hd {
+ rs485-hd-hog {
gpio-hog;
gpios = <2 GPIO_ACTIVE_HIGH>;
output-high;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
index c0f6aece844a0..b1a9f35e1dfa7 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
@@ -16,7 +16,7 @@
/plugin/;
&gpio4 {
- rs485_en {
+ rs485-en-hog {
gpio-hog;
gpios = <0 GPIO_ACTIVE_HIGH>;
output-low;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
index e909cbd8db4bc..44ebc0a58c51a 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
@@ -19,14 +19,14 @@
/plugin/;
&gpio4 {
- rs485_en {
+ rs485-en-hog {
gpio-hog;
gpios = <0 GPIO_ACTIVE_HIGH>;
output-high;
line-name = "rs485_en";
};
- rs485_hd {
+ rs485-hd-hog {
gpio-hog;
gpios = <2 GPIO_ACTIVE_HIGH>;
output-low;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
index 1508cb5175b3e..2f8a7ac408739 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
@@ -19,14 +19,14 @@
/plugin/;
&gpio4 {
- rs485_en {
+ rs485-en-hog {
gpio-hog;
gpios = <0 GPIO_ACTIVE_HIGH>;
output-high;
line-name = "rs485_en";
};
- rs485_hd {
+ rs485-hd-hog {
gpio-hog;
gpios = <2 GPIO_ACTIVE_HIGH>;
output-high;
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 04/12] arm64: dts: fsl-ls1012a-frdm: move clock-sc16is7xx under root node
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (2 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 03/12] arm64: dts: imx: rename gpio hog as <gpio name>-hog Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 05/12] arm64: dts: layerscape: rename mdio-mux-emi to mdio-mux@<addr> Frank Li
` (8 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Move fixed clock "clock-sc16is7x" from dspi to root node to fix below
warning:
arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dtb:
serial@0: Unevaluated properties are not allowed ('clock-sc16is7xx' was unexpected)
Acked-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
index 2517528f684fe..75081ce3e9a6f 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
@@ -20,6 +20,12 @@ sys_mclk: clock-mclk {
clock-frequency = <25000000>;
};
+ sc16is7xx_clk: clock-sc16is7xx {
+ compatible = "fixed-clock";
+ #clock-cells = <0>;
+ clock-frequency = <24000000>;
+ };
+
reg_1p8v: regulator-1p8v {
compatible = "regulator-fixed";
regulator-name = "1P8V";
@@ -69,12 +75,6 @@ serial@0 {
clocks = <&sc16is7xx_clk>;
interrupt-parent = <&gpio1>;
interrupts = <13 IRQ_TYPE_EDGE_FALLING>;
-
- sc16is7xx_clk: clock-sc16is7xx {
- compatible = "fixed-clock";
- #clock-cells = <0>;
- clock-frequency = <24000000>;
- };
};
};
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 05/12] arm64: dts: layerscape: rename mdio-mux-emi to mdio-mux@<addr>
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (3 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 04/12] arm64: dts: fsl-ls1012a-frdm: move clock-sc16is7xx under root node Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 06/12] arm64: dts: fsl-ls1028: add missed supply for wm8904 Frank Li
` (7 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Rename node name mdio-mux-emi<n>@<addr> to mdio-mux@<addr>.
Fix below warning:
arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dtb: board-control@2,0: 'mdio-mux-emi1@54' does not match any of the regexes: '^mdio-mux@[a-f0-9,]+$', 'pinctrl-[0-9]+'
from schema $id: http://devicetree.org/schemas/board/fsl,fpga-qixis.yaml
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 2 +-
arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts | 2 +-
arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
index 11b1356e95d5b..e850551b16acb 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
@@ -211,7 +211,7 @@ ethernet@f0000 { /* DTSEC9/10GEC1 */
};
&fpga {
- mdio-mux-emi1@54 {
+ mdio-mux@54 {
compatible = "mdio-mux-mmioreg", "mdio-mux";
mdio-parent-bus = <&mdio0>;
#address-cells = <1>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
index 25eb1d5a69381..736722b58e77f 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
@@ -237,7 +237,7 @@ &fpga {
#address-cells = <1>;
#size-cells = <1>;
- mdio-mux-emi1 {
+ mdio-mux@54 {
compatible = "mdio-mux-mmioreg", "mdio-mux";
mdio-parent-bus = <&mdio0>;
#address-cells = <1>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi
index 9178cd61c7869..556d8c5f3180a 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi
@@ -64,7 +64,7 @@ boardctrl: board-control@3,0 {
reg = <3 0 0x1000>;
ranges = <0 3 0 0x1000>;
- mdio-mux-emi1@54 {
+ mdio-mux@54 {
compatible = "mdio-mux-mmioreg", "mdio-mux";
mdio-parent-bus = <&emdio1>;
reg = <0x54 1>; /* BRDCFG4 */
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 06/12] arm64: dts: fsl-ls1028: add missed supply for wm8904
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (4 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 05/12] arm64: dts: layerscape: rename mdio-mux-emi to mdio-mux@<addr> Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 07/12] arm64: dts: imx8mm-venice-gw7902(3): add #address-cells for gsc@20 Frank Li
` (6 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Add A(CP, DB, DC, MIC)VDD-supply for wm8904 to fix below warning:
arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dtb:
audio-codec@1a: 'AVDD-supply' is a required property
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
.../dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts
index 195bdbafdf7c9..d9fac647f4327 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts
@@ -26,6 +26,13 @@ pwm-fan {
cooling-levels = <1 128 192 255>;
};
+ reg_3p3v: regulator-3p3v {
+ compatible = "regulator-fixed";
+ regulator-name = "3P3V";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
sound {
#address-cells = <1>;
#size-cells = <0>;
@@ -107,6 +114,11 @@ wm8904: audio-codec@1a {
clock-names = "mclk";
assigned-clocks = <&mclk>;
assigned-clock-rates = <1250000>;
+ AVDD-supply = <®_3p3v>;
+ CPVDD-supply = <®_3p3v>;
+ DBVDD-supply = <®_3p3v>;
+ DCVDD-supply = <®_3p3v>;
+ MICVDD-supply = <®_3p3v>;
};
};
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 07/12] arm64: dts: imx8mm-venice-gw7902(3): add #address-cells for gsc@20
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (5 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 06/12] arm64: dts: fsl-ls1028: add missed supply for wm8904 Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 08/12] arm64: dts: fsl-lx2160a-tqmlx2160a: change "vcc" to "vdd" for hub* Frank Li
` (5 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Add #address-cells and #size-cells for gsc@20 node to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dtb:
gsc@20: '#address-cells' is a required property
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts | 2 ++
arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts | 2 ++
arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts | 2 ++
arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts | 2 ++
4 files changed, 8 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
index 1d56f2a6c06a5..c11260c26d0b4 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
@@ -314,6 +314,8 @@ gsc: gsc@20 {
interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
interrupt-controller;
#interrupt-cells = <1>;
+ #address-cells = <1>;
+ #size-cells = <0>;
adc {
compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
index 45470160f98f5..db1737bf637df 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
@@ -280,6 +280,8 @@ gsc: gsc@20 {
interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
interrupt-controller;
#interrupt-cells = <1>;
+ #address-cells = <1>;
+ #size-cells = <0>;
adc {
compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
index ef951bc9f0dd4..05489a31e7fd8 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
@@ -330,6 +330,8 @@ gsc: gsc@20 {
interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
interrupt-controller;
#interrupt-cells = <1>;
+ #address-cells = <1>;
+ #size-cells = <0>;
adc {
compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
index 72004ab6bda55..0b1fa04f1d678 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
@@ -312,6 +312,8 @@ gsc: gsc@20 {
interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
interrupt-controller;
#interrupt-cells = <1>;
+ #address-cells = <1>;
+ #size-cells = <0>;
adc {
compatible = "gw,gsc-adc";
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 08/12] arm64: dts: fsl-lx2160a-tqmlx2160a: change "vcc" to "vdd" for hub*
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (6 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 07/12] arm64: dts: imx8mm-venice-gw7902(3): add #address-cells for gsc@20 Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 09/12] arm64: dts: imx8mp-venice: add vddl and vana for sensor@10 Frank Li
` (4 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
According to binging doc usb/ti,usb8041.yaml and in
drivers/usb/misc/onboard_usb_dev.h
ti_tusb8041_data = { .supply_names = { "vdd" },};
It should vdd-supply instead vcc-supply.
Fixes: 04b77e0124ef ("arm64: dts: freescale: add fsl-lx2160a-mblx2160a board")
Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
index da0f58e26b9aa..f6a4f8d543015 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
@@ -320,7 +320,7 @@ hub_2_0: hub@1 {
reg = <1>;
peer-hub = <&hub_3_0>;
reset-gpios = <&gpioex1 0 GPIO_ACTIVE_LOW>;
- vcc-supply = <®_vcc3v3>;
+ vdd-supply = <®_vcc3v3>;
};
hub_3_0: hub@2 {
@@ -328,7 +328,7 @@ hub_3_0: hub@2 {
reg = <2>;
peer-hub = <&hub_2_0>;
reset-gpios = <&gpioex1 0 GPIO_ACTIVE_LOW>;
- vcc-supply = <®_vcc3v3>;
+ vdd-supply = <®_vcc3v3>;
};
};
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 09/12] arm64: dts: imx8mp-venice: add vddl and vana for sensor@10
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (7 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 08/12] arm64: dts: fsl-lx2160a-tqmlx2160a: change "vcc" to "vdd" for hub* Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 10/12] arm64: dts: fsl-ls1088a-ten64: change to low case hex value Frank Li
` (3 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Add vddl and vana for sensor@10 to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtb:
sensor@10: 'VANA-supply' is a required property
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
.../dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso | 16 ++++++++++++++++
.../dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso | 16 ++++++++++++++++
.../boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso | 16 ++++++++++++++++
3 files changed, 48 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso
index 4eaf8aabcbfff..c09aa80d2ba23 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso
@@ -13,6 +13,20 @@
&{/} {
compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
+ reg_vana: regulator-2p8v {
+ compatible = "regulator-fixed";
+ regulator-name = "2P8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ };
+
+ reg_vddl: regulator-1p2v {
+ compatible = "regulator-fixed";
+ regulator-name = "1P2V";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ };
+
reg_cam: regulator-cam {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_reg_cam>;
@@ -45,6 +59,8 @@ imx219: sensor@10 {
reg = <0x10>;
clocks = <&cam24m>;
VDIG-supply = <®_cam>;
+ VANA-supply = <®_vana>;
+ VDDL-supply = <®_vddl>;
port {
/* MIPI CSI-2 bus endpoint */
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso
index f3ece4b7fbbde..cfc014eb038d6 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso
@@ -13,6 +13,20 @@
&{/} {
compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
+ reg_vana: regulator-2p8v {
+ compatible = "regulator-fixed";
+ regulator-name = "2P8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ };
+
+ reg_vddl: regulator-1p2v {
+ compatible = "regulator-fixed";
+ regulator-name = "1P2V";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ };
+
reg_cam: regulator-cam {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_reg_cam>;
@@ -45,6 +59,8 @@ imx219: sensor@10 {
reg = <0x10>;
clocks = <&cam24m>;
VDIG-supply = <®_cam>;
+ VANA-supply = <®_vana>;
+ VDDL-supply = <®_vddl>;
port {
/* MIPI CSI-2 bus endpoint */
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso
index eb673a947484a..7d9fcdee58a7f 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso
@@ -11,6 +11,20 @@
/plugin/;
&{/} {
+ reg_vana: regulator-2p8v {
+ compatible = "regulator-fixed";
+ regulator-name = "2P8V";
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+ };
+
+ reg_vddl: regulator-1p2v {
+ compatible = "regulator-fixed";
+ regulator-name = "1P2V";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
+ };
+
reg_cam: regulator-cam {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_reg_cam>;
@@ -39,6 +53,8 @@ imx219: sensor@10 {
reg = <0x10>;
clocks = <&cam24m>;
VDIG-supply = <®_cam>;
+ VANA-supply = <®_vana>;
+ VDDL-supply = <®_vddl>;
port {
/* MIPI CSI-2 bus endpoint */
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 10/12] arm64: dts: fsl-ls1088a-ten64: change to low case hex value
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (8 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 09/12] arm64: dts: imx8mp-venice: add vddl and vana for sensor@10 Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 11/12] arm64: dts: s32v234: remove fallback compatible string arm,cortex-a9-gic Frank Li
` (2 subsequent siblings)
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Change to low case for hex value to fix below warning:
arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: flash@0:
partitions: Unevaluated properties are not allowed ('partition@5C0000' was unexpected)
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
index 57f2082143c8a..bc0d89427fbe5 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
@@ -323,9 +323,9 @@ partition@580000 {
reg = <0x580000 0x40000>;
};
- partition@5C0000 {
+ partition@5c0000 {
label = "dpc";
- reg = <0x5C0000 0x40000>;
+ reg = <0x5c0000 0x40000>;
};
partition@600000 {
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 11/12] arm64: dts: s32v234: remove fallback compatible string arm,cortex-a9-gic
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (9 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 10/12] arm64: dts: fsl-ls1088a-ten64: change to low case hex value Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:01 ` [PATCH v2 12/12] arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply Frank Li
2024-08-31 13:58 ` [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Shawn Guo
12 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
Remove fallback comaptible string arm,cortex-a9-gic to fix below warning:
/home/lizhi/source/linux-upstream-dts/arch/arm64/boot/dts/freescale/s32v234-evb.dtb: interrupt-controller@7d001000: compatible: 'oneOf' conditional failed, one must be fixed:
['arm,cortex-a15-gic', 'arm,cortex-a9-gic'] is too long
/home/lizhi/source/linux-upstream-dts/arch/arm64/boot/dts/freescale/s32v234-evb.dtb: interrupt-controller@7d001000: compatible: 'oneOf' conditional failed, one must be fixed:
['arm,cortex-a15-gic', 'arm,cortex-a9-gic'] is too long
'nvidia,tegra210-agic' was expected
'arm,cortex-a15-gic' is not one of ['nvidia,tegra186-agic', 'nvidia,tegra194-agic', 'nvidia,tegra234-agic']
'arm,gic-400' was expected
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
arch/arm64/boot/dts/freescale/s32v234.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/s32v234.dtsi b/arch/arm64/boot/dts/freescale/s32v234.dtsi
index 42409ec567925..bf608ded5dda6 100644
--- a/arch/arm64/boot/dts/freescale/s32v234.dtsi
+++ b/arch/arm64/boot/dts/freescale/s32v234.dtsi
@@ -89,7 +89,7 @@ IRQ_TYPE_LEVEL_LOW)>,
};
gic: interrupt-controller@7d001000 {
- compatible = "arm,cortex-a15-gic", "arm,cortex-a9-gic";
+ compatible = "arm,cortex-a15-gic";
#interrupt-cells = <3>;
#address-cells = <0>;
interrupt-controller;
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v2 12/12] arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (10 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 11/12] arm64: dts: s32v234: remove fallback compatible string arm,cortex-a9-gic Frank Li
@ 2024-08-19 17:01 ` Frank Li
2024-08-19 17:23 ` Fabio Estevam
2024-08-31 13:58 ` [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Shawn Guo
12 siblings, 1 reply; 16+ messages in thread
From: Frank Li @ 2024-08-19 17:01 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, linux, Frank Li
According to binding doc, DVDD-supply and DOVDD-supply is required
properties. Add these to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dtb: camera@10: 'DVDD-supply' is a required proper
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
.../boot/dts/freescale/imx8mm-beacon-baseboard.dtsi | 16 ++++++++++++++++
.../boot/dts/freescale/imx8mn-beacon-baseboard.dtsi | 16 ++++++++++++++++
2 files changed, 32 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
index 6086dae2e5fbe..ea1d5b9c6bae0 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
@@ -56,6 +56,20 @@ pcie0_refclk_gated: pcie0-refclk-gated {
enable-gpios = <&pca6416_1 2 GPIO_ACTIVE_LOW>;
};
+ reg_1v5: regulator-1v5 {
+ compatible = "regulator-fixed";
+ regulator-name = "1V5";
+ regulator-min-microvolt = <1500000>;
+ regulator-max-microvolt = <1500000>;
+ };
+
+ reg_1v8: regulator-1v8 {
+ compatible = "regulator-fixed";
+ regulator-name = "1V8";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ };
+
reg_audio: regulator-audio {
compatible = "regulator-fixed";
regulator-name = "3v3_aud";
@@ -187,6 +201,8 @@ camera@10 {
assigned-clock-parents = <&clk IMX8MM_CLK_24M>;
assigned-clock-rates = <24000000>;
AVDD-supply = <®_camera>; /* 2.8v */
+ DVDD-supply = <®_1v5>;
+ DOVDD-supply = <®_1v8>;
powerdown-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>;
reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
index 20018ee2c803e..77d14ea459e57 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
@@ -40,6 +40,20 @@ led-3 {
};
};
+ reg_1v5: regulator-1v5 {
+ compatible = "regulator-fixed";
+ regulator-name = "1V5";
+ regulator-min-microvolt = <1500000>;
+ regulator-max-microvolt = <1500000>;
+ };
+
+ reg_1v8: regulator-1v8 {
+ compatible = "regulator-fixed";
+ regulator-name = "1V8";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+ };
+
reg_audio: regulator-audio {
compatible = "regulator-fixed";
regulator-name = "3v3_aud";
@@ -158,6 +172,8 @@ camera@10 {
assigned-clock-parents = <&clk IMX8MN_CLK_24M>;
assigned-clock-rates = <24000000>;
AVDD-supply = <®_camera>; /* 2.8v */
+ DVDD-supply = <®_1v5>;
+ DOVDD-supply = <®_1v8>;
powerdown-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>;
reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [PATCH v2 12/12] arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply
2024-08-19 17:01 ` [PATCH v2 12/12] arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply Frank Li
@ 2024-08-19 17:23 ` Fabio Estevam
2024-08-19 17:56 ` Adam Ford
0 siblings, 1 reply; 16+ messages in thread
From: Fabio Estevam @ 2024-08-19 17:23 UTC (permalink / raw)
To: Frank Li, Adam Ford
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Gregor Herburger,
Alexander Stein, devicetree, imx, linux-arm-kernel, linux-kernel,
linux
Adding Adam.
On Mon, Aug 19, 2024 at 2:03 PM Frank Li <Frank.Li@nxp.com> wrote:
>
> According to binding doc, DVDD-supply and DOVDD-supply is required
> properties. Add these to fix below warning:
> arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dtb: camera@10: 'DVDD-supply' is a required proper
>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
> .../boot/dts/freescale/imx8mm-beacon-baseboard.dtsi | 16 ++++++++++++++++
> .../boot/dts/freescale/imx8mn-beacon-baseboard.dtsi | 16 ++++++++++++++++
> 2 files changed, 32 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
> index 6086dae2e5fbe..ea1d5b9c6bae0 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
> @@ -56,6 +56,20 @@ pcie0_refclk_gated: pcie0-refclk-gated {
> enable-gpios = <&pca6416_1 2 GPIO_ACTIVE_LOW>;
> };
>
> + reg_1v5: regulator-1v5 {
> + compatible = "regulator-fixed";
> + regulator-name = "1V5";
> + regulator-min-microvolt = <1500000>;
> + regulator-max-microvolt = <1500000>;
> + };
> +
> + reg_1v8: regulator-1v8 {
> + compatible = "regulator-fixed";
> + regulator-name = "1V8";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + };
> +
> reg_audio: regulator-audio {
> compatible = "regulator-fixed";
> regulator-name = "3v3_aud";
> @@ -187,6 +201,8 @@ camera@10 {
> assigned-clock-parents = <&clk IMX8MM_CLK_24M>;
> assigned-clock-rates = <24000000>;
> AVDD-supply = <®_camera>; /* 2.8v */
> + DVDD-supply = <®_1v5>;
> + DOVDD-supply = <®_1v8>;
> powerdown-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>;
> reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
> index 20018ee2c803e..77d14ea459e57 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
> @@ -40,6 +40,20 @@ led-3 {
> };
> };
>
> + reg_1v5: regulator-1v5 {
> + compatible = "regulator-fixed";
> + regulator-name = "1V5";
> + regulator-min-microvolt = <1500000>;
> + regulator-max-microvolt = <1500000>;
> + };
> +
> + reg_1v8: regulator-1v8 {
> + compatible = "regulator-fixed";
> + regulator-name = "1V8";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + };
> +
> reg_audio: regulator-audio {
> compatible = "regulator-fixed";
> regulator-name = "3v3_aud";
> @@ -158,6 +172,8 @@ camera@10 {
> assigned-clock-parents = <&clk IMX8MN_CLK_24M>;
> assigned-clock-rates = <24000000>;
> AVDD-supply = <®_camera>; /* 2.8v */
> + DVDD-supply = <®_1v5>;
> + DOVDD-supply = <®_1v8>;
> powerdown-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>;
> reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
>
>
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread* Re: [PATCH v2 12/12] arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply
2024-08-19 17:23 ` Fabio Estevam
@ 2024-08-19 17:56 ` Adam Ford
0 siblings, 0 replies; 16+ messages in thread
From: Adam Ford @ 2024-08-19 17:56 UTC (permalink / raw)
To: Fabio Estevam
Cc: Frank Li, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
Gregor Herburger, Alexander Stein, devicetree, imx,
linux-arm-kernel, linux-kernel, linux
On Mon, Aug 19, 2024 at 12:24 PM Fabio Estevam <festevam@gmail.com> wrote:
>
> Adding Adam.
>
> On Mon, Aug 19, 2024 at 2:03 PM Frank Li <Frank.Li@nxp.com> wrote:
> >
> > According to binding doc, DVDD-supply and DOVDD-supply is required
> > properties. Add these to fix below warning:
> > arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dtb: camera@10: 'DVDD-supply' is a required proper
> >
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
Thanks for doing that!
Reviewed-by: Adam Ford <aford173@gmail.com>
> > ---
> > .../boot/dts/freescale/imx8mm-beacon-baseboard.dtsi | 16 ++++++++++++++++
> > .../boot/dts/freescale/imx8mn-beacon-baseboard.dtsi | 16 ++++++++++++++++
> > 2 files changed, 32 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
> > index 6086dae2e5fbe..ea1d5b9c6bae0 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
> > @@ -56,6 +56,20 @@ pcie0_refclk_gated: pcie0-refclk-gated {
> > enable-gpios = <&pca6416_1 2 GPIO_ACTIVE_LOW>;
> > };
> >
> > + reg_1v5: regulator-1v5 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "1V5";
> > + regulator-min-microvolt = <1500000>;
> > + regulator-max-microvolt = <1500000>;
> > + };
> > +
> > + reg_1v8: regulator-1v8 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "1V8";
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1800000>;
> > + };
> > +
> > reg_audio: regulator-audio {
> > compatible = "regulator-fixed";
> > regulator-name = "3v3_aud";
> > @@ -187,6 +201,8 @@ camera@10 {
> > assigned-clock-parents = <&clk IMX8MM_CLK_24M>;
> > assigned-clock-rates = <24000000>;
> > AVDD-supply = <®_camera>; /* 2.8v */
> > + DVDD-supply = <®_1v5>;
> > + DOVDD-supply = <®_1v8>;
> > powerdown-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>;
> > reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
> > index 20018ee2c803e..77d14ea459e57 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
> > @@ -40,6 +40,20 @@ led-3 {
> > };
> > };
> >
> > + reg_1v5: regulator-1v5 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "1V5";
> > + regulator-min-microvolt = <1500000>;
> > + regulator-max-microvolt = <1500000>;
> > + };
> > +
> > + reg_1v8: regulator-1v8 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "1V8";
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1800000>;
> > + };
> > +
> > reg_audio: regulator-audio {
> > compatible = "regulator-fixed";
> > regulator-name = "3v3_aud";
> > @@ -158,6 +172,8 @@ camera@10 {
> > assigned-clock-parents = <&clk IMX8MN_CLK_24M>;
> > assigned-clock-rates = <24000000>;
> > AVDD-supply = <®_camera>; /* 2.8v */
> > + DVDD-supply = <®_1v5>;
> > + DOVDD-supply = <®_1v8>;
> > powerdown-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>;
> > reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
> >
> >
> > --
> > 2.34.1
> >
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up
2024-08-19 17:01 [PATCH v2 00/12] arm64: dts: imx: 3rd dtb warning clearn up Frank Li
` (11 preceding siblings ...)
2024-08-19 17:01 ` [PATCH v2 12/12] arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply Frank Li
@ 2024-08-31 13:58 ` Shawn Guo
12 siblings, 0 replies; 16+ messages in thread
From: Shawn Guo @ 2024-08-31 13:58 UTC (permalink / raw)
To: Frank Li
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
Gregor Herburger, Alexander Stein, devicetree, imx,
linux-arm-kernel, linux-kernel, linux
On Mon, Aug 19, 2024 at 01:01:09PM -0400, Frank Li wrote:
> Frank Li (12):
> arm64: dts: imx8-ss-img: remove undocument slot for jpeg
> arm64: dts: fsl-ls1043a: move "fsl,ls1043a-qdma" ahead "fsl,ls1021a-qdma"
> arm64: dts: imx: rename gpio hog as <gpio name>-hog
Applied all except #3 which doesn't apply to imx/dt64 branch.
Shawn
> arm64: dts: fsl-ls1012a-frdm: move clock-sc16is7xx under root node
> arm64: dts: layerscape: rename mdio-mux-emi to mdio-mux@<addr>
> arm64: dts: fsl-ls1028: add missed supply for wm8904
> arm64: dts: imx8mm-venice-gw7902(3): add #address-cells for gsc@20
> arm64: dts: fsl-lx2160a-tqmlx2160a: change "vcc" to "vdd" for hub*
> arm64: dts: imx8mp-venice: add vddl and vana for sensor@10
> arm64: dts: fsl-ls1088a-ten64: change to low case hex value
> arm64: dts: s32v234: remove fallback compatible string arm,cortex-a9-gic
> arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply
^ permalink raw reply [flat|nested] 16+ messages in thread