[U-Boot] [PATCH] mkimage : Fix generating multi and script images

Simon Glass sjg at chromium.org
Mon Dec 7 14:06:48 CET 2015


Hi Philippe,

On 7 December 2015 at 01:02, Philippe De Swert
<philippedeswert at gmail.com> wrote:
> Hi Simon,
>
> On 07/12/15 06:31, Simon Glass wrote:
>
>
> -       dfd = open(params.datafile, O_RDONLY | O_BINARY);
> -       if (dfd < 0) {
> -               fprintf(stderr, "%s: Can't open %s: %s\n",
> -                       params.cmdname, params.datafile, strerror(errno));
> -               exit(EXIT_FAILURE);
> -       }
> +       if (!params.type == IH_TYPE_MULTI ||
> +           !params.type == IH_TYPE_SCRIPT) {
>
> This breaks rockchip image generation. I'm not sure what the above two
> lines are supposed to do, but if they are correct they are very
> confusing. Can you please take a look? I'll send a revert in the
> meantime.
>
>
> Weird, those lines only do something if  -T multi or -T script. These seem
> have command lines that will
> not work at all due to things not being expanded and the open below failing.
> I have no idea what the used command line would be for rockchip though, so I
> would need more
> info to avoid breaking that with a new patch. Are you btw sure this is the
> commit that breaks it?

Not as written they don't. Do you mean this:

if (params.type != IH_TYPE_MULTI && params.type != IH_TYPE_SCRIPT)

or should the ! have brackets after it?

You can build firefly-rk3288 and then run the mkimage command under
'Booting from an SD card' in README.rockchip:

./firefly-rk3288/tools/mkimage -T rksd -d \
firefly-rk3288/spl/u-boot-spl-dtb.bin out

assuming you built it with O=firefly-rk3288

>
> +               dfd = open(params.datafile, O_RDONLY | O_BINARY);
> +               if (dfd < 0) {
> +                       fprintf(stderr, "%s: Can't open %s: %s\n",
> +                               params.cmdname, params.datafile,
> +                               strerror(errno));
> +                       exit(EXIT_FAILURE);
> +               }

Regards,
Simon


More information about the U-Boot mailing list