* [PATCH 1/9] arm64: dts: imx8ulp: Add DMA channel properties and use eDMA flag macros
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
2026-07-15 12:33 ` [PATCH 2/9] arm64: dts: imx8ulp: Add I3C2 controller node Peng Fan (OSS)
` (7 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Add DMA channel (dmas/dma-names) properties to peripheral nodes that
were missing them:
- LPI2C: lpi2c4, lpi2c5, lpi2c6, lpi2c7
- LPUART: lpuart4, lpuart5, lpuart6, lpuart7
- LPSPI: lpspi4, lpspi5
Also replace hardcoded numeric DMA direction flags with the proper
FSL_EDMA_RX and FSL_EDMA_MULTI_FIFO macros for SAI (sai4, sai5, sai6,
sai7) and SPDIF nodes.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 32 +++++++++++++++++++++++++-----
1 file changed, 27 insertions(+), 5 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
index c6d1bb9edf388..5438958176985 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
@@ -4,6 +4,7 @@
*/
#include <dt-bindings/clock/imx8ulp-clock.h>
+#include <dt-bindings/dma/fsl-edma.h>
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/arm-gic.h>
#include <dt-bindings/power/imx8ulp-power.h>
@@ -370,6 +371,8 @@ lpi2c4: i2c@29370000 {
assigned-clocks = <&pcc3 IMX8ULP_CLK_LPI2C4>;
assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
assigned-clock-rates = <48000000>;
+ dmas = <&edma1 46 0 0>, <&edma1 45 0 FSL_EDMA_RX>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -383,6 +386,8 @@ lpi2c5: i2c@29380000 {
assigned-clocks = <&pcc3 IMX8ULP_CLK_LPI2C5>;
assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
assigned-clock-rates = <48000000>;
+ dmas = <&edma1 48 0 0>, <&edma1 47 0 FSL_EDMA_RX>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -392,6 +397,8 @@ lpuart4: serial@29390000 {
interrupts = <GIC_SPI 99 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc3 IMX8ULP_CLK_LPUART4>;
clock-names = "ipg";
+ dmas = <&edma1 55 0 FSL_EDMA_RX>, <&edma1 56 0 0> ;
+ dma-names = "rx", "tx";
status = "disabled";
};
@@ -401,6 +408,8 @@ lpuart5: serial@293a0000 {
interrupts = <GIC_SPI 100 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc3 IMX8ULP_CLK_LPUART5>;
clock-names = "ipg";
+ dmas = <&edma1 57 0 FSL_EDMA_RX>, <&edma1 58 0 0> ;
+ dma-names = "rx", "tx";
status = "disabled";
};
@@ -416,6 +425,8 @@ lpspi4: spi@293b0000 {
assigned-clocks = <&pcc3 IMX8ULP_CLK_LPSPI4>;
assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
assigned-clock-rates = <48000000>;
+ dmas = <&edma1 64 0 0>, <&edma1 63 0 FSL_EDMA_RX>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -431,6 +442,8 @@ lpspi5: spi@293c0000 {
assigned-clocks = <&pcc3 IMX8ULP_CLK_LPSPI5>;
assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
assigned-clock-rates = <48000000>;
+ dmas = <&edma1 66 0 0>, <&edma1 65 0 FSL_EDMA_RX>;
+ dma-names = "tx", "rx";
status = "disabled";
};
};
@@ -474,6 +487,8 @@ lpi2c6: i2c@29840000 {
assigned-clocks = <&pcc4 IMX8ULP_CLK_LPI2C6>;
assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
assigned-clock-rates = <48000000>;
+ dmas = <&edma1 50 0 0>, <&edma1 49 0 FSL_EDMA_RX>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -487,6 +502,8 @@ lpi2c7: i2c@29850000 {
assigned-clocks = <&pcc4 IMX8ULP_CLK_LPI2C7>;
assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
assigned-clock-rates = <48000000>;
+ dmas = <&edma1 52 0 0>, <&edma1 51 0 FSL_EDMA_RX>;
+ dma-names = "tx", "rx";
status = "disabled";
};
@@ -496,6 +513,8 @@ lpuart6: serial@29860000 {
interrupts = <GIC_SPI 101 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc4 IMX8ULP_CLK_LPUART6>;
clock-names = "ipg";
+ dmas = <&edma1 59 0 FSL_EDMA_RX>, <&edma1 60 0 0>;
+ dma-names = "rx", "tx";
status = "disabled";
};
@@ -505,6 +524,8 @@ lpuart7: serial@29870000 {
interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc4 IMX8ULP_CLK_LPUART7>;
clock-names = "ipg";
+ dmas = <&edma1 61 0 FSL_EDMA_RX>, <&edma1 62 0 0>;
+ dma-names = "rx", "tx";
status = "disabled";
};
@@ -516,7 +537,7 @@ sai4: sai@29880000 {
<&cgc1 IMX8ULP_CLK_SAI4_SEL>, <&cgc1 IMX8ULP_CLK_DUMMY>,
<&cgc1 IMX8ULP_CLK_DUMMY>;
clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3";
- dmas = <&edma1 67 0 1>, <&edma1 68 0 0>;
+ dmas = <&edma1 67 0 FSL_EDMA_RX>, <&edma1 68 0 0>;
dma-names = "rx", "tx";
#sound-dai-cells = <0>;
fsl,dataline = <0 0x03 0x03>;
@@ -531,7 +552,7 @@ sai5: sai@29890000 {
<&cgc1 IMX8ULP_CLK_SAI5_SEL>, <&cgc1 IMX8ULP_CLK_DUMMY>,
<&cgc1 IMX8ULP_CLK_DUMMY>;
clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3";
- dmas = <&edma1 69 0 1>, <&edma1 70 0 0>;
+ dmas = <&edma1 69 0 FSL_EDMA_RX>, <&edma1 70 0 0>;
dma-names = "rx", "tx";
#sound-dai-cells = <0>;
fsl,dataline = <0 0x0f 0x0f>;
@@ -814,7 +835,7 @@ sai6: sai@2da90000 {
<&cgc2 IMX8ULP_CLK_SAI6_SEL>, <&cgc1 IMX8ULP_CLK_DUMMY>,
<&cgc1 IMX8ULP_CLK_DUMMY>;
clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3";
- dmas = <&edma2 71 0 1>, <&edma2 72 0 0>;
+ dmas = <&edma2 71 0 FSL_EDMA_RX>, <&edma2 72 0 0>;
dma-names = "rx", "tx";
#sound-dai-cells = <0>;
fsl,dataline = <0 0x0f 0x0f>;
@@ -829,7 +850,7 @@ sai7: sai@2daa0000 {
<&cgc2 IMX8ULP_CLK_SAI7_SEL>, <&cgc1 IMX8ULP_CLK_DUMMY>,
<&cgc1 IMX8ULP_CLK_DUMMY>;
clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3";
- dmas = <&edma2 73 0 1>, <&edma2 74 0 0>;
+ dmas = <&edma2 73 0 FSL_EDMA_RX>, <&edma2 74 0 0>;
dma-names = "rx", "tx";
#sound-dai-cells = <0>;
fsl,dataline = <0 0x0f 0x0f>;
@@ -855,7 +876,8 @@ spdif: spdif@2dab0000 {
"rxtx3", "rxtx4",
"rxtx5", "rxtx6",
"rxtx7", "spba";
- dmas = <&edma2 75 0 5>, <&edma2 76 0 4>;
+ dmas = <&edma2 75 0 (FSL_EDMA_RX | FSL_EDMA_MULTI_FIFO)>,
+ <&edma2 76 0 FSL_EDMA_MULTI_FIFO>;
dma-names = "rx", "tx";
status = "disabled";
};
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH 2/9] arm64: dts: imx8ulp: Add I3C2 controller node
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
2026-07-15 12:33 ` [PATCH 1/9] arm64: dts: imx8ulp: Add DMA channel properties and use eDMA flag macros Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
2026-07-15 12:42 ` sashiko-bot
2026-07-15 12:33 ` [PATCH 3/9] arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2 Peng Fan (OSS)
` (6 subsequent siblings)
8 siblings, 1 reply; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Add the I3C2 (i3c-master@29360000) node to the imx8ulp device tree
using the Silvaco I3C master IP. Configure PCC3-gated clock with SOSC
DIV2 as parent at 24 MHz for pclk, fast_clk and slow_clk.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
index 5438958176985..c82c3388cbe79 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
@@ -361,6 +361,22 @@ tpm5: tpm@29340000 {
status = "disabled";
};
+ i3c2: i3c@29360000 {
+ compatible = "silvaco,i3c-master-v1";
+ reg = <0x29360000 0x10000>;
+ interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
+ #address-cells = <3>;
+ #size-cells = <0>;
+ clocks = <&pcc3 IMX8ULP_CLK_I3C2>,
+ <&pcc3 IMX8ULP_CLK_I3C2>,
+ <&cgc1 IMX8ULP_CLK_DUMMY>;
+ clock-names = "pclk", "fast_clk", "slow_clk";
+ assigned-clocks = <&pcc3 IMX8ULP_CLK_I3C2>;
+ assigned-clock-parents = <&cgc1 IMX8ULP_CLK_SOSC_DIV2>;
+ assigned-clock-rates = <24000000>;
+ status = "disabled";
+ };
+
lpi2c4: i2c@29370000 {
compatible = "fsl,imx8ulp-lpi2c", "fsl,imx7ulp-lpi2c";
reg = <0x29370000 0x10000>;
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [PATCH 2/9] arm64: dts: imx8ulp: Add I3C2 controller node
2026-07-15 12:33 ` [PATCH 2/9] arm64: dts: imx8ulp: Add I3C2 controller node Peng Fan (OSS)
@ 2026-07-15 12:42 ` sashiko-bot
2026-07-15 15:12 ` Frank Li
0 siblings, 1 reply; 16+ messages in thread
From: sashiko-bot @ 2026-07-15 12:42 UTC (permalink / raw)
To: Peng Fan (OSS); +Cc: conor+dt, imx, robh, Frank.Li, devicetree
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The device tree assigns a dummy clock for 'slow_clk', contradicting the commit message which states it is configured to use the 24 MHz PCC3-gated clock.
--
--- Patch [2]: [PATCH 2/9] arm64: dts: imx8ulp: Add I3C2 controller node ---
commit 56028cef3837517bbf3f1ac9d58c6983788955ee
Author: Peng Fan <peng.fan@nxp.com>
arm64: dts: imx8ulp: Add I3C2 controller node
This commit adds the I3C2 node to the imx8ulp device tree using the
Silvaco I3C master IP. It also configures the controller's interrupts
and clock sources.
> diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> index 5438958176985..c82c3388cbe79 100644
> --- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
[ ... ]
> @@ -361,6 +361,22 @@ tpm5: tpm@29340000 {
[ ... ]
> + i3c2: i3c@29360000 {
> + compatible = "silvaco,i3c-master-v1";
> + reg = <0x29360000 0x10000>;
> + interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
> + #address-cells = <3>;
> + #size-cells = <0>;
> + clocks = <&pcc3 IMX8ULP_CLK_I3C2>,
> + <&pcc3 IMX8ULP_CLK_I3C2>,
> + <&cgc1 IMX8ULP_CLK_DUMMY>;
> + clock-names = "pclk", "fast_clk", "slow_clk";
[Severity: Medium]
Does this clock assignment match the intention described in the commit message?
The commit message states that the PCC3-gated clock is configured "for pclk,
fast_clk and slow_clk". However, the device tree node assigns the dummy clock
(&cgc1 IMX8ULP_CLK_DUMMY) for slow_clk instead.
Should slow_clk be using &pcc3 IMX8ULP_CLK_I3C2 as described, or should the
commit message be updated to reflect the use of the dummy clock?
> + assigned-clocks = <&pcc3 IMX8ULP_CLK_I3C2>;
> + assigned-clock-parents = <&cgc1 IMX8ULP_CLK_SOSC_DIV2>;
> + assigned-clock-rates = <24000000>;
> + status = "disabled";
> + };
[ ... ]
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260715-imx8ulp-dts-v1-0-19651358b599@nxp.com?part=2
^ permalink raw reply [flat|nested] 16+ messages in thread* Re: [PATCH 2/9] arm64: dts: imx8ulp: Add I3C2 controller node
2026-07-15 12:42 ` sashiko-bot
@ 2026-07-15 15:12 ` Frank Li
0 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2026-07-15 15:12 UTC (permalink / raw)
To: sashiko-reviews; +Cc: Peng Fan (OSS), conor+dt, imx, robh, Frank.Li, devicetree
On Wed, Jul 15, 2026 at 12:42:07PM +0000, sashiko-bot@kernel.org wrote:
> Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> - [Medium] The device tree assigns a dummy clock for 'slow_clk', contradicting the commit message which states it is configured to use the 24 MHz PCC3-gated clock.
> --
>
> --- Patch [2]: [PATCH 2/9] arm64: dts: imx8ulp: Add I3C2 controller node ---
> commit 56028cef3837517bbf3f1ac9d58c6983788955ee
> Author: Peng Fan <peng.fan@nxp.com>
>
> arm64: dts: imx8ulp: Add I3C2 controller node
>
> This commit adds the I3C2 node to the imx8ulp device tree using the
> Silvaco I3C master IP. It also configures the controller's interrupts
> and clock sources.
>
> > diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> > index 5438958176985..c82c3388cbe79 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> [ ... ]
> > @@ -361,6 +361,22 @@ tpm5: tpm@29340000 {
> [ ... ]
> > + i3c2: i3c@29360000 {
> > + compatible = "silvaco,i3c-master-v1";
> > + reg = <0x29360000 0x10000>;
> > + interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
> > + #address-cells = <3>;
> > + #size-cells = <0>;
> > + clocks = <&pcc3 IMX8ULP_CLK_I3C2>,
> > + <&pcc3 IMX8ULP_CLK_I3C2>,
> > + <&cgc1 IMX8ULP_CLK_DUMMY>;
> > + clock-names = "pclk", "fast_clk", "slow_clk";
>
> [Severity: Medium]
> Does this clock assignment match the intention described in the commit message?
>
> The commit message states that the PCC3-gated clock is configured "for pclk,
> fast_clk and slow_clk". However, the device tree node assigns the dummy clock
> (&cgc1 IMX8ULP_CLK_DUMMY) for slow_clk instead.
>
> Should slow_clk be using &pcc3 IMX8ULP_CLK_I3C2 as described, or should the
> commit message be updated to reflect the use of the dummy clock?
Make sense, update commit messge or set slow_clk to &pcc3 IMX8ULP_CLK_I3C2
Frank
>
> > + assigned-clocks = <&pcc3 IMX8ULP_CLK_I3C2>;
> > + assigned-clock-parents = <&cgc1 IMX8ULP_CLK_SOSC_DIV2>;
> > + assigned-clock-rates = <24000000>;
> > + status = "disabled";
> > + };
> [ ... ]
>
> --
> Sashiko AI review · https://sashiko.dev/#/patchset/20260715-imx8ulp-dts-v1-0-19651358b599@nxp.com?part=2
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 3/9] arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
2026-07-15 12:33 ` [PATCH 1/9] arm64: dts: imx8ulp: Add DMA channel properties and use eDMA flag macros Peng Fan (OSS)
2026-07-15 12:33 ` [PATCH 2/9] arm64: dts: imx8ulp: Add I3C2 controller node Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
2026-07-15 12:47 ` sashiko-bot
2026-07-15 12:33 ` [PATCH 4/9] arm64: dts: imx8ulp: Add assigned clock properties for LPUART nodes Peng Fan (OSS)
` (5 subsequent siblings)
8 siblings, 1 reply; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
The SSRAM P2 partition starts at 0x22010000 with a size of 64 KB.
The previous node only mapped the last 4 KB (0x2201f000, 0x1000) which
was used for the SCMI shared memory buffer. Update the base address and
size to cover the full P2 region so it can also serve as a general
purpose SRAM pool.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
index c82c3388cbe79..173d11dea5d4f 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
@@ -154,9 +154,9 @@ sosc: clock-sosc {
#clock-cells = <0>;
};
- sram@2201f000 {
+ sram@22010000 {
compatible = "mmio-sram";
- reg = <0x0 0x2201f000 0x0 0x1000>;
+ reg = <0x0 0x22010000 0x0 0x10000>;
#address-cells = <1>;
#size-cells = <1>;
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [PATCH 3/9] arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2
2026-07-15 12:33 ` [PATCH 3/9] arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2 Peng Fan (OSS)
@ 2026-07-15 12:47 ` sashiko-bot
2026-07-15 15:14 ` Frank Li
0 siblings, 1 reply; 16+ messages in thread
From: sashiko-bot @ 2026-07-15 12:47 UTC (permalink / raw)
To: Peng Fan (OSS); +Cc: robh, Frank.Li, imx, conor+dt, devicetree
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The `ranges` property of the `sram` node and the `scmi_buf` child node were not updated to match the newly expanded SRAM `reg` region.
--
--- Patch [3]: [PATCH 3/9] arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2 ---
commit e7f90a5991db01135d232289fb75ec4528de4fca
Author: Peng Fan <peng.fan@nxp.com>
arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2
The SSRAM P2 partition starts at 0x22010000 with a size of 64 KB.
The previous node only mapped the last 4 KB (0x2201f000, 0x1000) which
was used for the SCMI shared memory buffer. Update the base address and
size to cover the full P2 region so it can also serve as a general
purpose SRAM pool.
> diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> --- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> @@ -154,9 +154,9 @@ sosc: clock-sosc {
> #clock-cells = <0>;
> };
>
> - sram@2201f000 {
> + sram@22010000 {
> compatible = "mmio-sram";
> - reg = <0x0 0x2201f000 0x0 0x1000>;
> + reg = <0x0 0x22010000 0x0 0x10000>;
>
> #address-cells = <1>;
> #size-cells = <1>;
> ranges = <0 0x0 0x2201f000 0x1000>;
>
> scmi_buf: scmi-sram-section@0 {
> compatible = "arm,scmi-shmem";
> reg = <0x0 0x400>;
> };
[Severity: Medium]
Since the reg property was expanded to cover the full 64KB region starting
at 0x22010000, should the ranges property and the scmi_buf node also be
updated to match?
Right now, ranges still only maps the last 4KB (0x2201f000). Any new child
nodes attempting to use the lower 60KB (0x22010000 to 0x2201efff) as a
general-purpose pool will fail to resolve their addresses.
Also, if ranges is updated to map from the new 0x22010000 base, wouldn't
the scmi_buf child node offset need to be updated to 0xf000 to maintain
its previous absolute address?
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260715-imx8ulp-dts-v1-0-19651358b599@nxp.com?part=3
^ permalink raw reply [flat|nested] 16+ messages in thread* Re: [PATCH 3/9] arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2
2026-07-15 12:47 ` sashiko-bot
@ 2026-07-15 15:14 ` Frank Li
0 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2026-07-15 15:14 UTC (permalink / raw)
To: sashiko-reviews; +Cc: Peng Fan (OSS), robh, Frank.Li, imx, conor+dt, devicetree
On Wed, Jul 15, 2026 at 12:47:36PM +0000, sashiko-bot@kernel.org wrote:
> Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> - [Medium] The `ranges` property of the `sram` node and the `scmi_buf` child node were not updated to match the newly expanded SRAM `reg` region.
> --
>
> --- Patch [3]: [PATCH 3/9] arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2 ---
> commit e7f90a5991db01135d232289fb75ec4528de4fca
> Author: Peng Fan <peng.fan@nxp.com>
>
> arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2
>
> The SSRAM P2 partition starts at 0x22010000 with a size of 64 KB.
> The previous node only mapped the last 4 KB (0x2201f000, 0x1000) which
> was used for the SCMI shared memory buffer. Update the base address and
> size to cover the full P2 region so it can also serve as a general
> purpose SRAM pool.
>
> > diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> > --- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> > @@ -154,9 +154,9 @@ sosc: clock-sosc {
> > #clock-cells = <0>;
> > };
> >
> > - sram@2201f000 {
> > + sram@22010000 {
> > compatible = "mmio-sram";
> > - reg = <0x0 0x2201f000 0x0 0x1000>;
> > + reg = <0x0 0x22010000 0x0 0x10000>;
> >
> > #address-cells = <1>;
> > #size-cells = <1>;
> > ranges = <0 0x0 0x2201f000 0x1000>;
> >
> > scmi_buf: scmi-sram-section@0 {
> > compatible = "arm,scmi-shmem";
> > reg = <0x0 0x400>;
> > };
>
> [Severity: Medium]
> Since the reg property was expanded to cover the full 64KB region starting
> at 0x22010000, should the ranges property and the scmi_buf node also be
> updated to match?
>
> Right now, ranges still only maps the last 4KB (0x2201f000). Any new child
> nodes attempting to use the lower 60KB (0x22010000 to 0x2201efff) as a
> general-purpose pool will fail to resolve their addresses.
>
> Also, if ranges is updated to map from the new 0x22010000 base, wouldn't
> the scmi_buf child node offset need to be updated to 0xf000 to maintain
> its previous absolute address?
Is it true? look like make sense?
Frank
>
> --
> Sashiko AI review · https://sashiko.dev/#/patchset/20260715-imx8ulp-dts-v1-0-19651358b599@nxp.com?part=3
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 4/9] arm64: dts: imx8ulp: Add assigned clock properties for LPUART nodes
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
` (2 preceding siblings ...)
2026-07-15 12:33 ` [PATCH 3/9] arm64: dts: imx8ulp: Correct SRAM node address and size to cover full SSRAM P2 Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
2026-07-15 12:33 ` [PATCH 5/9] arm64: dts: imx8ulp-evk: Add gpio-keys node for power button Peng Fan (OSS)
` (4 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Add assigned-clocks, assigned-clock-parents and assigned-clock-rates
properties to lpuart4 through lpuart7, so the LPUART input clock is
configured automatically by the clock framework during probe.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
index 173d11dea5d4f..80e08ecb38490 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
@@ -413,6 +413,9 @@ lpuart4: serial@29390000 {
interrupts = <GIC_SPI 99 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc3 IMX8ULP_CLK_LPUART4>;
clock-names = "ipg";
+ assigned-clocks = <&pcc3 IMX8ULP_CLK_LPUART4>;
+ assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
+ assigned-clock-rates = <48000000>;
dmas = <&edma1 55 0 FSL_EDMA_RX>, <&edma1 56 0 0> ;
dma-names = "rx", "tx";
status = "disabled";
@@ -424,6 +427,9 @@ lpuart5: serial@293a0000 {
interrupts = <GIC_SPI 100 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc3 IMX8ULP_CLK_LPUART5>;
clock-names = "ipg";
+ assigned-clocks = <&pcc3 IMX8ULP_CLK_LPUART5>;
+ assigned-clock-parents = <&cgc1 IMX8ULP_CLK_SOSC_DIV2>;
+ assigned-clock-rates = <24000000>;
dmas = <&edma1 57 0 FSL_EDMA_RX>, <&edma1 58 0 0> ;
dma-names = "rx", "tx";
status = "disabled";
@@ -529,6 +535,9 @@ lpuart6: serial@29860000 {
interrupts = <GIC_SPI 101 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc4 IMX8ULP_CLK_LPUART6>;
clock-names = "ipg";
+ assigned-clocks = <&pcc4 IMX8ULP_CLK_LPUART6>;
+ assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
+ assigned-clock-rates = <48000000>;
dmas = <&edma1 59 0 FSL_EDMA_RX>, <&edma1 60 0 0>;
dma-names = "rx", "tx";
status = "disabled";
@@ -540,6 +549,9 @@ lpuart7: serial@29870000 {
interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc4 IMX8ULP_CLK_LPUART7>;
clock-names = "ipg";
+ assigned-clocks = <&pcc4 IMX8ULP_CLK_LPUART7>;
+ assigned-clock-parents = <&cgc1 IMX8ULP_CLK_FROSC_DIV2>;
+ assigned-clock-rates = <48000000>;
dmas = <&edma1 61 0 FSL_EDMA_RX>, <&edma1 62 0 0>;
dma-names = "rx", "tx";
status = "disabled";
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH 5/9] arm64: dts: imx8ulp-evk: Add gpio-keys node for power button
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
` (3 preceding siblings ...)
2026-07-15 12:33 ` [PATCH 4/9] arm64: dts: imx8ulp: Add assigned clock properties for LPUART nodes Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
2026-07-15 12:33 ` [PATCH 6/9] arm64: dts: imx8ulp-evk: Enable LPUART6 for Bluetooth Peng Fan (OSS)
` (3 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Add a gpio-keys node for the power-on button, report it as KEY_POWER to
user space.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts | 18 ++++++++++++++++++
arch/arm64/boot/dts/freescale/imx8ulp-evk.dts | 19 +++++++++++++++++++
2 files changed, 37 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
index 5497e3d78136f..af8f0d6087f71 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
@@ -10,6 +10,18 @@
/ {
model = "NXP i.MX8ULP EVK9";
compatible = "fsl,imx8ulp-9x9-evk", "fsl,imx8ulp";
+
+ gpio-keys {
+ compatible = "gpio-keys";
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_gpio_keys>;
+
+ button-power-on {
+ label = "PowerOn";
+ gpios = <&gpiof 11 GPIO_ACTIVE_LOW>;
+ linux,code = <KEY_POWER>;
+ };
+ };
};
&btcpu {
@@ -43,6 +55,12 @@ MX8ULP_PAD_PTF10__ENET0_1588_CLKIN 0x43
>;
};
+&pinctrl_gpio_keys {
+ fsl,pins = <
+ MX8ULP_PAD_PTF11__PTF11 0x3
+ >;
+};
+
&pinctrl_usb1 {
fsl,pins = <
MX8ULP_PAD_PTE16__USB0_ID 0x10003
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
index 5dea66c1e7aa0..6a92c5ca9a412 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
@@ -5,6 +5,7 @@
/dts-v1/;
+#include <dt-bindings/input/input.h>
#include "imx8ulp.dtsi"
/ {
@@ -20,6 +21,18 @@ chosen {
stdout-path = &lpuart5;
};
+ gpio-keys {
+ compatible = "gpio-keys";
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_gpio_keys>;
+
+ button-power-on {
+ label = "PowerOn";
+ gpios = <&gpiof 31 GPIO_ACTIVE_LOW>;
+ linux,code = <KEY_POWER>;
+ };
+ };
+
memory@80000000 {
device_type = "memory";
reg = <0x0 0x80000000 0 0x80000000>;
@@ -323,6 +336,12 @@ MX8ULP_PAD_PTD22__FLEXSPI2_A_DATA4 0x42
>;
};
+ pinctrl_gpio_keys: gpiokeysgrp {
+ fsl,pins = <
+ MX8ULP_PAD_PTF31__PTF31 0x3
+ >;
+ };
+
pinctrl_lpuart5: lpuart5grp {
fsl,pins = <
MX8ULP_PAD_PTF14__LPUART5_TX 0x3
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH 6/9] arm64: dts: imx8ulp-evk: Enable LPUART6 for Bluetooth
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
` (4 preceding siblings ...)
2026-07-15 12:33 ` [PATCH 5/9] arm64: dts: imx8ulp-evk: Add gpio-keys node for power button Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
2026-07-15 15:17 ` Frank Li
2026-07-15 12:33 ` [PATCH 7/9] arm64: dts: imx8ulp-evk: Enable LPUART7 Peng Fan (OSS)
` (2 subsequent siblings)
8 siblings, 1 reply; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan,
Sherry Sun
From: Sherry Sun <sherry.sun@nxp.com>
Enable lpuart6 with RTS/CTS flow control on the i.MX 8ULP EVK board
and add an NXP 88W8987 Bluetooth companion node to pair the UART
transport with the wireless chip.
i.MX8ULP 9X9 EVK use different lpuart6 pins which used for M.2 bluetooth,
also use the sai6 instead of sai5 for PCM interface on M.2, so need to
update the pin setting in dts.
Signed-off-by: Sherry Sun <sherry.sun@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts | 17 +++++++++++++++++
arch/arm64/boot/dts/freescale/imx8ulp-evk.dts | 21 +++++++++++++++++++++
2 files changed, 38 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
index af8f0d6087f71..71c66c58edc4d 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
@@ -75,6 +75,23 @@ MX8ULP_PAD_PTE20__USB1_OC 0x10003
>;
};
+&pinctrl_lpuart6 {
+ fsl,pins = <
+ MX8ULP_PAD_PTF18__LPUART6_TX 0x3
+ MX8ULP_PAD_PTF19__LPUART6_RX 0x3
+ MX8ULP_PAD_PTF16__LPUART6_CTS_B 0x3
+ MX8ULP_PAD_PTF17__LPUART6_RTS_B 0x3
+ >;
+};
+
+&lpspi5 { /* conflict with lpuart6 PAD_PTF16-19 */
+ status = "disabled";
+};
+
+&sai5 {
+ status = "disabled";
+};
+
&sai6 {
pinctrl-names = "default", "sleep";
pinctrl-0 = <&pinctrl_sai6>;
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
index 6a92c5ca9a412..66e99cd24040f 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
@@ -167,6 +167,18 @@ &lpuart5 {
status = "okay";
};
+&lpuart6 {
+ /* BT */
+ pinctrl-names = "default", "sleep";
+ pinctrl-0 = <&pinctrl_lpuart6>;
+ pinctrl-1 = <&pinctrl_lpuart6>;
+ status = "okay";
+
+ bluetooth {
+ compatible = "nxp,88w8987-bt";
+ };
+};
+
&lpi2c7 {
#address-cells = <1>;
#size-cells = <0>;
@@ -349,6 +361,15 @@ MX8ULP_PAD_PTF15__LPUART5_RX 0x3
>;
};
+ pinctrl_lpuart6: lpuart6grp {
+ fsl,pins = <
+ MX8ULP_PAD_PTE10__LPUART6_TX 0x3
+ MX8ULP_PAD_PTE11__LPUART6_RX 0x3
+ MX8ULP_PAD_PTE9__LPUART6_RTS_B 0x3
+ MX8ULP_PAD_PTE8__LPUART6_CTS_B 0x3
+ >;
+ };
+
pinctrl_lpi2c7: lpi2c7grp {
fsl,pins = <
MX8ULP_PAD_PTE12__LPI2C7_SCL 0x20
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [PATCH 6/9] arm64: dts: imx8ulp-evk: Enable LPUART6 for Bluetooth
2026-07-15 12:33 ` [PATCH 6/9] arm64: dts: imx8ulp-evk: Enable LPUART6 for Bluetooth Peng Fan (OSS)
@ 2026-07-15 15:17 ` Frank Li
0 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2026-07-15 15:17 UTC (permalink / raw)
To: Peng Fan (OSS)
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, linux-kernel, Peng Fan, Sherry Sun
On Wed, Jul 15, 2026 at 08:33:07PM +0800, Peng Fan (OSS) wrote:
> From: Sherry Sun <sherry.sun@nxp.com>
>
> Enable lpuart6 with RTS/CTS flow control on the i.MX 8ULP EVK board
> and add an NXP 88W8987 Bluetooth companion node to pair the UART
> transport with the wireless chip.
>
> i.MX8ULP 9X9 EVK use different lpuart6 pins which used for M.2 bluetooth,
> also use the sai6 instead of sai5 for PCM interface on M.2, so need to
> update the pin setting in dts.
>
> Signed-off-by: Sherry Sun <sherry.sun@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts | 17 +++++++++++++++++
> arch/arm64/boot/dts/freescale/imx8ulp-evk.dts | 21 +++++++++++++++++++++
> 2 files changed, 38 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
> index af8f0d6087f71..71c66c58edc4d 100644
> --- a/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
> @@ -75,6 +75,23 @@ MX8ULP_PAD_PTE20__USB1_OC 0x10003
> >;
> };
>
> +&pinctrl_lpuart6 {
> + fsl,pins = <
> + MX8ULP_PAD_PTF18__LPUART6_TX 0x3
> + MX8ULP_PAD_PTF19__LPUART6_RX 0x3
> + MX8ULP_PAD_PTF16__LPUART6_CTS_B 0x3
> + MX8ULP_PAD_PTF17__LPUART6_RTS_B 0x3
> + >;
> +};
> +
> +&lpspi5 { /* conflict with lpuart6 PAD_PTF16-19 */
> + status = "disabled";
> +};
> +
> +&sai5 {
> + status = "disabled";
> +};
> +
> &sai6 {
> pinctrl-names = "default", "sleep";
> pinctrl-0 = <&pinctrl_sai6>;
> diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> index 6a92c5ca9a412..66e99cd24040f 100644
> --- a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> @@ -167,6 +167,18 @@ &lpuart5 {
> status = "okay";
> };
>
> +&lpuart6 {
> + /* BT */
> + pinctrl-names = "default", "sleep";
> + pinctrl-0 = <&pinctrl_lpuart6>;
> + pinctrl-1 = <&pinctrl_lpuart6>;
sleep is the same as default state, are you sure need both?
Frank
> + status = "okay";
> +
> + bluetooth {
> + compatible = "nxp,88w8987-bt";
> + };
> +};
> +
> &lpi2c7 {
> #address-cells = <1>;
> #size-cells = <0>;
> @@ -349,6 +361,15 @@ MX8ULP_PAD_PTF15__LPUART5_RX 0x3
> >;
> };
>
> + pinctrl_lpuart6: lpuart6grp {
> + fsl,pins = <
> + MX8ULP_PAD_PTE10__LPUART6_TX 0x3
> + MX8ULP_PAD_PTE11__LPUART6_RX 0x3
> + MX8ULP_PAD_PTE9__LPUART6_RTS_B 0x3
> + MX8ULP_PAD_PTE8__LPUART6_CTS_B 0x3
> + >;
> + };
> +
> pinctrl_lpi2c7: lpi2c7grp {
> fsl,pins = <
> MX8ULP_PAD_PTE12__LPI2C7_SCL 0x20
>
> --
> 2.34.1
>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 7/9] arm64: dts: imx8ulp-evk: Enable LPUART7
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
` (5 preceding siblings ...)
2026-07-15 12:33 ` [PATCH 6/9] arm64: dts: imx8ulp-evk: Enable LPUART6 for Bluetooth Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
2026-07-15 15:18 ` Frank Li
2026-07-15 12:33 ` [PATCH 8/9] arm64: dts: imx8ulp-evk: Change the values of some PCRs of ENET Peng Fan (OSS)
2026-07-15 12:33 ` [PATCH 9/9] arm64: dts: imx8ulp-9x9-evk: Rename model string to reflect die size Peng Fan (OSS)
8 siblings, 1 reply; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan,
Sherry Sun
From: Sherry Sun <sherry.sun@nxp.com>
The i.MX 8ULP EVK proto1B revision routes LPUART7 TX/RX to the FT4232
PortD connector (J25/J26 pins 2-3). Enable the port by default so the
additional serial console is available out of the box.
Signed-off-by: Sherry Sun <sherry.sun@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp-evk.dts | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
index 66e99cd24040f..938ead54ff50c 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
@@ -179,6 +179,14 @@ bluetooth {
};
};
+&lpuart7 {
+ /* FT4232 PortD: need to connect J25/J26 2-3 */
+ pinctrl-names = "default", "sleep";
+ pinctrl-0 = <&pinctrl_lpuart7>;
+ pinctrl-1 = <&pinctrl_lpuart7>;
+ status = "okay";
+};
+
&lpi2c7 {
#address-cells = <1>;
#size-cells = <0>;
@@ -370,6 +378,13 @@ MX8ULP_PAD_PTE8__LPUART6_CTS_B 0x3
>;
};
+ pinctrl_lpuart7: lpuart7grp {
+ fsl,pins = <
+ MX8ULP_PAD_PTF22__LPUART7_TX 0x3
+ MX8ULP_PAD_PTF23__LPUART7_RX 0x3
+ >;
+ };
+
pinctrl_lpi2c7: lpi2c7grp {
fsl,pins = <
MX8ULP_PAD_PTE12__LPI2C7_SCL 0x20
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [PATCH 7/9] arm64: dts: imx8ulp-evk: Enable LPUART7
2026-07-15 12:33 ` [PATCH 7/9] arm64: dts: imx8ulp-evk: Enable LPUART7 Peng Fan (OSS)
@ 2026-07-15 15:18 ` Frank Li
0 siblings, 0 replies; 16+ messages in thread
From: Frank Li @ 2026-07-15 15:18 UTC (permalink / raw)
To: Peng Fan (OSS)
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
imx, linux-arm-kernel, linux-kernel, Peng Fan, Sherry Sun
On Wed, Jul 15, 2026 at 08:33:08PM +0800, Peng Fan (OSS) wrote:
> From: Sherry Sun <sherry.sun@nxp.com>
>
> The i.MX 8ULP EVK proto1B revision routes LPUART7 TX/RX to the FT4232
> PortD connector (J25/J26 pins 2-3). Enable the port by default so the
> additional serial console is available out of the box.
>
> Signed-off-by: Sherry Sun <sherry.sun@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> arch/arm64/boot/dts/freescale/imx8ulp-evk.dts | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> index 66e99cd24040f..938ead54ff50c 100644
> --- a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> @@ -179,6 +179,14 @@ bluetooth {
> };
> };
>
> +&lpuart7 {
> + /* FT4232 PortD: need to connect J25/J26 2-3 */
> + pinctrl-names = "default", "sleep";
> + pinctrl-0 = <&pinctrl_lpuart7>;
> + pinctrl-1 = <&pinctrl_lpuart7>;
The same question here?
why need sleep state
Frank
> + status = "okay";
> +};
> +
> &lpi2c7 {
> #address-cells = <1>;
> #size-cells = <0>;
> @@ -370,6 +378,13 @@ MX8ULP_PAD_PTE8__LPUART6_CTS_B 0x3
> >;
> };
>
> + pinctrl_lpuart7: lpuart7grp {
> + fsl,pins = <
> + MX8ULP_PAD_PTF22__LPUART7_TX 0x3
> + MX8ULP_PAD_PTF23__LPUART7_RX 0x3
> + >;
> + };
> +
> pinctrl_lpi2c7: lpi2c7grp {
> fsl,pins = <
> MX8ULP_PAD_PTE12__LPI2C7_SCL 0x20
>
> --
> 2.34.1
>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 8/9] arm64: dts: imx8ulp-evk: Change the values of some PCRs of ENET
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
` (6 preceding siblings ...)
2026-07-15 12:33 ` [PATCH 7/9] arm64: dts: imx8ulp-evk: Enable LPUART7 Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
2026-07-15 12:33 ` [PATCH 9/9] arm64: dts: imx8ulp-9x9-evk: Rename model string to reflect die size Peng Fan (OSS)
8 siblings, 0 replies; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan,
Wei Fang
From: Wei Fang <wei.fang@nxp.com>
Change to use standard drive strength and slow slew rate on
PTE14-17, 22, 23, otherwise the overshoot of ENET is too high.
Signed-off-by: Wei Fang <wei.fang@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp-evk.dts | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
index 938ead54ff50c..81adc8a6f34cd 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
@@ -325,15 +325,15 @@ &mu {
&iomuxc1 {
pinctrl_enet: enetgrp {
fsl,pins = <
- MX8ULP_PAD_PTE15__ENET0_MDC 0x43
- MX8ULP_PAD_PTE14__ENET0_MDIO 0x43
- MX8ULP_PAD_PTE17__ENET0_RXER 0x43
+ MX8ULP_PAD_PTE15__ENET0_MDC 0x7
+ MX8ULP_PAD_PTE14__ENET0_MDIO 0x7
+ MX8ULP_PAD_PTE17__ENET0_RXER 0x7
MX8ULP_PAD_PTE18__ENET0_CRS_DV 0x43
MX8ULP_PAD_PTF1__ENET0_RXD0 0x43
MX8ULP_PAD_PTE20__ENET0_RXD1 0x43
- MX8ULP_PAD_PTE16__ENET0_TXEN 0x43
- MX8ULP_PAD_PTE23__ENET0_TXD0 0x43
- MX8ULP_PAD_PTE22__ENET0_TXD1 0x43
+ MX8ULP_PAD_PTE16__ENET0_TXEN 0x7
+ MX8ULP_PAD_PTE23__ENET0_TXD0 0x7
+ MX8ULP_PAD_PTE22__ENET0_TXD1 0x7
MX8ULP_PAD_PTE19__ENET0_REFCLK 0x43
MX8ULP_PAD_PTF10__ENET0_1588_CLKIN 0x43
>;
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH 9/9] arm64: dts: imx8ulp-9x9-evk: Rename model string to reflect die size
2026-07-15 12:33 [PATCH 0/9] arm64: dts: imx8ulp: Device tree updates for i.MX 8ULP EVK boards Peng Fan (OSS)
` (7 preceding siblings ...)
2026-07-15 12:33 ` [PATCH 8/9] arm64: dts: imx8ulp-evk: Change the values of some PCRs of ENET Peng Fan (OSS)
@ 2026-07-15 12:33 ` Peng Fan (OSS)
8 siblings, 0 replies; 16+ messages in thread
From: Peng Fan (OSS) @ 2026-07-15 12:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: devicetree, imx, linux-arm-kernel, linux-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Rename the board model from "NXP i.MX8ULP EVK9" to "NXP i.MX8ULP 9x9
EVK" to follow the i.MX naming convention where the die size (9x9 mm)
is used instead of an opaque board revision suffix.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
index 71c66c58edc4d..e6bc6e6555f57 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8ulp-9x9-evk.dts
@@ -8,7 +8,7 @@
#include "imx8ulp-evk.dts"
/ {
- model = "NXP i.MX8ULP EVK9";
+ model = "NXP i.MX8ULP 9x9 EVK";
compatible = "fsl,imx8ulp-9x9-evk", "fsl,imx8ulp";
gpio-keys {
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread