On Wed, Jan 04, 2023 at 03:13:44PM +0530, Sinthu Raja wrote:

[snip]
> +#ifdef CONFIG_SPL_LOAD_FIT
> +int board_fit_config_name_match(const char *name)
> +{
> +     bool eeprom_read = board_ti_was_eeprom_read();
> +
> +     if (!eeprom_read || board_is_j721s2_som()) {
> +             if (!strcmp(name, "k3-j721s2-common-proc-board"))
> +                     return 0;

What I'd like to see is a comment above this part noting that this is
the safe default for the SoC.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to