Re: [PATCH 06/11] ARM: imxrt_defconfig: Remove NOMMU platform support

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:
- [Medium] Leftover references, platform-specific structures, compatible strings, and early console declarations for the removed i.MXRT and VF610M4 platforms remain in multiple drivers and DT bindings.
- [Low] Deleted clock dt-binding header is still included in the `imxrt1050-clock.yaml` dt-binding YAML example, breaking `dt_binding_check`.
--

commit 3968f192513e74f4358aed03401b5e2ab20e266b
Author: Frank Li <Frank.Li@nxp.com>

ARM: imxrt_defconfig: Remove NOMMU platform support

Commercial users and hardware vendors migrated to Zephyr or other RTOS
solutions years ago, leaving the NOMMU platform support effectively
unused and unmaintained.

> Remove the obsolete support to reduce maintenance burden and simplify the
> i.MX platform code.

[Severity: Medium]
Since this series removes the NOMMU platform support for i.MXRT and VF610M4,
should the remaining platform-specific structures and references also be
cleaned up?

There are leftover configurations such as usdhc_imxrt1050_data in
drivers/mmc/host/sdhci-esdhc-imx.c:

    static struct esdhc_soc_data usdhc_imxrt1050_data = {

imxrt1050_data in drivers/tty/serial/fsl_lpuart.c:

    static const struct lpuart_soc_data imxrt1050_data = {

early console declarations in drivers/tty/serial/fsl_lpuart.c:

    OF_EARLYCON_DECLARE(lpuart32, "fsl,imxrt1050-lpuart", lpuart32_imx_early_console_setup);

and the fsl,vf610m4 compatibility string in arch/arm/mach-imx/mach-vf610.c:

    "fsl,vf610m4",

Because the core architecture files and Kconfigs are removed, these remnants
become unreachable orphaned dead code.


[Severity: Low]
Does this series also need to update the dt-binding examples?

The deleted include/dt-bindings/clock/imxrt1050-clock.h header is still
included in the Documentation/devicetree/bindings/clock/imxrt1050-clock.yaml
example block:

    #include <dt-bindings/clock/imxrt1050-clock.h>

When the dt_binding_check target attempts to preprocess this YAML example,
the C preprocessor fails with a "No such file or directory" error.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260619-dts_cleanup_arm_mcore-v1-0-0101795a2662@nxp.com?part=6