[PATCH 08/10] board: beagle: beagleboneai64: Set fdtfile from C code instead of findfdt script

Nishanth Menon nm at ti.com
Mon Jan 8 21:04:32 CET 2024


On 13:00-20240108, Andrew Davis wrote:
> > +#ifdef CONFIG_BOARD_LATE_INIT
> > +int board_late_init(void)
> > +{
> > +	char fdtfile[50];
> > +
> > +	snprintf(fdtfile, sizeof(fdtfile), "%s/%s.dtb",
> > +		 CONFIG_TI_EVM_FDT_FOLDER_PATH, CONFIG_DEFAULT_DEVICE_TREE);
> > +
> 
> Why not use your new ti_set_fdt_env() helper? It makes this board

I had it originally, but did not want the code to depend on
board/ti/common functions. So dropped it in later version.

> consistent with TI boards (sets "name_fdt" and adds warning for
> "findfdt" like your commit message says you do..).

Will fix up the commit message in the next revision


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


More information about the U-Boot mailing list