public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Peng Fan <peng.fan@oss.nxp.com>
To: alice.guo@oss.nxp.com
Cc: "NXP i.MX U-Boot Team" <uboot-imx@nxp.com>,
	u-boot@lists.denx.de, "Stefano Babic" <sbabic@nabladev.com>,
	"Fabio Estevam" <festevam@gmail.com>,
	"Tom Rini" <trini@konsulko.com>, "Peng Fan" <peng.fan@nxp.com>,
	"Ye Li" <ye.li@nxp.com>, "Jindong Yue" <jindong.yue@nxp.com>,
	"Jacky Bai" <ping.bai@nxp.com>,
	"Marek Vasut" <marek.vasut+renesas@mailbox.org>,
	"Rasmus Villemoes" <ravi@prevas.dk>,
	"Fedor Ross" <fedor.ross@ifm.com>,
	"João Paulo Gonçalves" <joao.goncalves@toradex.com>,
	"Patrice Chotard" <patrice.chotard@foss.st.com>,
	"Valentin Caron" <valentin.caron@foss.st.com>,
	"Vinh Nguyen" <vinh.nguyen.xz@renesas.com>,
	"Sam Protsenko" <semen.protsenko@linaro.org>,
	"Ranjani Vaidyanathan" <Ranjani.Vaidyanathan@nxp.com>,
	"Ji Luo" <ji.luo@nxp.com>, "Frank Li" <Frank.Li@nxp.com>,
	"Sumit Garg" <sumit.garg@oss.qualcomm.com>,
	"Adam Ford" <aford173@gmail.com>,
	"Frieder Schrempf" <frieder.schrempf@kontron.de>,
	"David Zang" <davidzangcs@gmail.com>,
	"Andrew Goodbody" <andrew.goodbody@linaro.org>,
	"Sumit Garg" <sumit.garg@kernel.org>,
	"Marek Vasut" <marex@nabladev.com>,
	"Joseph Guo" <qijian.guo@nxp.com>,
	"Tim Harvey" <tharvey@gateworks.com>,
	"Alice Guo" <alice.guo@nxp.com>
Subject: Re: [PATCH v1 11/12] arm: dts: Add U-Boot device tree for i.MX952 EVK
Date: Fri, 6 Feb 2026 08:51:28 +0800	[thread overview]
Message-ID: <aYU7EMvw/l6E+Cdt@shlinux89> (raw)
In-Reply-To: <20260205-imx952-v1-11-2103d0222a0b@nxp.com>

