* [PATCH v2 0/7] various imx8m*-venice dt fixes
@ 2025-09-18 15:44 Tim Harvey
2025-09-18 15:44 ` [PATCH v2 1/7] arm64: dts: freescale: imx8mp-venice-gw7905-2x: remove duplicate usdhc1 props Tim Harvey
` (7 more replies)
0 siblings, 8 replies; 12+ messages in thread
From: Tim Harvey @ 2025-09-18 15:44 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Tim Harvey
This series contains various imx8m*-venice board dt fixes.
v2:
- add additional info to commit explaining changing TXC slew rate for
improved signal quality and emiisions (Peng)
- add tags
Tim Harvey (7):
arm64: dts: freescale: imx8mp-venice-gw7905-2x: remove duplicate
usdhc1 props
arm64: dts: imx8m{m,n,p}-venice: disable unused clk output for TI PHY
arm64: dts: imx8mp-venice-gw702x: reduce RGMII CLK drive strength
arm64: dts: imx8mm-venice-gw700x: reduce RGMII CLK drive strength
arm64: dts: imx8mm-venice-gw72xx: remove unused sdhc1 pinctrl
arm64: dts: imx8mp-venice-gw702x: remove off-board uart
arm64: dts: imx8mp-venice-gw702x: remove off-board sdhc1
.../dts/freescale/imx8mm-venice-gw700x.dtsi | 3 +-
.../dts/freescale/imx8mm-venice-gw72xx.dtsi | 11 ----
.../dts/freescale/imx8mm-venice-gw7902.dts | 1 +
.../dts/freescale/imx8mn-venice-gw7902.dts | 1 +
.../dts/freescale/imx8mp-venice-gw702x.dtsi | 54 +------------------
.../dts/freescale/imx8mp-venice-gw72xx.dtsi | 11 ----
.../dts/freescale/imx8mp-venice-gw74xx.dts | 1 +
7 files changed, 7 insertions(+), 75 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2 1/7] arm64: dts: freescale: imx8mp-venice-gw7905-2x: remove duplicate usdhc1 props
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
@ 2025-09-18 15:44 ` Tim Harvey
2025-09-18 15:44 ` [PATCH v2 2/7] arm64: dts: imx8m{m,n,p}-venice: disable unused clk output for TI PHY Tim Harvey
` (6 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Tim Harvey @ 2025-09-18 15:44 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Tim Harvey,
Peng Fan
Remove the un-intended duplicate properties from usdhc1.
Fixes: 0d5b288c2110e ("arm64: dts: freescale: Add imx8mp-venice-gw7905-2x")
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
---
v2:
- add tags
---
arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
index cbf0c9a740fa..303995a8adce 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
@@ -423,9 +423,6 @@ &usdhc1 {
bus-width = <4>;
non-removable;
status = "okay";
- bus-width = <4>;
- non-removable;
- status = "okay";
};
/* eMMC */
--
2.25.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 2/7] arm64: dts: imx8m{m,n,p}-venice: disable unused clk output for TI PHY
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
2025-09-18 15:44 ` [PATCH v2 1/7] arm64: dts: freescale: imx8mp-venice-gw7905-2x: remove duplicate usdhc1 props Tim Harvey
@ 2025-09-18 15:44 ` Tim Harvey
2025-09-18 15:44 ` [PATCH v2 3/7] arm64: dts: imx8mp-venice-gw702x: reduce RGMII CLK drive strength Tim Harvey
` (5 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Tim Harvey @ 2025-09-18 15:44 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Tim Harvey,
Peng Fan
Disable the unused refclk output for the TI DP83867 PHY used on
Gateworks Venice boards.
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
---
v2:
- add tags
---
arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi | 1 +
arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts | 1 +
arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts | 1 +
arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi | 1 +
arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts | 1 +
5 files changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi
index 37db4f0dd505..baf46fe28133 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi
@@ -115,6 +115,7 @@ mdio {
ethphy0: ethernet-phy@0 {
compatible = "ethernet-phy-ieee802.3-c22";
reg = <0>;
+ ti,clk-output-sel = <DP83867_CLK_O_SEL_OFF>;
ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
tx-fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
index c09b40fc6dec..468c7e993c52 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
@@ -253,6 +253,7 @@ mdio {
ethphy0: ethernet-phy@0 {
compatible = "ethernet-phy-ieee802.3-c22";
reg = <0>;
+ ti,clk-output-sel = <DP83867_CLK_O_SEL_OFF>;
ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
tx-fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
index a5f52f60169e..5aa0e2cd155e 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
@@ -248,6 +248,7 @@ mdio {
ethphy0: ethernet-phy@0 {
compatible = "ethernet-phy-ieee802.3-c22";
reg = <0>;
+ ti,clk-output-sel = <DP83867_CLK_O_SEL_OFF>;
ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
tx-fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
index 303995a8adce..a1232a4f8485 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
@@ -101,6 +101,7 @@ ethphy0: ethernet-phy@0 {
reg = <0x0>;
interrupt-parent = <&gpio3>;
interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
+ ti,clk-output-sel = <DP83867_CLK_O_SEL_OFF>;
ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
tx-fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts
index 12de7cf1e853..7662663ff5da 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts
@@ -228,6 +228,7 @@ mdio {
ethphy0: ethernet-phy@0 {
compatible = "ethernet-phy-ieee802.3-c22";
reg = <0x0>;
+ ti,clk-output-sel = <DP83867_CLK_O_SEL_OFF>;
ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>;
tx-fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
--
2.25.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 3/7] arm64: dts: imx8mp-venice-gw702x: reduce RGMII CLK drive strength
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
2025-09-18 15:44 ` [PATCH v2 1/7] arm64: dts: freescale: imx8mp-venice-gw7905-2x: remove duplicate usdhc1 props Tim Harvey
2025-09-18 15:44 ` [PATCH v2 2/7] arm64: dts: imx8m{m,n,p}-venice: disable unused clk output for TI PHY Tim Harvey
@ 2025-09-18 15:44 ` Tim Harvey
2025-09-19 4:46 ` Peng Fan
2025-09-18 15:44 ` [PATCH v2 4/7] arm64: dts: imx8mm-venice-gw700x: " Tim Harvey
` (4 subsequent siblings)
7 siblings, 1 reply; 12+ messages in thread
From: Tim Harvey @ 2025-09-18 15:44 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Tim Harvey
The i.MX8M Plus EQOS RGMII tracelength is less than 1in and does not
require a x6 drive strength. Reduce the CLK drive strength to x1 for
lower emissions. Additionally since TXC is not a high frequency clock,
use slow slew rate (FSEL=0) for lower emmissions and improved signal
quality.
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
v2: add FSEL detail to log (Peng)
---
arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
index a1232a4f8485..dd9eeb3479fd 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
@@ -462,7 +462,7 @@ MX8MP_IOMUXC_ENET_TD1__ENET_QOS_RGMII_TD1 0x16
MX8MP_IOMUXC_ENET_TD2__ENET_QOS_RGMII_TD2 0x16
MX8MP_IOMUXC_ENET_TD3__ENET_QOS_RGMII_TD3 0x16
MX8MP_IOMUXC_ENET_TX_CTL__ENET_QOS_RGMII_TX_CTL 0x16
- MX8MP_IOMUXC_ENET_TXC__CCM_ENET_QOS_CLOCK_GENERATE_TX_CLK 0x16
+ MX8MP_IOMUXC_ENET_TXC__CCM_ENET_QOS_CLOCK_GENERATE_TX_CLK 0x0
>;
};
--
2.25.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 4/7] arm64: dts: imx8mm-venice-gw700x: reduce RGMII CLK drive strength
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
` (2 preceding siblings ...)
2025-09-18 15:44 ` [PATCH v2 3/7] arm64: dts: imx8mp-venice-gw702x: reduce RGMII CLK drive strength Tim Harvey
@ 2025-09-18 15:44 ` Tim Harvey
2025-09-19 4:46 ` Peng Fan
2025-09-18 15:44 ` [PATCH v2 5/7] arm64: dts: imx8mm-venice-gw72xx: remove unused sdhc1 pinctrl Tim Harvey
` (3 subsequent siblings)
7 siblings, 1 reply; 12+ messages in thread
From: Tim Harvey @ 2025-09-18 15:44 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Tim Harvey
The i.MX8M Mini FEC RGMII tracelength is less than 1in and does not
require a x6 drive strength. Reduce the CLK drive strength to x1 for
lower emissions. Additionally since TXC is not a high frequency clock,
use slow slew rate (FSEL=0) for lower emmissions and improved signal
quality.
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
v2: add FSEL detail to log (Peng)
---
arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi
index baf46fe28133..dca213c85cc3 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi
@@ -446,7 +446,7 @@ MX8MM_IOMUXC_ENET_RD3_ENET1_RGMII_RD3 0x91
MX8MM_IOMUXC_ENET_RD2_ENET1_RGMII_RD2 0x91
MX8MM_IOMUXC_ENET_RD1_ENET1_RGMII_RD1 0x91
MX8MM_IOMUXC_ENET_RD0_ENET1_RGMII_RD0 0x91
- MX8MM_IOMUXC_ENET_TXC_ENET1_RGMII_TXC 0x1f
+ MX8MM_IOMUXC_ENET_TXC_ENET1_RGMII_TXC 0x0
MX8MM_IOMUXC_ENET_RXC_ENET1_RGMII_RXC 0x91
MX8MM_IOMUXC_ENET_RX_CTL_ENET1_RGMII_RX_CTL 0x91
MX8MM_IOMUXC_ENET_TX_CTL_ENET1_RGMII_TX_CTL 0x1f
--
2.25.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 5/7] arm64: dts: imx8mm-venice-gw72xx: remove unused sdhc1 pinctrl
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
` (3 preceding siblings ...)
2025-09-18 15:44 ` [PATCH v2 4/7] arm64: dts: imx8mm-venice-gw700x: " Tim Harvey
@ 2025-09-18 15:44 ` Tim Harvey
2025-09-18 15:44 ` [PATCH v2 6/7] arm64: dts: imx8mp-venice-gw702x: remove off-board uart Tim Harvey
` (2 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Tim Harvey @ 2025-09-18 15:44 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Tim Harvey,
Peng Fan
The SDHC1 interface is not used on the imx8mm-venice-gw72xx. Remove the
unused pinctrl_usdhc1 iomux node.
Fixes: 6f30b27c5ef5 ("arm64: dts: imx8mm: Add Gateworks i.MX 8M Mini Development Kits")
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
---
v2:
- add tags
---
.../boot/dts/freescale/imx8mm-venice-gw72xx.dtsi | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx.dtsi
index 752caa38eb03..266038fbbef9 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx.dtsi
@@ -351,17 +351,6 @@ MX8MM_IOMUXC_UART4_TXD_UART4_DCE_TX 0x140
>;
};
- pinctrl_usdhc1: usdhc1grp {
- fsl,pins = <
- MX8MM_IOMUXC_SD1_CLK_USDHC1_CLK 0x190
- MX8MM_IOMUXC_SD1_CMD_USDHC1_CMD 0x1d0
- MX8MM_IOMUXC_SD1_DATA0_USDHC1_DATA0 0x1d0
- MX8MM_IOMUXC_SD1_DATA1_USDHC1_DATA1 0x1d0
- MX8MM_IOMUXC_SD1_DATA2_USDHC1_DATA2 0x1d0
- MX8MM_IOMUXC_SD1_DATA3_USDHC1_DATA3 0x1d0
- >;
- };
-
pinctrl_usdhc2: usdhc2grp {
fsl,pins = <
MX8MM_IOMUXC_SD2_CLK_USDHC2_CLK 0x190
--
2.25.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 6/7] arm64: dts: imx8mp-venice-gw702x: remove off-board uart
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
` (4 preceding siblings ...)
2025-09-18 15:44 ` [PATCH v2 5/7] arm64: dts: imx8mm-venice-gw72xx: remove unused sdhc1 pinctrl Tim Harvey
@ 2025-09-18 15:44 ` Tim Harvey
2025-09-18 15:44 ` [PATCH v2 7/7] arm64: dts: imx8mp-venice-gw702x: remove off-board sdhc1 Tim Harvey
2025-10-21 8:30 ` [PATCH v2 0/7] various imx8m*-venice dt fixes Shawn Guo
7 siblings, 0 replies; 12+ messages in thread
From: Tim Harvey @ 2025-09-18 15:44 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Tim Harvey,
Peng Fan
UART1 and UART3 go to a connector for use on a baseboard and as such are
defined in the baseboard device-trees. Remove them from the gw702x SOM
device-tree.
Fixes: 0d5b288c2110 ("arm64: dts: freescale: Add imx8mp-venice-gw7905-2x")
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
---
v2:
- add tags
---
.../dts/freescale/imx8mp-venice-gw702x.dtsi | 28 -------------------
1 file changed, 28 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
index dd9eeb3479fd..dba35b3394bc 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
@@ -396,13 +396,6 @@ &i2c3 {
status = "okay";
};
-/* off-board header */
-&uart1 {
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_uart1>;
- status = "okay";
-};
-
/* console */
&uart2 {
pinctrl-names = "default";
@@ -410,13 +403,6 @@ &uart2 {
status = "okay";
};
-/* off-board header */
-&uart3 {
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_uart3>;
- status = "okay";
-};
-
/* off-board */
&usdhc1 {
pinctrl-names = "default";
@@ -521,13 +507,6 @@ MX8MP_IOMUXC_I2C3_SDA__GPIO5_IO19 0x400001c2
>;
};
- pinctrl_uart1: uart1grp {
- fsl,pins = <
- MX8MP_IOMUXC_UART1_RXD__UART1_DCE_RX 0x140
- MX8MP_IOMUXC_UART1_TXD__UART1_DCE_TX 0x140
- >;
- };
-
pinctrl_uart2: uart2grp {
fsl,pins = <
MX8MP_IOMUXC_UART2_RXD__UART2_DCE_RX 0x140
@@ -535,13 +514,6 @@ MX8MP_IOMUXC_UART2_TXD__UART2_DCE_TX 0x140
>;
};
- pinctrl_uart3: uart3grp {
- fsl,pins = <
- MX8MP_IOMUXC_UART3_RXD__UART3_DCE_RX 0x140
- MX8MP_IOMUXC_UART3_TXD__UART3_DCE_TX 0x140
- >;
- };
-
pinctrl_usdhc1: usdhc1grp {
fsl,pins = <
MX8MP_IOMUXC_SD1_CLK__USDHC1_CLK 0x190
--
2.25.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 7/7] arm64: dts: imx8mp-venice-gw702x: remove off-board sdhc1
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
` (5 preceding siblings ...)
2025-09-18 15:44 ` [PATCH v2 6/7] arm64: dts: imx8mp-venice-gw702x: remove off-board uart Tim Harvey
@ 2025-09-18 15:44 ` Tim Harvey
2025-09-19 4:45 ` Peng Fan
2025-10-21 8:30 ` [PATCH v2 0/7] various imx8m*-venice dt fixes Shawn Guo
7 siblings, 1 reply; 12+ messages in thread
From: Tim Harvey @ 2025-09-18 15:44 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Tim Harvey
SDHC1 on the GW702x SOM routes to a connector for use on a baseboard
and as such are defined in the baseboard device-trees.
Remove it from the gw702x SOM device-tree.
Fixes: 0d5b288c2110 ("arm64: dts: freescale: Add imx8mp-venice-gw7905-2x")
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
v2:
- no changes
---
.../dts/freescale/imx8mp-venice-gw702x.dtsi | 20 -------------------
.../dts/freescale/imx8mp-venice-gw72xx.dtsi | 11 ----------
2 files changed, 31 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
index dba35b3394bc..de852ebff571 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw702x.dtsi
@@ -403,15 +403,6 @@ &uart2 {
status = "okay";
};
-/* off-board */
-&usdhc1 {
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_usdhc1>;
- bus-width = <4>;
- non-removable;
- status = "okay";
-};
-
/* eMMC */
&usdhc3 {
pinctrl-names = "default", "state_100mhz", "state_200mhz";
@@ -514,17 +505,6 @@ MX8MP_IOMUXC_UART2_TXD__UART2_DCE_TX 0x140
>;
};
- pinctrl_usdhc1: usdhc1grp {
- fsl,pins = <
- MX8MP_IOMUXC_SD1_CLK__USDHC1_CLK 0x190
- MX8MP_IOMUXC_SD1_CMD__USDHC1_CMD 0x1d0
- MX8MP_IOMUXC_SD1_DATA0__USDHC1_DATA0 0x1d0
- MX8MP_IOMUXC_SD1_DATA1__USDHC1_DATA1 0x1d0
- MX8MP_IOMUXC_SD1_DATA2__USDHC1_DATA2 0x1d0
- MX8MP_IOMUXC_SD1_DATA3__USDHC1_DATA3 0x1d0
- >;
- };
-
pinctrl_usdhc3: usdhc3grp {
fsl,pins = <
MX8MP_IOMUXC_NAND_WE_B__USDHC3_CLK 0x190
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi
index cf747ec6fa16..76020ef89bf3 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi
@@ -365,17 +365,6 @@ MX8MP_IOMUXC_UART4_TXD__UART4_DCE_TX 0x140
>;
};
- pinctrl_usdhc1: usdhc1grp {
- fsl,pins = <
- MX8MP_IOMUXC_SD1_CLK__USDHC1_CLK 0x190
- MX8MP_IOMUXC_SD1_CMD__USDHC1_CMD 0x1d0
- MX8MP_IOMUXC_SD1_DATA0__USDHC1_DATA0 0x1d0
- MX8MP_IOMUXC_SD1_DATA1__USDHC1_DATA1 0x1d0
- MX8MP_IOMUXC_SD1_DATA2__USDHC1_DATA2 0x1d0
- MX8MP_IOMUXC_SD1_DATA3__USDHC1_DATA3 0x1d0
- >;
- };
-
pinctrl_usdhc2: usdhc2grp {
fsl,pins = <
MX8MP_IOMUXC_SD2_CLK__USDHC2_CLK 0x190
--
2.25.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v2 7/7] arm64: dts: imx8mp-venice-gw702x: remove off-board sdhc1
2025-09-18 15:44 ` [PATCH v2 7/7] arm64: dts: imx8mp-venice-gw702x: remove off-board sdhc1 Tim Harvey
@ 2025-09-19 4:45 ` Peng Fan
0 siblings, 0 replies; 12+ messages in thread
From: Peng Fan @ 2025-09-19 4:45 UTC (permalink / raw)
To: Tim Harvey
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, Sep 18, 2025 at 08:44:51AM -0700, Tim Harvey wrote:
>SDHC1 on the GW702x SOM routes to a connector for use on a baseboard
>and as such are defined in the baseboard device-trees.
>
>Remove it from the gw702x SOM device-tree.
>
>Fixes: 0d5b288c2110 ("arm64: dts: freescale: Add imx8mp-venice-gw7905-2x")
>Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 3/7] arm64: dts: imx8mp-venice-gw702x: reduce RGMII CLK drive strength
2025-09-18 15:44 ` [PATCH v2 3/7] arm64: dts: imx8mp-venice-gw702x: reduce RGMII CLK drive strength Tim Harvey
@ 2025-09-19 4:46 ` Peng Fan
0 siblings, 0 replies; 12+ messages in thread
From: Peng Fan @ 2025-09-19 4:46 UTC (permalink / raw)
To: Tim Harvey
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, Sep 18, 2025 at 08:44:47AM -0700, Tim Harvey wrote:
>The i.MX8M Plus EQOS RGMII tracelength is less than 1in and does not
>require a x6 drive strength. Reduce the CLK drive strength to x1 for
>lower emissions. Additionally since TXC is not a high frequency clock,
>use slow slew rate (FSEL=0) for lower emmissions and improved signal
>quality.
>
>Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 4/7] arm64: dts: imx8mm-venice-gw700x: reduce RGMII CLK drive strength
2025-09-18 15:44 ` [PATCH v2 4/7] arm64: dts: imx8mm-venice-gw700x: " Tim Harvey
@ 2025-09-19 4:46 ` Peng Fan
0 siblings, 0 replies; 12+ messages in thread
From: Peng Fan @ 2025-09-19 4:46 UTC (permalink / raw)
To: Tim Harvey
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, Sep 18, 2025 at 08:44:48AM -0700, Tim Harvey wrote:
>The i.MX8M Mini FEC RGMII tracelength is less than 1in and does not
>require a x6 drive strength. Reduce the CLK drive strength to x1 for
>lower emissions. Additionally since TXC is not a high frequency clock,
>use slow slew rate (FSEL=0) for lower emmissions and improved signal
>quality.
>
>Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 0/7] various imx8m*-venice dt fixes
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
` (6 preceding siblings ...)
2025-09-18 15:44 ` [PATCH v2 7/7] arm64: dts: imx8mp-venice-gw702x: remove off-board sdhc1 Tim Harvey
@ 2025-10-21 8:30 ` Shawn Guo
7 siblings, 0 replies; 12+ messages in thread
From: Shawn Guo @ 2025-10-21 8:30 UTC (permalink / raw)
To: Tim Harvey
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, Sep 18, 2025 at 08:44:44AM -0700, Tim Harvey wrote:
> This series contains various imx8m*-venice board dt fixes.
>
> v2:
> - add additional info to commit explaining changing TXC slew rate for
> improved signal quality and emiisions (Peng)
> - add tags
>
> Tim Harvey (7):
> arm64: dts: freescale: imx8mp-venice-gw7905-2x: remove duplicate
> usdhc1 props
> arm64: dts: imx8m{m,n,p}-venice: disable unused clk output for TI PHY
> arm64: dts: imx8mp-venice-gw702x: reduce RGMII CLK drive strength
> arm64: dts: imx8mm-venice-gw700x: reduce RGMII CLK drive strength
> arm64: dts: imx8mm-venice-gw72xx: remove unused sdhc1 pinctrl
> arm64: dts: imx8mp-venice-gw702x: remove off-board uart
> arm64: dts: imx8mp-venice-gw702x: remove off-board sdhc1
Applied all, thanks!
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2025-10-21 8:30 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-18 15:44 [PATCH v2 0/7] various imx8m*-venice dt fixes Tim Harvey
2025-09-18 15:44 ` [PATCH v2 1/7] arm64: dts: freescale: imx8mp-venice-gw7905-2x: remove duplicate usdhc1 props Tim Harvey
2025-09-18 15:44 ` [PATCH v2 2/7] arm64: dts: imx8m{m,n,p}-venice: disable unused clk output for TI PHY Tim Harvey
2025-09-18 15:44 ` [PATCH v2 3/7] arm64: dts: imx8mp-venice-gw702x: reduce RGMII CLK drive strength Tim Harvey
2025-09-19 4:46 ` Peng Fan
2025-09-18 15:44 ` [PATCH v2 4/7] arm64: dts: imx8mm-venice-gw700x: " Tim Harvey
2025-09-19 4:46 ` Peng Fan
2025-09-18 15:44 ` [PATCH v2 5/7] arm64: dts: imx8mm-venice-gw72xx: remove unused sdhc1 pinctrl Tim Harvey
2025-09-18 15:44 ` [PATCH v2 6/7] arm64: dts: imx8mp-venice-gw702x: remove off-board uart Tim Harvey
2025-09-18 15:44 ` [PATCH v2 7/7] arm64: dts: imx8mp-venice-gw702x: remove off-board sdhc1 Tim Harvey
2025-09-19 4:45 ` Peng Fan
2025-10-21 8:30 ` [PATCH v2 0/7] various imx8m*-venice dt fixes Shawn Guo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.