Hi Bryan, On Fri, 7 Aug 2026 at 07:45, Bryan Brattlof <[email protected]> wrote: > > On August 6, 2026 thus sayeth Simon Glass: > > Hi Bryan, > > > > On 2026-08-04T21:55:02, Bryan Brattlof <[email protected]> wrote: > > > configs: am62lx_evm: add initial defconfig > > > > > > Add the initial minimum configuration needed to boot the AM62L3-SK > > > reference boards. > > > > The commit message refers to AM62L3-SK but the defconfig is named > > am62lx_evm_defconfig and selects CONFIG_TARGET_AM62L3_EVM. Please make > > the naming consistent - either the board is an SK or an EVM, not both. > > I'll fix the commit message. > > > > > > > > > Tested-by: Anshul Dalal <[email protected]> > > > Signed-off-by: Bryan Brattlof <[email protected]> > > > > > > configs/am62lx_evm_defconfig | 138 > > > +++++++++++++++++++++++++++++++++++++++++++ > > > 1 file changed, 138 insertions(+) > > > > > diff --git a/configs/am62lx_evm_defconfig b/configs/am62lx_evm_defconfig > > > @@ -0,0 +1,138 @@ > > > +CONFIG_SPL_LOAD_FIT_ADDRESS=0x83000000 > > > +CONFIG_BOOTSTD_FULL=y > > > +# CONFIG_BOARD_INIT is not set > > > > Unlike the other AM62x A53 defconfigs there is no CONFIG_BOOTCOMMAND > > and no CONFIG_BOARD_LATE_INIT here, and the .env file added in patch 6 > > also has no bootcmd. Just to check - is this intentional (relying > > entirely on the default bootstd behaviour), or an omission? > > The downstream teams modify this for their needs[0]. The goal here is to > support as many boot methods as possible and let meta-ti do whatever > they want for the downstream fork. > > [0] > https://git.ti.com/cgit/ti-u-boot/ti-u-boot/commit/?h=ti-u-boot-2026.01&id=c754b0419ef89c20e0ffb5e1beb82b4606953b13 >
OK sounds good. Regards, SImon > ~Bryan
