linux-kernel - Re: [PATCH v2 2/2] arm64: dts: freescale: add NXP FRDM-IMX91S board support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <aUJFC4ll033PzX7L@lsvm11u0000395.swis.ap-northeast-2.aws.nxp.com>
Date: Wed, 17 Dec 2025 14:52:11 +0900
From: Joseph Guo <qijian.guo@....com>
To: Yanan Yang <yanan.yang@....com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
	Sascha Hauer <s.hauer@...gutronix.de>,
	Pengutronix Kernel Team <kernel@...gutronix.de>,
	Fabio Estevam <festevam@...il.com>, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, imx@...ts.linux.dev,
	linux-arm-kernel@...ts.infradead.org, justin.jiang@....com,
	Lei Xu <lei.xu@....com>, Xiaofeng Wei <xiaofeng.wei@....com>,
	qijian.guo@....nxp.com
Subject: Re: [PATCH v2 2/2] arm64: dts: freescale: add NXP FRDM-IMX91S board
 support

On Mon, Dec 15, 2025 at 05:42:52PM +0900, Yanan Yang wrote:
> Add DeviceTree support for the NXP FRDM-IMX91S development board based
> on the i.MX91 SoC. FRDM-IMX91S is a cost-optimized variant of FRDM-IMX91
> and differs in memory, storage, Ethernet, and PMIC configuration:
> 
> - 512MB LPDDR4 (FRDM-IMX91 uses 1GB)
> - 256MB FlexSPI-NAND (FRDM-IMX91 uses 8GB eMMC)
> - Single GbE port (FRDM-IMX91 has dual GbE)
> - PMIC PF9453 (FRDM-IMX91 uses PCA9451A)
> 
> This DT enables:
> - ADC1
> - UART1 and UART5
> - I2C bus and children nodes
> - USB and related nodes
> - uSDHC1 and uSDHC2
> - FlexSPI NAND flash
> - Watchdog3
> - Ethernet (eqos)
> - FlexCAN
> - MQS
> 
> Link: https://www.nxp.com/design/design-center/development-boards-and-designs/FRDM-IMX91S
> (FRDM-IMX91S board page)
> Link: https://www.nxp.com/design/design-center/development-boards-and-designs/FRDM-IMX91
> (FRDM-IMX91 board page)
> 
> Co-developed-by: Lei Xu <lei.xu@....com>
> Signed-off-by: Lei Xu <lei.xu@....com>
> Co-developed-by: Xiaofeng Wei <xiaofeng.wei@....com>
> Signed-off-by: Xiaofeng Wei <xiaofeng.wei@....com>
> Signed-off-by: Yanan Yang <yanan.yang@....com>
> 
> ---
> Changes in v2:
> - Correct author name: removed extra period in "Yanan.Yang"
> - Removed aliases pointing to disabled nodes from DTS
> - Add "rtc0 = &pcf2131;" in aliases to make external RTC the primary device
> - Fix DT node name: change
>   "usdhc1_pwrseq: usdhc1_pwrseq" to "usdhc1_pwrseq: usdhc1-pwrseq"
> - Renamed LED node from 'status' to 'led-0' to comply with gpio-leds binding
> - Replace regulator-based transceiver (reg_can_stby) with CAN PHY (nxp,tja1051)
> - Replaced 'xceiver-supply' property with 'phys' in flexcan1 node
> - Adjusted alignment in fsl,pins property: used spaces between pin names and values for proper DTS style.
> ---
>  arch/arm64/boot/dts/freescale/Makefile             |   1 +
>  .../boot/dts/freescale/imx91-11x11-frdm-s.dts      | 677 +++++++++++++++++++++
>  2 files changed, 678 insertions(+)
>

Hi Yanan,

According to u-boot requirement. We should ensure the minimum difference to the U-boot dts and kernel DTS.
So I suggesst to add bootph- properties to the nodes should be initialized at u-boot stage.

Regards,
Joseph