On 18:20-20240109, Roger Quadros wrote:
>
>
> On 08/01/2024 19:32, Nishanth Menon wrote:
> > We now can provide a map and have the standard fdtfile variable set from
> > code itself. This allows for bootstd to "just work".
> >
> > While at this, replace findfdt in environment with a warning as it is no
> > longer needed.
> >
> > Signed-off-by: Nishanth Menon <n...@ti.com>
> > ---
> > board/ti/j721e/evm.c | 8 ++++++++
> > board/ti/j721e/j721e.env | 10 ----------
> > 2 files changed, 8 insertions(+), 10 deletions(-)
> >
> > diff --git a/board/ti/j721e/evm.c b/board/ti/j721e/evm.c
> > index c541880107ec..ad6ef4553e04 100644
> > --- a/board/ti/j721e/evm.c
> > +++ b/board/ti/j721e/evm.c
> > @@ -16,6 +16,7 @@
> > #include <dm.h>
> >
> > #include "../common/board_detect.h"
> > +#include "../common/fdt_ops.h"
> >
> > #define board_is_j721e_som() (board_ti_k3_is("J721EX-PM1-SOM") || \
> > board_ti_k3_is("J721EX-PM2-SOM"))
> > @@ -424,6 +425,12 @@ void configure_serdes_sierra(void)
> > }
> >
> > #ifdef CONFIG_BOARD_LATE_INIT
> > +static struct ti_fdt_map ti_j721e_evm_fdt_map[] = {
> > + {"j721e", "k3-j721e-common-proc-board.dtb"},
> > + {"j721e-sk", "k3-j721e-sk.dtb"},
>
> You missed "j721e-eaik"
Nope, it is intentional - eaik (the older name) is handled as sk in
setup_board_eeprom_env board_name[1] ==> board_is_j721e_sk provides
that abstraction[2]
[1] https://github.com/u-boot/u-boot/blob/master/board/ti/j721e/evm.c#L427
[2] https://github.com/u-boot/u-boot/blob/master/board/ti/j721e/evm.c#L23
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
849D 1736 249D
