[U-Boot] [PATCH 5/7] socfpga: Simplify Makefile filenames

[Simon Glass]

Hi Marek,

On 25 January 2016 at 13:43, Marek Vasut <marex at denx.de> wrote:
> On Monday, January 25, 2016 at 09:30:29 PM, Simon Glass wrote:
>> We don't need the -dtb suffix anymore, so drop it.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> ---
>
> You should also fix the MCV:
>
> $ git grep u-boot-with-spl-dtb.sfp
> ...
> include/configs/socfpga_mcvevk.h:  "update_filename=u-boot-with-spl-dtb.sfp\0"
>

OK

> I have to admit, I am not a big fan of this renaming, since it's just
> introducing naming chaos between versions.

I'm rather hoping that the impact will be fairly small for the reasons
I explained in the cover letter.

>
>>  Makefile             | 10 +++++-----
>>  scripts/Makefile.spl |  6 +++---
>>  2 files changed, 8 insertions(+), 8 deletions(-)
>>
>> diff --git a/Makefile b/Makefile
>> index fa55c71..66e96cb 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -1015,10 +1015,10 @@ u-boot.spr: spl/u-boot-spl.img u-boot.img FORCE
>>
>>  ifneq ($(CONFIG_ARCH_SOCFPGA),)
>>  quiet_cmd_socboot = SOCBOOT $@
>> -cmd_socboot = cat    spl/u-boot-spl-dtb.sfp spl/u-boot-spl-dtb.sfp   \
>> -                     spl/u-boot-spl-dtb.sfp spl/u-boot-spl-dtb.sfp   \
>> -                     u-boot-dtb.img > $@ || rm -f $@
>> -u-boot-with-spl-dtb.sfp: spl/u-boot-spl-dtb.sfp u-boot-dtb.img FORCE
>> +cmd_socboot = cat    spl/u-boot-spl.sfp spl/u-boot-spl.sfp   \
>> +                     spl/u-boot-spl.sfp spl/u-boot-spl.sfp   \
>> +                     u-boot.img > $@ || rm -f $@
>> +u-boot-with-spl.sfp: spl/u-boot-spl.sfp u-boot.img FORCE
>>       $(call if_changed,socboot)
>>  endif
>>
>> @@ -1323,7 +1323,7 @@ spl/u-boot-spl: tools prepare $(if
>> $(CONFIG_OF_SEPARATE),dts/dt.dtb) spl/sunxi-spl.bin: spl/u-boot-spl
>>       @:
>>
>> -spl/u-boot-spl-dtb.sfp: spl/u-boot-spl
>> +spl/u-boot-spl.sfp: spl/u-boot-spl
>>       @:
>>
>>  spl/boot.bin: spl/u-boot-spl
>> diff --git a/scripts/Makefile.spl b/scripts/Makefile.spl
>> index 7bf471a..aa7091c 100644
>> --- a/scripts/Makefile.spl
>> +++ b/scripts/Makefile.spl
>> @@ -146,7 +146,7 @@ ALL-y     += $(obj)/$(BOARD)-spl.bin
>>  endif
>>
>>  ifdef CONFIG_ARCH_SOCFPGA
>> -ALL-y        += $(obj)/$(SPL_BIN)-dtb.sfp
>> +ALL-y        += $(obj)/$(SPL_BIN).sfp
>>  endif
>>
>>  ifdef CONFIG_SUNXI
>> @@ -230,8 +230,8 @@ LDFLAGS_$(SPL_BIN) += -Ttext $(CONFIG_SPL_TEXT_BASE)
>>  endif
>>
>>  ifdef CONFIG_ARCH_SOCFPGA
>> -MKIMAGEFLAGS_$(SPL_BIN)-dtb.sfp = -T socfpgaimage
>> -$(obj)/$(SPL_BIN)-dtb.sfp: $(obj)/$(SPL_BIN)-dtb.bin FORCE
>> +MKIMAGEFLAGS_$(SPL_BIN).sfp = -T socfpgaimage
>> +$(obj)/$(SPL_BIN).sfp: $(obj)/$(SPL_BIN).bin FORCE
>>       $(call if_changed,mkimage)
>>  endif
>
> Best regards,
> Marek Vasut

Regards,
Simon