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

On 15:08-20240109, Roger Quadros wrote: [..]

...

#ifdef CONFIG_BOARD_LATE_INIT +static struct ti_fdt_map ti_am64_evm_fdt_map[] = {

• {"am64x_gpevm", "k3-am642-evm.dtb"},
• {"am64x_skevm", "k3-am642-sk.dtb"},

"am64x_gpevm" and "am64x_skevm" strings are used multiple times in this file. see setup_board_eeprom_env()

Please use a MACRO for them.

What is the logic of choosing this name and can it be updated at this point? e.g."gp" is misleading in the board name as the boards are now shipped with HS-FS chip and are no longer GP.

Intent of the series was to get rid of findfdt - it was not meant to cleanup existing name usage in the files. If there is a desire to do so, please - patches are welcome.