[U-Boot] [PATCH 08/11] Makefile.spl: Move generate AT91SAM NAND image boot.bin to spl directory

Heiko Schocher hs at denx.de
Tue Mar 26 07:06:16 UTC 2019


Hello Stefan,

Am 25.03.2019 um 15:24 schrieb Stefan Roese:
> On 25.03.19 15:22, Eugen.Hristev at microchip.com wrote:
>>
>>
>> On 19.03.2019 17:56, Stefan Roese wrote:
>>> External E-Mail
>>>
>>>
>>> This patch moves the AT91SAM NAND booting SPL image "boot.bin" which
>>> includes the ECC values from the root directory into the spl directory,
>>> where all SPL related images are located.
>>
>> Hi Stefan,
>>
>> Yes, indeed, but someone may be using this fact? Removing it might break
>> someone's expectations (scripts). Can we make a copy to spl/ dir and
>> leave the boot.bin in the root as-is ?
> 
> That would be possible, but I find the solution to move the file
> to the correct directory cleaner / better. Who uses this image?
> I'm pretty sure that Heiko does.
> 
> Heiko, do you have any comments on this SPL binary file move? Is
> somebody else using it?

I am fine with this move, as I also think, spl directory is the
correct place for it. Yes, I have some scripts, but they are
fixed very fast.

Acked-by: Heiko Schocher<hs at denx.de>

bye,
Heiko
> 
> Thanks,
> Stefan
> 
>> Unless someone has objections of course...
>>
>> Eugen
>>
>>>
>>> Signed-off-by: Stefan Roese <sr at denx.de>
>>> Cc: Heiko Schocher <hs at denx.de>
>>> Cc: Andreas Bießmann <andreas at biessmann.org>
>>> Cc: Eugen Hristev <eugen.hristev at microchip.com>
>>> ---
>>>    scripts/Makefile.spl | 6 +++---
>>>    1 file changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/scripts/Makefile.spl b/scripts/Makefile.spl
>>> index 9d5921606e..37be18da4b 100644
>>> --- a/scripts/Makefile.spl
>>> +++ b/scripts/Makefile.spl
>>> @@ -179,10 +179,10 @@ MKIMAGEFLAGS_boot.bin = -T atmelimage
>>>    ifeq ($(CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER),y)
>>>    MKIMAGEFLAGS_boot.bin += -n $(shell $(obj)/../tools/atmel_pmecc_params)
>>> -boot.bin: $(obj)/../tools/atmel_pmecc_params
>>> +$(obj)/boot.bin: $(obj)/../tools/atmel_pmecc_params
>>>    endif
>>> -boot.bin: $(obj)/u-boot-spl.bin FORCE
>>> +$(obj)/boot.bin: $(obj)/u-boot-spl.bin FORCE
>>>        $(call if_changed,mkimage)
>>>    else
>>>    ifdef CONFIG_ARCH_ZYNQ
>>> @@ -225,7 +225,7 @@ endif
>>>    endif
>>>    ifeq ($(CONFIG_SYS_SOC),"at91")
>>> -ALL-y    += boot.bin
>>> +ALL-y    += $(obj)/boot.bin
>>>    endif
>>>    ALL-$(CONFIG_SPL_X86_16BIT_INIT) += $(obj)/u-boot-x86-16bit-spl.bin
>>>
> 
> Viele Grüße,
> Stefan
> 

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de


More information about the U-Boot mailing list