Re: CVS commit: src/distrib/utils/embedded

2018-11-23 Thread Aymeric Vincent
Jared McNeill  writes:

> You shouldn't set fdtfile in this script because U-Boot already sets
> it. From socfpga_de0_nano_soc_defconfig:
>
> CONFIG_DEFAULT_FDT_FILE="socfpga_cyclone5_de0_nano_soc.dtb"
>
> This way, if you match on the SoC type in this file, you can run on
> any board using this SoC as long as you have a dtb for it. We
> shouldn't have to update the boot script to add new boards.

Thanks, agreed. The board has two names so I'll look into the problem in
the coming days to determine which files should be renamed to what in
order to be consistent.

Regards,
 Aymeric


Re: CVS commit: src/distrib/utils/embedded

2018-11-23 Thread Jared McNeill

On Thu, 22 Nov 2018, Aymeric Vincent wrote:


src/distrib/utils/embedded/files: armv7_boot.cmd


You shouldn't set fdtfile in this script because U-Boot already sets it. 

From socfpga_de0_nano_soc_defconfig:


CONFIG_DEFAULT_FDT_FILE="socfpga_cyclone5_de0_nano_soc.dtb"

This way, if you match on the SoC type in this file, you can run on any 
board using this SoC as long as you have a dtb for it. We shouldn't have 
to update the boot script to add new boards.