[U-Boot] [PATCH 01/12] kbuild, mxs: use short logs for MXS images

[Masahiro Yamada]

Hello Marek,


> > Cc: Marek Vasut <marex at denx.de>
> > Cc: Fabio Estevam <fabio.estevam at freescale.com>
> > Cc: Otavio Salvador <otavio at ossystems.com.br>
> > Cc: Lauri Hintsala <lauri.hintsala at bluegiga.com>
> 
> You can drop Lauri from CC , I think this address is dead.

It looks like Cc to Lauri is not bouncing so far.




> b5d5feab4a6974fe00272d3a3408300d8c8a473d  u-boot.bin
> 
> Given that u-boot.bin and spl/u-boot-spl.bin are packed into u-boot.sb , this is 
> why you always see a different u-boot.sb .

For other boot images, binary compare matched except a few bytes
difference which comes from time stamp.

But, when  I compared u-boot.sb
I had much more diffs than I expect for the difference of timestamp.

I still cannot understand what is different from the others.

Anyway, that's good to know it worked on run test.


> > I wish someone could do run test this patch.
> 
> The patch is good, I have just a minor rant below, otherwise add my:
> 
> Acked-by: Marek Vasut <marex at denx.de>

It's really appreciated. Thanks!


> > -$(OBJTREE)/mxsimage.cfg:
> > $(SRCTREE)/$(CPUDIR)/$(SOC)/mxsimage.$(MKIMAGE_TARGET-y).cfg -	sed
> > "s at OBJTREE@$(OBJTREE)@g" $^ > $@
> > +quiet_cmd_mkimage_mxs = UIMAGE  $@
> 
> Why don't we use 'MKIMAGE' here to be consistent ? Or 'MXSIMAGE' if we want to 
> differentiate ?

Good point.
MXSIMAGE is no good because the short log must be within 7 characters.
MKIMAGE is OK.

Why UIMAGE?

All explanation I can give is 
just the mimic of scripts/Makefile.build which originates
from Linux Kernel.

Around line 340,

quiet_cmd_uimage = UIMAGE  $(UIMAGE_OUT)
      cmd_uimage = $(CONFIG_SHELL) $(MKIMAGE) -A $(UIMAGE_ARCH) -O linux \


Anyway I do not persist on "UIMAGE".
If requested (or you can post a patch), it's  OK to rename it,
althogh naming must be consistent throughout the code.


Best Regards
Masahiro Yamada