[PATCH v1 2/3] board: toradex: add Toradex Verdin iMX95

[Simon Glass]

Hi Emanuele,

On 2026-04-16T08:45:22, Emanuele Ghidoli <ghidoliemanuele at gmail.com> wrote:
> board: toradex: add Toradex Verdin iMX95
>
> Add support for the Toradex Verdin iMX95.
>
> Link: https://www.toradex.com/computer-on-modules/verdin-arm-family/nxp-imx95
> Link: https://www.toradex.com/products/carrier-board/verdin-development-board-kit
> Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli at toradex.com>
> Co-developed-by: Ernest Van Hoecke <ernest.vanhoecke at toradex.com>
> Signed-off-by: Ernest Van Hoecke <ernest.vanhoecke at toradex.com>
> Co-developed-by: Francesco Dolcini <francesco.dolcini at toradex.com>
> Signed-off-by: Francesco Dolcini <francesco.dolcini at toradex.com>
>
> arch/arm/dts/imx95-verdin-dev.dtsi             |  239 +++++
>  arch/arm/dts/imx95-verdin-wifi-dev-u-boot.dtsi |  112 +++
>  arch/arm/dts/imx95-verdin-wifi-dev.dts         |   21 +
>  arch/arm/dts/imx95-verdin-wifi.dtsi            |   50 +
>  arch/arm/dts/imx95-verdin.dtsi                 | 1172 ++++++++++++++++++++++++
>  arch/arm/mach-imx/imx9/Kconfig                 |    5 +
>  board/toradex/verdin-imx95/Kconfig             |   36 +
>  board/toradex/verdin-imx95/MAINTAINERS         |   13 +
>  board/toradex/verdin-imx95/Makefile            |    8 +
>  board/toradex/verdin-imx95/spl.c               |   75 ++
>  board/toradex/verdin-imx95/verdin-imx95.c      |   79 ++
>  board/toradex/verdin-imx95/verdin-imx95.env    |   20 +
>  configs/verdin-imx95_defconfig                 |  183 ++++
>  doc/board/toradex/index.rst                    |    1 +
>  doc/board/toradex/verdin-imx95.rst             |  171 ++++
>  include/configs/verdin-imx95.h                 |   27 +
>  16 files changed, 2212 insertions(+)

> diff --git a/include/configs/verdin-imx95.h b/include/configs/verdin-imx95.h
> @@ -0,0 +1,27 @@
> +/* For 32GB modules: 2GB from 0x80000000..0xffffffff, 30GB above.
> + * Actual size is determited at runtime.

determined

> diff --git a/board/toradex/verdin-imx95/verdin-imx95.c b/board/toradex/verdin-imx95/verdin-imx95.c
> @@ -0,0 +1,79 @@
> +const struct ram_alias_check ram_alias_check[] = {
> +     { (void *)(PHYS_SDRAM + SZ_8G), (void *)(PHYS_SDRAM), SZ_16G },
> +     { (void *)(PHYS_SDRAM + SZ_4G), (void *)(PHYS_SDRAM), SZ_8G },
> +     { (void *)(PHYS_SDRAM + SZ_2G), (void *)(PHYS_SDRAM), SZ_4G },
> +     { (void *)(PHYS_SDRAM + SZ_1G), (void *)(PHYS_SDRAM), SZ_2G },
> +     { NULL }
> +};

How about ram_alias_checks[] to avoid confusion with the type. Also,
can it be static?

Reviewed-by: Simon Glass <sjg at chromium.org>

Regards,
Simon