[PATCH] imx: fix imx8mm_evk config

[Peng Fan]

Hi Fabio,

> Subject: Re: [PATCH] imx: fix imx8mm_evk config
> 
> Hi Peng,
> 
> On Thu, Feb 13, 2025 at 2:34 PM Ludwig Nussel
> <ludwig.nussel at siemens.com> wrote:
> >
> > Commit 364ba68ed1a adjusted Kconfig for imx8mn_evk instead of
> 
> You haven't tested 364ba68ed1a, have you?

Not test all the targets, 93 and 8mp were tested.

Stupid issues, copy paste from one board to the other, but forgot me
update the name. shame on me.

Sorry for this.

Would you expect a v2 from Ludwig or need me to address those in
a separate patch?

Thanks,
Peng.
> 
> > imx8mm_evk for bootstd. Therefore mm is actually left building
> without
> > bootstd by default.
> >
> > Also the name of the dtb referenced mp instead of mm. IIUC
> > CONFIG_DEFAULT_FDT_FILE is meant to point to the path of the dtb
> > within the kernel install directory. Therefore adding freescale/ as
> > subdir too.
> 
> > --- a/configs/imx8mm_evk_defconfig
> > +++ b/configs/imx8mm_evk_defconfig
> > @@ -25,7 +25,7 @@ CONFIG_FIT=y
> >  CONFIG_FIT_EXTERNAL_OFFSET=0x3000
> >  CONFIG_SPL_LOAD_FIT=y
> >  CONFIG_OF_SYSTEM_SETUP=y
> > -CONFIG_DEFAULT_FDT_FILE="imx8mp-evk.dtb"
> > +CONFIG_DEFAULT_FDT_FILE="freescale/imx8mm-evk.dtb"
> 
> I think this could be CONFIG_DEFAULT_FDT_FILE="imx8mm-evk.dtb"
> 
> >  CONFIG_SYS_CBSIZE=2048
> >  CONFIG_SYS_PBSIZE=2074
> >  CONFIG_BOARD_LATE_INIT=y
> > diff --git a/configs/imx8mm_evk_fspi_defconfig
> > b/configs/imx8mm_evk_fspi_defconfig
> > index 4a8d4994b80..a06b47ac3ba 100644
> > --- a/configs/imx8mm_evk_fspi_defconfig
> > +++ b/configs/imx8mm_evk_fspi_defconfig
> > @@ -29,7 +29,7 @@ CONFIG_FIT=y
> >  CONFIG_FIT_EXTERNAL_OFFSET=0x3000
> >  CONFIG_SPL_LOAD_FIT=y
> >  CONFIG_OF_SYSTEM_SETUP=y
> > -CONFIG_DEFAULT_FDT_FILE="imx8mp-evk.dtb"
> > +CONFIG_DEFAULT_FDT_FILE="freescale/imx8mm-evk.dtb"
> 
> Ditto.
> 
> Another error I see with 364ba68ed1a is this hunk:
> 
> --- a/configs/imx8mm_evk_fspi_defconfig
> +++ b/configs/imx8mm_evk_fspi_defconfig
> @@ -5,6 +5,7 @@ CONFIG_SYS_MALLOC_LEN=0x2000000
> CONFIG_SPL_GPIO=y  CONFIG_SPL_LIBCOMMON_SUPPORT=y
> CONFIG_SPL_LIBGENERIC_SUPPORT=y
> +CONFIG_ENV_SOURCE_FILE="imx8mn_evk"
> 
> You cannot use the environment for imx8mn_evk on
> imx8mm_evk_fspi.
> 
> Just remove this line, and the correct env will be used.