On Thu, Feb 05, 2026 at 09:55:17PM +0800, alice.guo@oss.nxp.com wrote:
>From: Alice Guo <alice.guo@nxp.com>
>
>Add U-Boot specific device tree configuration for i.MX952 EVK board.
>
>Signed-off-by: Alice Guo <alice.guo@nxp.com>
>---
> arch/arm/dts/imx952-evk-u-boot.dtsi |  58 +++++++
> arch/arm/dts/imx952-u-boot.dtsi     | 327 ++++++++++++++++++++++++++++++++++++
> 2 files changed, 385 insertions(+)
>
>diff --git a/arch/arm/dts/imx952-evk-u-boot.dtsi b/arch/arm/dts/imx952-evk-u-boot.dtsi
>new file mode 100644
>index 00000000000..1b2c314a41b
>--- /dev/null
>+++ b/arch/arm/dts/imx952-evk-u-boot.dtsi
>@@ -0,0 +1,58 @@
>+// SPDX-License-Identifier: GPL-2.0+
>+/*
>+ * Copyright 2026 NXP
>+ */
>+
>+#include "imx952-u-boot.dtsi"
>+
>+&i2c3_pcal6408 {
>+	compatible = "ti,tca6408";
>+	label = "i2c3_io";
>+};
>+
>+&i2c4_pcal6408 {
>+	compatible = "ti,tca6408";
>+	label = "i2c4_io";
>+};
>+
>+&pcal6416 {
>+	compatible = "ti,tca6416";
>+	label = "i2c6_io";
>+};
>+
>+&pcal6524 {
>+	label = "i2c7_io";
>+};
>+
>+&usb1 {
>+	compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb";
>+	/delete-property/power-domains;
>+	bootph-pre-ram;
>+	extcon = <&ptn5110>;
>+};
>+
>+&usb2 {
>+	compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb";
>+	/delete-property/power-domains;
>+	bootph-pre-ram;
>+};
>+
>+&usbmisc1 {
>+	bootph-pre-ram;
>+};
>+
>+&usbmisc2 {
>+	bootph-pre-ram;
>+};
>+
>+&usbphynop1 {
>+	bootph-pre-ram;
>+};
>+
>+&usbphynop2 {
>+	bootph-pre-ram;
>+};
>+
>+&wdog3 {
>+	status = "disabled";
>+};
>diff --git a/arch/arm/dts/imx952-u-boot.dtsi b/arch/arm/dts/imx952-u-boot.dtsi
>new file mode 100644
>index 00000000000..208931ba884
>--- /dev/null
>+++ b/arch/arm/dts/imx952-u-boot.dtsi
>@@ -0,0 +1,327 @@
>+// SPDX-License-Identifier: GPL-2.0+
>+/*
>+ * Copyright 2026 NXP
>+ */
>+
>+/ {
>+	binman {
>+		multiple-images;
>+
>+		m33-oei-ddrfw {
>+			pad-byte = <0x00>;
>+			align-size = <0x8>;
>+			filename = "m33-oei-ddrfw.bin";
>+
>+			oei-m33-ddr {
>+				align-size = <0x4>;
>+				filename = "oei-m33-ddr.bin";
>+				type = "blob-ext";
>+			};
>+
>+			imx-lpddr {
>+				type = "nxp-header-ddrfw";
>+
>+				imx-lpddr-imem {
>+					filename = "lpddr4x_imem_v202409.bin";
>+					type = "blob-ext";
>+				};
>+
>+				imx-lpddr-dmem {
>+					filename = "lpddr4x_dmem_v202409.bin";
>+					type = "blob-ext";
>+				};
>+			};
>+
>+			imx-lpddr-qb {
>+				type = "nxp-header-ddrfw";
>+
>+				imx-lpddr-imem-qb {
>+					filename = "lpddr4x_imem_qb_v202409.bin";
>+					type = "blob-ext";
>+				};
>+
>+				imx-lpddr-dmem-qb {
>+					filename = "lpddr4x_dmem_qb_v202409.bin";
>+					type = "blob-ext";
>+				};
>+			};
>+		};
>+
>+		imx-boot {
>+			filename = "flash.bin";
>+			pad-byte = <0x00>;
>+
>+			spl {
>+				type = "nxp-imx9image";
>+				cfg-path = "spl/u-boot-spl.cfgout";
>+				args;
>+
>+				cntr-version = <2>;
>+				boot-from = "sd";
>+				soc-type = "IMX9";
>+				append = "mx952a0-ahab-container.img";
>+				container;
>+				dummy-ddr;
>+				image0 = "oei", "m33-oei-ddrfw.bin", "0x1ffc0000";
>+				hold = <0x10000>;
>+				image1 = "m33", "m33_image.bin", "0x1ffc0000";
>+				image2 = "a55", "spl/u-boot-spl.bin", "0x20480000";
>+				dummy-v2x = <0x8b000000>;
>+			};
>+
>+			u-boot {
>+				type = "nxp-imx9image";
>+				cfg-path = "u-boot-container.cfgout";
>+				args;
>+
>+				cntr-version = <2>;
>+				boot-from = "sd";
>+				soc-type = "IMX9";
>+				container;
>+				image0 = "a55", "bl31.bin", "0x8a200000";
>+				image1 = "a55", "u-boot.bin", "0x90200000";
>+			};
>+		};
>+	};
>+
>+	chosen {
>+		bootargs = "console=ttyLP0,115200 earlycon";
>+	};
>+
>+	aliases {
>+		usb_gadget0 = &usbg1;
>+		usb_gadget1 = &usbg2;
>+	};
>+
>+	usbg1: usbg1 {
>+		compatible = "fsl,imx27-usb-gadget";
>+		dr_mode = "peripheral";
>+		chipidea,usb = <&usb1>;
>+		bootph-pre-ram;
>+		status = "okay";
>+	};
>+
>+	usbg2: usbg2 {
>+		compatible = "fsl,imx27-usb-gadget";
>+		dr_mode = "peripheral";
>+		chipidea,usb = <&usb2>;
>+		bootph-pre-ram;
>+		status = "okay";
>+	};

The above two nodes does not have a working driver in upstream.

>+
>+	reg_m2_pwr: regulator-m2-pwr {
>+		compatible = "regulator-fixed";
>+		regulator-name = "M.2-power";
>+		regulator-min-microvolt = <3300000>;
>+		regulator-max-microvolt = <3300000>;
>+		gpio = <&pcal6524 20 GPIO_ACTIVE_HIGH>;
>+		enable-active-high;
>+		/*
>+		 * M.2 device only can be enabled(W_DISABLE1#) after all Power
>+		 * Rails reach their minimum operating voltage (PCI Express M.2
>+		 * Specification r5.1 3.1.4 Power-up Timing).
>+		 * Set a delay equal to the max value of Tsettle here.
>+		 */
>+		startup-delay-us = <5000>;
>+	};
>+};
>+
>+&A55_0 {
>+	clocks = <&scmi_clk IMX952_CLK_ARMPLL_PFD0>;
>+	/delete-property/ power-domains;
>+};
>+
>+&A55_1 {
>+	clocks = <&scmi_clk IMX952_CLK_ARMPLL_PFD0>;
>+	/delete-property/ power-domains;
>+};
>+
>+&A55_2 {
>+	clocks = <&scmi_clk IMX952_CLK_ARMPLL_PFD0>;
>+	/delete-property/ power-domains;
>+};
>+
>+&A55_3 {
>+	clocks = <&scmi_clk IMX952_CLK_ARMPLL_PFD0>;
>+	/delete-property/ power-domains;
>+};
>+
>+&aips1 {
>+	bootph-all;
>+};
>+
>+&aips2 {
>+	bootph-all;
>+};
>+
>+&aips3 {
>+	bootph-all;
>+};
>+
>+&clk_ext1 {
>+	bootph-all;
>+};
>+
>+&clk_dummy {
>+	bootph-all;
>+};
>+
>+&clk_osc_24m {
>+	bootph-all;
>+};
>+
>+&elemu1 {
>+	status = "okay";
>+	bootph-all;
>+};
>+
>+&elemu3 {
>+	status = "okay";
>+	bootph-all;
>+};
>+
>+&{/firmware} {
>+	bootph-all;
>+};
>+
>+&{/firmware/scmi} {
>+	bootph-all;
>+};
>+
>+&{/firmware/scmi/protocol@11} {
>+	bootph-all;
>+};
>+
>+&{/firmware/scmi/protocol@13} {
>+	bootph-all;
>+};
>+
>+&{/firmware/scmi/protocol@14} {
>+	bootph-all;
>+};
>+
>+&{/firmware/scmi/protocol@15} {
>+	bootph-all;
>+};
>+
>+&{/firmware/scmi/protocol@19} {
>+	bootph-all;
>+};
>+
>+&gpio1 {
>+	reg = <0 0x47400000 0 0x1000>, <0 0x47400040 0 0x40>;
>+};
>+
>+&gpio2 {
>+	reg = <0 0x43810000 0 0x1000>, <0 0x43810040 0 0x40>;
>+	bootph-pre-ram;
>+	/*
>+	 * Use one SPL/U-Boot for mx952evk and mx952evkrpmsg, since GPIO2
>+	 * is assigned to M7, disable gpio2 here
>+	 */
>+	status = "disabled";
>+};
>+
>+&gpio3 {
>+	reg = <0 0x43820000 0 0x1000>, <0 0x43820040 0 0x40>;
>+	bootph-pre-ram;
>+};
>+
>+&gpio4 {
>+	reg = <0 0x43840000 0 0x1000>, <0 0x43840040 0 0x40>;
>+	bootph-pre-ram;
>+};
>+
>+&gpio5 {
>+	reg = <0 0x43850000 0 0x1000>, <0 0x43850040 0 0x40>;
>+	bootph-pre-ram;
>+};
>+
>+&lpuart1 {
>+	bootph-pre-ram;
>+};
>+
>+&mu2 {
>+	bootph-all;
>+};
>+
>+&reg_usdhc2_vmmc {
>+	bootph-pre-ram;
>+};
>+
>+&scmi_buf0 {
>+	reg = <0x0 0x400>;
>+	bootph-all;
>+};
>+
>+&scmi_buf1 {
>+	bootph-all;
>+};
>+
>+&{/soc} {
>+	bootph-all;
>+};
>+
>+&sram0 {
>+	bootph-all;
>+};
>+
>+&usdhc1 {
>+	bootph-pre-ram;
>+};
>+
>+&usdhc2 {
>+	bootph-pre-ram;
>+};
>+
>+&scmi_iomuxc {
>+	pinctrl-names = "default";
>+	pinctrl-0 = <&pinctrl_hog>;
>+
>+	pinctrl_hog: hoggrp {
>+		bootph-pre-ram;
>+
>+		fsl,pins = <
>+			IMX952_PAD_XSPI1_SS1_B__WAKEUPMIX_TOP_GPIO5_IO_11	0x3fe
>+			IMX952_PAD_GPIO_IO12__WAKEUPMIX_TOP_TPM3_CH2		0x51e
>+			IMX952_PAD_CCM_CLKO1__WAKEUPMIX_TOP_GPIO3_IO_26		0x3fe
>+			IMX952_PAD_CCM_CLKO2__WAKEUPMIX_TOP_GPIO3_IO_27		0x3fe
>+		>;

Add a comment for the settings, why they needs to be set in hog.

>+	};
>+};
>+
>+&pinctrl_uart1 {
>+	bootph-pre-ram;
>+};
>+
>+&pinctrl_usdhc1 {
>+	bootph-pre-ram;
>+};
>+
>+&pinctrl_usdhc1_100mhz {
>+	bootph-pre-ram;
>+};
>+
>+&pinctrl_usdhc1_200mhz {
>+	bootph-pre-ram;
>+};
>+
>+&pinctrl_usdhc2 {
>+	bootph-pre-ram;
>+};
>+
>+&pinctrl_usdhc2_100mhz {
>+	bootph-pre-ram;
>+};
>+
>+&pinctrl_usdhc2_200mhz {
>+	bootph-pre-ram;
>+};
>+
>+&pinctrl_usdhc2_gpio {
>+	bootph-pre-ram;
>+};
>+
>+&pinctrl_reg_usdhc2_vmmc {
>+	bootph-pre-ram;
>+};

