[U-Boot] Unable to boot the new FIT image

[Simon Glass]

Hi Harsha,

On 21 May 2014 06:46, Harsha Kiran <harshakiran443 at gmail.com> wrote:
> Hi Simon,
>
> I have added the next patch in the series i.e., (....../339610/) But the
> result is still the same. This particular patch just created a device tree
> In the arch/arm/dts folder For the beaglebone. I have already tried creating
> for am335x-evmsk and was able to build the u-boot-dtb.bin.
>
> I have noticed that MLO is configured to read only the u-boot.img not any
> other binaries (eg U-boot.bin or u-boot-dtb.bin).
> It gives me the following error..
> spl_load_image_fat: error reading image u-boot.img, err - -1 Which part of
> the configuration I should change so that MLO tries to read the
> u-boot-dtb.bin as the third stage bootloader??
> I tried loading MLO->u-boot.img first, but this time it reads the u-boot.img
> but gives the following error..
>
> reading u-boot.img
> reading u-boot.img
> No valid FDT found - please append one to U-Boot binary, use u-boot-dtb.bin
> or define CONFIG_OF_EMBED. For sandbox, use -d <file.dtb> ### ERROR ###
> Please RESET the board ###
>
> I tried applying the next patch in the series to add that extra padding to
> the image size (ie., .......org/patch/339611) But could not successfully
> apply the patch..
> harsha at harshakiran_kasha:/abb/Experiments/u-boot$ sudo patch -p1 -R <
> patch3.patch patching file tools/fit_image.c Hunk #1 FAILED at 35.
> Hunk #2 FAILED at 60.
> Hunk #3 succeeded at 38 (offset -137 lines).
> Hunk #4 FAILED at 210.
> Hunk #5 succeeded at 115 (offset -123 lines).
> 3 out of 5 hunks FAILED -- saving rejects to file tools/fit_image.c.rej
> patching file tools/image-host.c

You will need to resolve these problems and manually apply the patch.
Probably you don't need sudo.

>
> and then the compilation gave the following errors..
> harsha at harshakiran_kasha:/abb/Experiments/u-boot$ sudo make ARCH=arm
> CROSS_COMPILE=/abb/compilers/gcc-linaro-arm-linux-gnueabihf-4.7-2013.04-20130415_linux/bin/arm-linux-gnueabihf-
> -j8 DEV_TREE_BIN=./am335x-boneblack.dtb
>   CHK     include/config/uboot.release
>   CHK     include/generated/timestamp_autogenerated.h
>   UPD     include/generated/timestamp_autogenerated.h
>   CHK     include/generated/version_autogenerated.h
>   HOSTCC  tools/dumpimage.o
>   HOSTCC  tools/fit_image.o
>   HOSTCC  tools/image-host.o
>   HOSTCC  tools/mkenvimage.o
>   HOSTCC  tools/mkimage.o
> tools/fit_image.c: In function ‘fit_handle_file’:
> tools/fit_image.c:74:3: error: ‘destfd’ undeclared (first use in this
> function)   destfd = mmap_fdt(params->cmdname, params->keydest,
> tools/fit_image.c:74:3: note: each undeclared identifier is reported only
> once for each function it appears in
> tools/fit_image.c:75:8: error: ‘dest_blob’ undeclared (first use in this
> function)      &dest_blob, &sbuf, 1);
>         ^
> tools/fit_image.c:75:20: error: ‘sbuf’ undeclared (first use in this
> function)
>        &dest_blob, &sbuf, 1);
>                     ^
> tools/fit_image.c:78:3: error: ‘destfd_size’ undeclared (first use in this
> function)
>    destfd_size = sbuf.st_size;
>    ^
> tools/fit_image.c:81:2: error: ‘tfd’ undeclared (first use in this function)
>   tfd = mmap_fdt(params->cmdname, tmpfile, &ptr, &sbuf, 1);
>   ^
> tools/fit_image.c:81:44: error: ‘ptr’ undeclared (first use in this
> function)
>   tfd = mmap_fdt(params->cmdname, tmpfile, &ptr, &sbuf, 1);
>                                             ^
> tools/fit_image.c:98:3: error: label ‘err_add_timestamp’ used but not
> defined
>    goto err_add_timestamp;
>    ^
> tools/fit_image.c:91:3: error: label ‘err_add_hashes’ used but not defined
>    goto err_add_hashes;
>    ^
> tools/fit_image.c:83:3: error: label ‘err_mmap’ used but not defined
>    goto err_mmap;
>    ^
> tools/fit_image.c:77:4: error: label ‘err_keydest’ used but not defined
> goto err_keydest;
>     ^
> tools/fit_image.c:42:6: warning: unused variable ‘ret’ [-Wunused-variable]
>   int ret;
>       ^
> tools/fit_image.c:41:9: warning: unused variable ‘size_inc’
> [-Wunused-variable]
>   size_t size_inc;
>          ^
> make[1]: *** [tools/fit_image.o] Error 1
> make[1]: *** Waiting for unfinished jobs....
> make: *** [tools] Error 2
>
> Sorry for deleting the previous messages. U-boot mailing list is rejecting
> my emails If I include them

The objective is to get u-boot-dtb.bin packed into the .img file. I
had this booting fine, but it was some weeks ago. I will give it
another go when I get back from my trip next week.

Regards.
Simon