On 03. 12. 18 16:37, Michael Tretter wrote:
> The FSBL copies "Total Partition Word Length" * 4 bytes from the boot.bin,
> which implies that the partition size is 4 byte aligned. When writing the
> partition, mkimage calculates "Total Partition Word Length" by dividing
> the size by 4. This implicitly cuts unaligned bytes at the end of the
> added binary.
>
> Instead of rounding down, the size must be round up to 4 bytes and the
> binary padded accordingly.
>
> Signed-off-by: Michael Tretter <m.tret...@pengutronix.de>
> ---
> v2:
> - do not realloc() data but malloc() and memcpy() new data
> ---
> tools/zynqmpbif.c | 15 ++++++++++++---
> 1 file changed, 12 insertions(+), 3 deletions(-)
>
> diff --git a/tools/zynqmpbif.c b/tools/zynqmpbif.c
> index 6c8f66055d..a33c15e1f0 100644
> --- a/tools/zynqmpbif.c
> +++ b/tools/zynqmpbif.c
> @@ -319,16 +319,25 @@ static int bif_add_pmufw(struct bif_entry *bf, const
> char *data, size_t len)
> static int bif_add_part(struct bif_entry *bf, const char *data, size_t len)
> {
> size_t parthdr_offset = 0;
> + size_t len_padded = ROUND(len, 4);
> +
> struct partition_header parthdr = {
> - .len_enc = cpu_to_le32(len / 4),
> - .len_unenc = cpu_to_le32(len / 4),
> - .len = cpu_to_le32(len / 4),
> + .len_enc = cpu_to_le32(len_padded / 4),
> + .len_unenc = cpu_to_le32(len_padded / 4),
> + .len = cpu_to_le32(len_padded / 4),
> .entry_point = cpu_to_le64(bf->entry),
> .load_address = cpu_to_le64(bf->load),
> };
> int r;
> uint32_t csum;
>
> + if (len < len_padded) {
> + char *newdata = malloc(len_padded);
> + memcpy(newdata, data, len);
> + memset(newdata + len, 0, len_padded - len);
> + data = newdata;
> + }
> +
> if (bf->flags & (1ULL << BIF_FLAG_PMUFW_IMAGE))
> return bif_add_pmufw(bf, data, len);
>
>
Applied.
M
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