Regards
Peng
>
>-- 
>2.43.0
>

  reply	other threads:[~2026-02-06  0:49 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-05 13:55 [PATCH 00/12] imx: add i.MX952 support alice.guo
2026-02-05 13:55 ` [PATCH v1 01/12] imx9: scmi: Get DDR size through SM SCMI API alice.guo
2026-02-06  0:34   ` Peng Fan
2026-02-05 13:55 ` [PATCH v1 02/12] imx95/imx94: Remove board_phys_sdram_size from each board alice.guo
2026-02-06  0:35   ` Peng Fan
2026-02-05 13:55 ` [PATCH v1 03/12] pinctrl: nxp: Add i.MX952 support alice.guo
2026-02-06  0:36   ` Peng Fan
2026-02-05 13:55 ` [PATCH v1 04/12] arm: imx: Add i.MX952 CPU type support alice.guo
2026-02-06  0:37   ` Peng Fan
2026-02-05 13:55 ` [PATCH v1 05/12] imx: ele_ahab: Add i.MX952 support to display_life_cycle() alice.guo
2026-02-06  0:38   ` Peng Fan
2026-02-05 13:55 ` [PATCH v1 06/12] imx: container: Add i.MX952 support to get_imageset_end() alice.guo
2026-02-06  0:40   ` Peng Fan
2026-02-05 13:55 ` [PATCH v1 07/12] arm: imx952: Add watchdog base address definitions alice.guo
2026-02-06  0:41   ` Peng Fan
2026-02-06  3:59   ` Fabio Estevam
2026-02-09 10:24     ` 回复: " Alice Guo (OSS)
2026-02-09 11:21       ` Fabio Estevam
2026-02-05 13:55 ` [PATCH v1 08/12] arm: imx9: Add i.MX952 SoC support alice.guo
2026-02-06  0:45   ` Peng Fan
2026-02-22  1:34   ` David Zang
2026-02-05 13:55 ` [PATCH v1 09/12] cpu: imx952: Add i.MX952 support alice.guo
2026-02-06  0:46   ` Peng Fan
2026-02-05 13:55 ` [PATCH v1 10/12] dts: upstream: arm64: freescale: Sync i.MX952 device tree from Linux alice.guo
2026-02-05 15:38   ` Tom Rini
2026-02-09 10:34     ` 回复: " Alice Guo (OSS)
2026-02-09 14:03       ` Tom Rini
2026-02-05 13:55 ` [PATCH v1 11/12] arm: dts: Add U-Boot device tree for i.MX952 EVK alice.guo
2026-02-06  0:51   ` Peng Fan [this message]
2026-02-06  1:27     ` Tom Rini
2026-02-05 13:55 ` [PATCH v1 12/12] board: nxp: imx952_evk: Add i.MX952 15x15 lpddr4x board support alice.guo
2026-02-05 15:45   ` Tom Rini
2026-02-06  1:24   ` Peng Fan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aYU7EMvw/l6E+Cdt@shlinux89 \
    --to=peng.fan@oss.nxp.com \
    --cc=Frank.Li@nxp.com \
    --cc=Ranjani.Vaidyanathan@nxp.com \
    --cc=aford173@gmail.com \
    --cc=alice.guo@nxp.com \
    --cc=alice.guo@oss.nxp.com \
    --cc=andrew.goodbody@linaro.org \
    --cc=davidzangcs@gmail.com \
    --cc=fedor.ross@ifm.com \
    --cc=festevam@gmail.com \
    --cc=frieder.schrempf@kontron.de \
    --cc=ji.luo@nxp.com \
    --cc=jindong.yue@nxp.com \
    --cc=joao.goncalves@toradex.com \
    --cc=marek.vasut+renesas@mailbox.org \
    --cc=marex@nabladev.com \
    --cc=patrice.chotard@foss.st.com \
    --cc=peng.fan@nxp.com \
    --cc=ping.bai@nxp.com \
    --cc=qijian.guo@nxp.com \
    --cc=ravi@prevas.dk \
    --cc=sbabic@nabladev.com \
    --cc=semen.protsenko@linaro.org \
    --cc=sumit.garg@kernel.org \
    --cc=sumit.garg@oss.qualcomm.com \
    --cc=tharvey@gateworks.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=uboot-imx@nxp.com \
    --cc=valentin.caron@foss.st.com \
    --cc=vinh.nguyen.xz@renesas.com \
    --cc=ye.li@nxp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox