Re: [PATCH 02/10] board: ti: am62ax: Set fdtfile from C code instead of findfdt script

2024-01-09 Thread Nishanth Menon
On 20:48-20240108, Jon Humphreys wrote: [..] > > + > > +#ifdef CONFIG_BOARD_LATE_INIT > > +int board_late_init(void) > > +{ > > + ti_set_fdt_env(NULL, NULL); > > If a board only has one FDT possible, why not just hard code the path > here, rather than use the set_fdt_env() logic with NULLs and

Re: [PATCH 02/10] board: ti: am62ax: Set fdtfile from C code instead of findfdt script

2024-01-08 Thread Jon Humphreys
Nishanth Menon writes: > Stop using the findfdt script and switch to setting the fdtfile from > C code. > > While at this, replace findfdt in environment with a warning as it is > no longer needed > > Signed-off-by: Nishanth Menon > --- > board/ti/am62ax/am62ax.env | 1 - > board/ti/am62

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

2024-01-08 Thread Nishanth Menon
Stop using the findfdt script and switch to setting the fdtfile from C code. While at this, replace findfdt in environment with a warning as it is no longer needed Signed-off-by: Nishanth Menon --- board/ti/am62ax/am62ax.env | 1 - board/ti/am62ax/evm.c| 10 ++ config