[PATCH 06/10] board: ti: j721e: Set fdtfile from C code instead of findfdt script

[Nishanth Menon]

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 <nm at 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