Re: [RFC PATCH 2/8] rockchip: Add binman definitions for final images

[Alper Nebi Yasak <alpernebiyasak@gmail.com>]

On 27/04/2022 12:22, Johan Jonker wrote:
> [...]
> 
> There's a trend to solve everything in python.
> Don't have knowledge about python, but there's a 3rd image format for
> NAND [1]. Could someone advise how to fit the extra XXX padding option
> in the mkimage.py file? (Simon ?)

Binman's mkimage support is mostly a wrapper around the mkimage
executable, so that patch would need to be applied first. Assuming you
want to run something like the example in that patch:

> mkimage -n rk3066 -T rknand -d <tpl>:<spl> -X 16384,1 <out>

I think you'd only need something like the first patch of this series
for binman to get the multiple inputs working, and I expect a binman dtb
fragment like the following would then work for the above command:

    mkimage {
        args = "-n", CONFIG_SYS_SOC, "-T", "rknand", "-X", "16384,1";

        u-boot-tpl {
        };

        u-boot-spl {
        };
    };

> Could someone from Rockchip give info on whether the SD or SPI image
> format has similarity to the NAND image format to reuse code? (Kever ?)
> 
> Missing are u-boot documents on Rockchip file formats itself other then
> reading the /tools/rkxxx.c code.

(I don't know the formats' details, so can't comment on those.)

> Currently there's a small section about rkimage, rksd and rkspi in
> README.rockchip. Move that to rockchip.rst as well when busy with image
> formats in this serie?
> 
> Johan
> 
> ===
> 
> [1]
> [PATCH v4 11/19] rockchip: mkimage: add support for rockchip nand boot image
> https://lore.kernel.org/u-boot/45576df80a77657cb2245ea1d1fc2e69908a909d.1502974488.git.paweljarosz3691@gmail.com/