[U-Boot,v4,4/4] u-boot.elf: add quiet_cmd_u-boot-elf and cmd_u-boot-elf

Submitted by Álvaro Fernández Rojas on April 20, 2017, 6:36 p.m.

Details

Message ID 1492713388-310-5-git-send-email-noltari@gmail.com
State Accepted
Commit e69945ee8a30676bfafb9b7956026bd11733f9b3
Delegated to: Daniel Schwierzeck
Headers show

Commit Message

Álvaro Fernández Rojas April 20, 2017, 6:36 p.m.
This way we can see output about u-boot.elf being built or not.

Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
---
 v4: Introduce changes suggested by Tom Rini:
  - Add new patch to output u-boot.elf build.

 Makefile | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

Comments

Tom Rini April 20, 2017, 8:06 p.m.
On Thu, Apr 20, 2017 at 08:36:28PM +0200, Álvaro Fernández Rojas wrote:

> This way we can see output about u-boot.elf being built or not.
> 
> Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>

Reviewed-by: Tom Rini <trini@konsulko.com>
Daniel Schwierzeck April 30, 2017, 6:41 p.m.
Am 20.04.2017 um 20:36 schrieb Álvaro Fernández Rojas:
> This way we can see output about u-boot.elf being built or not.
> 
> Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
> ---
>  v4: Introduce changes suggested by Tom Rini:
>   - Add new patch to output u-boot.elf build.
> 
>  Makefile | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
> 

applied to u-boot-mips/next, thanks!

Patch hide | download patch | download mbox

diff --git a/Makefile b/Makefile
index 6cf2568..3722b19 100644
--- a/Makefile
+++ b/Makefile
@@ -1187,11 +1187,13 @@  u-boot-img-spl-at-end.bin: u-boot.img spl/u-boot-spl.bin FORCE
 ifndef PLATFORM_ELFENTRY
   PLATFORM_ELFENTRY = "_start"
 endif
+quiet_cmd_u-boot-elf ?= LD      $@
+	cmd_u-boot-elf ?= $(LD) u-boot-elf.o -o $@ \
+	--defsym=$(PLATFORM_ELFENTRY)=$(CONFIG_SYS_TEXT_BASE) \
+	-Ttext=$(CONFIG_SYS_TEXT_BASE)
 u-boot.elf: u-boot.bin
-	@$(OBJCOPY) -I binary $(PLATFORM_ELFFLAGS) $< u-boot-elf.o
-	@$(LD) u-boot-elf.o -o $@ \
-		--defsym=$(PLATFORM_ELFENTRY)=$(CONFIG_SYS_TEXT_BASE) \
-		-Ttext=$(CONFIG_SYS_TEXT_BASE)
+	$(Q)$(OBJCOPY) -I binary $(PLATFORM_ELFFLAGS) $< u-boot-elf.o
+	$(call if_changed,u-boot-elf)
 
 # Rule to link u-boot
 # May be overridden by arch/$(ARCH)/config.mk