[PATCH v2 2/2] imx: Support i.MX91 11x11 FRDM board

[Fabio Estevam]

On Tue, Dec 2, 2025 at 7:05 AM Joseph Guo <qijian.guo at nxp.com> wrote:

> +&usdhc2 {
> +       fsl,signal-voltage-switch-extra-delay-ms = <8>;

Why does U-Boot need this property? It doesn't exist in Linux,

> +};
> +
> +&fec {
> +       compatible = "fsl,imx91-fec", "fsl,imx93-fec", "fsl,imx8mq-fec";
> +       phy-reset-gpios = <&pcal6524 16 GPIO_ACTIVE_LOW>;
> +       phy-reset-duration = <15>;
> +       phy-reset-post-delay = <100>;

Using phy-reset-gpios is considered deprecated.

The PHY reset GPIO should be described inside the MDIO node.

> +&ethphy1 {
> +       reset-gpios = <&pcal6524 15 GPIO_ACTIVE_LOW>;
> +       reset-assert-us = <15000>;
> +       reset-deassert-us = <100000>;

Why is this inside the u-boot.dtsi instead of the main dts?


> +#define CFG_SYS_SDRAM_BASE     0x80000000
> +#define PHYS_SDRAM             0x80000000
> +#define PHYS_SDRAM_SIZE                0x80000000 /* 2GB DDR */

What about:

#define PHYS_SDRAM_SIZE   SIZE_2GB