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

Reply via email to