From: Masahiro Yamada <yamada.m@jp.panasonic.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 01/12] kbuild, mxs: use short logs for MXS images
Date: Thu, 13 Mar 2014 11:32:37 +0900 [thread overview]
Message-ID: <20140313113237.6F7A.AA925319@jp.panasonic.com> (raw)
In-Reply-To: <201403121146.11172.marex@denx.de>
Hello Marek,
> > > 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.
>
> Ah, right. The u-boot.sb is also encrypted with aes-128-cbc with zero key. The
> bootrom on mx23/mx28 has the encryption enabled by default (and it can be fused
> off), with the default key being all zeroes. That's why you see so many
> differences.
That makes sense. :-)
> Yes, I see. We shall eventually rename it to mkimage throughout the code.
>
> How shall we proceed? I see we have three options:
> - Post V2 of this patch
> - Fix the rest of the files which have this 'UIMAGE' in them and then post V2
> - Postpone the fix and then do it globally
>
> I would be in favor of the second option.
This patch is already on u-boot/master.
Can you post a follow-up patch?
I think you will have to modify 7 files.
./Makefile:731:quiet_cmd_mkimage = UIMAGE $@
./arch/arm/imx-common/Makefile:35:quiet_cmd_mkimage = UIMAGE $@
./arch/arm/cpu/arm926ejs/mxs/Makefile:20:quiet_cmd_mkimage_mxs = UIMAGE $@
./spl/Makefile:174:quiet_cmd_mkimage = UIMAGE $@
./board/cray/L1/Makefile:18:quiet_cmd_mkimage = UIMAGE $@
./board/matrix_vision/mvsmr/Makefile:15:quiet_cmd_mkimage = UIMAGE $@
./board/matrix_vision/mvblm7/Makefile:11:quiet_cmd_mkimage = UIMAGE $@
Best Regards
Masahiro Yamada
next prev parent reply other threads:[~2014-03-13 2:32 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-11 2:05 [U-Boot] [PATCH 0/12] Kbuild: A collection of more refactoring Masahiro Yamada
2014-03-11 2:05 ` [U-Boot] [PATCH 01/12] kbuild,mxs: use short logs for MXS images Masahiro Yamada
2014-03-11 11:52 ` [U-Boot] [PATCH 01/12] kbuild, mxs: " Marek Vasut
2014-03-12 10:19 ` Masahiro Yamada
2014-03-12 10:46 ` Marek Vasut
2014-03-12 12:46 ` Veli-Pekka Peltola
2014-03-12 12:49 ` Marek Vasut
2014-03-13 2:32 ` Masahiro Yamada [this message]
2014-03-13 4:01 ` Marek Vasut
2014-03-13 13:22 ` Tom Rini
2014-03-12 21:06 ` [U-Boot] [U-Boot, " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 02/12] kbuild: use short logs for some board specific make rules Masahiro Yamada
2014-03-12 21:06 ` [U-Boot] [U-Boot, " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 03/12] kbuild, x86: use a short log for arch/x86/lib/libgcc.a Masahiro Yamada
2014-03-12 21:06 ` [U-Boot] [U-Boot, " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 04/12] x86: specify CONFIG_USE_PRIVATE_LIBGCC more simply Masahiro Yamada
2014-03-12 21:06 ` [U-Boot] [U-Boot, " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 05/12] kbuild: delete redundant LDSCRIPT definition Masahiro Yamada
2014-03-11 9:09 ` Michal Simek
2014-03-12 21:06 ` [U-Boot] [U-Boot, " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 06/12] freescale: pblimage: refactor CONFIG_SYS_FSL_PBL_{PBI, RCW} Masahiro Yamada
2014-03-12 21:06 ` [U-Boot] [U-Boot, " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 07/12] kirkwood: kwbimage: refactor CONFIG_SYS_KWD_CONFIG Masahiro Yamada
2014-03-12 21:06 ` [U-Boot] [U-Boot, " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 08/12] kbuild: use $(KBUILD_SRC) to check out-of-tree build Masahiro Yamada
2014-03-12 21:07 ` [U-Boot] [U-Boot, " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 09/12] kbuild: rename TOPDIR to stctree Masahiro Yamada
2014-03-12 21:07 ` [U-Boot] [U-Boot,09/12] " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 10/12] kbuild: rename SRCTREE to srctree Masahiro Yamada
2014-03-12 21:07 ` [U-Boot] [U-Boot,10/12] " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 11/12] kbuild: rename OBJTREE to objtree Masahiro Yamada
2014-03-12 21:07 ` [U-Boot] [U-Boot,11/12] " Tom Rini
2014-03-11 2:05 ` [U-Boot] [PATCH 12/12] kbuild: delete SPLTREE and TPLTREE Masahiro Yamada
2014-03-12 21:07 ` [U-Boot] [U-Boot,12/12] " Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140313113237.6F7A.AA925319@jp.panasonic.com \
--to=yamada.m@jp.panasonic.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox