[PATCH v6 00/11] An effort to bring DT bindings compliance within U-Boot

[Sumit Garg]

Hi Fabio,

On Fri, 1 Mar 2024 at 22:20, Fabio Estevam <festevam at gmail.com> wrote:
>
> Hi Sumit,
>
> On Fri, Mar 1, 2024 at 9:57 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Thu, 22 Feb 2024 15:05:56 +0530, Sumit Garg wrote:
> >
> > > Changes in v6:
> > > --------------
> > > - v6_dt: https://github.com/b49020/u-boot/tree/v6_dt
> > > - Patch #3: Incorporate fix for sandbox CI failure.
> > > - Patch #6: Incorporate shell script comments from Marek.
> > > - Patch #8: Incorporate documentation review comments from Paul.
> > >
> > > [...]
> >
> > Applied to u-boot/next, thanks!
>
> I am trying to use OF_UPSTREAM with imx8mp_evk_defconfig:
>
> diff --git a/arch/arm/mach-imx/imx8m/Kconfig b/arch/arm/mach-imx/imx8m/Kconfig
> index 328c3e388051..29d0c5ce6485 100644
> --- a/arch/arm/mach-imx/imx8m/Kconfig
> +++ b/arch/arm/mach-imx/imx8m/Kconfig
> @@ -8,6 +8,7 @@ config IMX8M
>         select LTO
>         select ROM_UNIFIED_SECTIONS
>         select ARMV8_CRYPTO
> +       imply OF_UPSTREAM
>
>  config IMX8MQ
>         bool
> diff --git a/configs/imx8mp_evk_defconfig b/configs/imx8mp_evk_defconfig
> index 2350d2f409b7..cdf840c683c0 100644
> --- a/configs/imx8mp_evk_defconfig
> +++ b/configs/imx8mp_evk_defconfig
> @@ -8,7 +8,7 @@ CONFIG_SPL_LIBGENERIC_SUPPORT=y
>  CONFIG_ENV_SIZE=0x1000
>  CONFIG_ENV_OFFSET=0x400000
>  CONFIG_DM_GPIO=y
> -CONFIG_DEFAULT_DEVICE_TREE="imx8mp-evk"
> +CONFIG_DEFAULT_DEVICE_TREE="freescale/imx8mp-evk"
>  CONFIG_SPL_TEXT_BASE=0x920000
>  CONFIG_TARGET_IMX8MP_EVK=y
>  CONFIG_SYS_MONITOR_LEN=524288
>
> I ran "./dts/update-dts-subtree.sh pull v6.8-rc6-dts".
>
> make mrproper
> make imx8mp_evk_defconfig
> make
>
> The dtb is built: dts/upstream/src/arm64/freescale/imx8mp-evk.dtb
>
> but mkimage still looks for the old path:
>
>  MKIMAGE u-boot.img
> ./tools/mkimage: Can't open arch/arm/dts/freescale/imx8mp-evk.dtb: No
> such file or directory
> ./tools/mkimage: failed to build FIT
>
> Any suggestions?

I think you also need Bryan's patch [1] to be incorporated as well.
Give it a try and let me know if you still see any further issues.

[1] https://patchwork.ozlabs.org/project/uboot/patch/20240205-am62px-wip-rebasing-v3-11-04cbb42eaa6f@ti.com/

-Sumit

>
> Thanks