Re: [PATCH v4 08/11] configs: am62lx_evm: add initial defconfig

Bryan Brattlof <[email protected]>
Newsgroups gmane.comp.boot-loaders.u-boot
Message-ID <20260807134512.ri7cmbidhvjcytej__11002.1150483348$1786110342$gmane$org@bryanbrattlof.com>
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

~Bryan
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.