[Buildroot] [PATCH v2 4/7] altera: update text about the mkpimage

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Hello,

On Tue, 20 Oct 2015 13:32:22 +0200, Jan Viktorin wrote:
> As we've integrated the mkpimage, we just leave some notes
> about other available tools and give an advice how to enable
> this feature in Buildroot.
> 
> Signed-off-by: Jan Viktorin <viktorin@rehivetech.com>
> ---
>  board/altera/readme.txt | 16 +++++++++-------
>  1 file changed, 9 insertions(+), 7 deletions(-)
> 
> diff --git a/board/altera/readme.txt b/board/altera/readme.txt
> index ddd9e03..fdd7316 100644
> --- a/board/altera/readme.txt
> +++ b/board/altera/readme.txt
> @@ -75,21 +75,23 @@ After building, you should obtain this tree:
>      ??? socfpga.dtb
>      ??? u-boot.img
>      ??? u-boot-spl.bin
> +    ??? u-boot-spl.bin.crc
>      ??? uImage
>  
>  
>  Signing the Preloader
>  ---------------------
>  
> -*** BEWARE ****
> -The u-boot-spl.bin must be signed using the Altera's tool "mkpimage".
> -This tool comes as a part of the Altera development environnment (SoC EDS).
> -A fork of this tool have been done by Maxime Hadjinlian and can be found here :
> -https://github.com/maximeh/mkpimage
> +The u-boot-spl.bin needs a checksum computed by the Altera's tool "mkpimage".
> +This tool comes as a part of the Altera development environnment (SoC EDS),
> +as a standalone tool https://github.com/maximeh/mkpimage and as a part of the
> +Barebox project. The boards do not boot without the checksum.
>  
> -Remember that without signing the u-boot-spl.bin, the board will not boot !!!
> +The Buildroot integrates the mkpimage and you can find it in the configuration.
> +If you check: "Bootloaders/U-Boot/CRC SPL image for SoC FPGA" the build system
> +calls mkpimage on the SPL for you automatically.
>  
> -  $ mkpimage u-boot-spl.bin -o u-boot-spl-signed.bin
> +  $ mkpimage u-boot-spl.bin -o u-boot-spl.bin.crc

This is the wrong change. Instead, you should just modify the two
Altera defconfigs we have to use the new feature, and remove all that
text since the final bootloader image is now automatically generated by
Buildroot. You can leave a note like "The final U-Boot image with the
Altera required CRC is automatically generated by Buildroot using
mkpimage", but all the rest is useless once the defconfigs are changed.

I'll mark this patch as Changes Requested in our patch tracking system.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com