[PATCH] am625x_evm_a53: Tweak boot command to set fdt

[Martyn Welch]

On Tue, 2024-03-26 at 14:35 +0000, Christopher Obbard wrote:
> Hi Martyn.
> 
> On Tue, 2024-03-26 at 14:26 +0000, Martyn Welch wrote:
> > With the current config for tha SK-AM62, fdtfile isn't set in the
> > U-Boot
> > environment. When using bootflow to boot from a block device, where
> > the
> > extlinux.conf file specifies `fdtdir`, a fallback device tree is
> > being
> > constructed from the `soc` (`k3`) and `board` (`am62x`) environment
> > variables, resulting in u-Boot trying to retrieve
> > `/dtbs/6.8.1+/k3-am62x.dtb`. This file doesn't exist.
> > 
> > The environment variables `default_device_tree` and
> > `default_device_tree_arch` are set in the config, the `findfdt`
> > script
> > can be called to construct `fdtfile` from the environment variables
> > set
> > by these config options, however this script currently isn't being
> > run.
> > 
> > Calling this script results in the correct device tree being
> > retrieved:
> > 
> >    Retrieving file: /dtbs/6.8.1+/ti/k3-am625-sk.dtb
> > 
> > Many boards are calling this script as part of their boot command.
> > The
> > am62x currently isn't. Rectify this so that booting works
> > correctly.
> > 
> > Signed-off-by: Martyn Welch <martyn.welch at collabora.com>
> 
> Would this also be appropriate to add to the defconfig for other
> AM62x boards
> (e.g beagleplay) ?
> 

Hi Chris,

Probably, but this is the only board I have available to test.

Martyn

> Anyway, for this device:
> 
> Reviewed-by: Christopher Obbard <chris.obbard at collabora.com>
> 
> > ---
> >  configs/am62x_evm_a53_defconfig | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/configs/am62x_evm_a53_defconfig
> > b/configs/am62x_evm_a53_defconfig
> > index 78287ec18a..e373e44a93 100644
> > --- a/configs/am62x_evm_a53_defconfig
> > +++ b/configs/am62x_evm_a53_defconfig
> > @@ -30,7 +30,7 @@ CONFIG_SPL_LOAD_FIT=y
> >  CONFIG_SPL_LOAD_FIT_ADDRESS=0x81000000
> >  CONFIG_BOOTSTD_FULL=y
> >  CONFIG_SYS_BOOTM_LEN=0x800000
> > -CONFIG_BOOTCOMMAND="run envboot; bootflow scan -lb"
> > +CONFIG_BOOTCOMMAND="run findfdt; run envboot; bootflow scan -lb"
> >  CONFIG_BOARD_LATE_INIT=y
> >  CONFIG_SPL_MAX_SIZE=0x58000
> >  CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
> 
> Thanks!
> 
> Chris