Re: [PATCH v4 08/11] configs: am62lx_evm: add initial defconfig
Simon Glass <[email protected]>
| Newsgroups | gmane.comp.boot-loaders.u-boot |
|---|---|
| Message-ID | <CAFLszTj2jqcQG9P4nWWiXjJ-7cHko8WghVqCbBb-FExnH+gUFw__35691.6874575958$1786050720$gmane$org@mail.gmail.com> |
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. > > 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? Regards, Simon