From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lokesh Vutla Date: Tue, 3 May 2016 19:26:21 +0530 Subject: [U-Boot] [PATCH] mkimage: fix generation of FIT image In-Reply-To: <1462281423-7649-1-git-send-email-andreas@biessmann.org> References: <1462281423-7649-1-git-send-email-andreas@biessmann.org> Message-ID: <5728AE05.7070204@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tuesday 03 May 2016 06:47 PM, Andreas Bie?mann wrote: > Commit 7a439cadcf3192eb012a2432ca34670b676c74d2 broke generation of SPL > loadable FIT images (CONFIG_SPL_LOAD_FIT). > Fix it by removing the unnecessary storage of expected image type. This was a > left over of the previous implementation. It is not longer necessary since the > mkimage -b switch always has one parameter. Thanks for the Patch. This fixes the build with CONFIG_SPL_LOAD_FIT enabled. Tested-by: Lokesh Vutla Thanks and regards, Lokesh > > Signed-off-by: Andreas Bie?mann > --- > > tools/mkimage.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/tools/mkimage.c b/tools/mkimage.c > index b407aed..93d1c16 100644 > --- a/tools/mkimage.c > +++ b/tools/mkimage.c > @@ -133,10 +133,8 @@ static void process_args(int argc, char **argv) > char *ptr; > int type = IH_TYPE_INVALID; > char *datafile = NULL; > - int expecting; > int opt; > > - expecting = IH_TYPE_COUNT; /* Unknown */ > while ((opt = getopt(argc, argv, > "a:A:b:cC:d:D:e:Ef:Fk:K:ln:O:rR:sT:vVx")) != -1) { > switch (opt) { > @@ -154,8 +152,7 @@ static void process_args(int argc, char **argv) > usage("Invalid architecture"); > break; > case 'b': > - expecting = IH_TYPE_FLATDT; > - if (add_content(expecting, optarg)) { > + if (add_content(IH_TYPE_FLATDT, optarg)) { > fprintf(stderr, > "%s: Out of memory adding content '%s'", > params.cmdname, optarg); > @@ -238,7 +235,6 @@ static void process_args(int argc, char **argv) > show_image_types(); > usage("Invalid image type"); > } > - expecting = type; > break; > case 'v': > params.vflag++; > @@ -254,7 +250,8 @@ static void process_args(int argc, char **argv) > } > } > > - if (optind < argc && expecting == type) > + /* The last parameter is expected to be the imagefile */ > + if (optind < argc) > params.imagefile = argv[optind]; > > /* >