On Wed, Jan 28, 2026 at 06:23:20PM +0100, Marek Vasut wrote:

Thanks for taking my request and doing what you're suggesting should be
done.

[snip]
> -     for (node = fdt_first_subnode(fdt, images);
> -          node >= 0;
> -          node = fdt_next_subnode(fdt, node)) {
> -             const char *data;
> -             int len;
> +     fdt_for_each_subnode(node, fdt, images) {
> +             const char *data, *type;
> +             int pl;
> +
> +             if (params->bl_len)
> +                     align_size = 8;//params->bl_len;

This should be params->bl_len, not 8.

> +             else {
> +                     type = fdt_getprop(fdt, node, FIT_TYPE_PROP, &pl);
> +                     if (type && pl == sizeof("flat_dt") && !memcmp(type, 
> "flat_dt", pl))
> +                             align_size = 8; // or params->bl_len rounded up 
> to 8 ?
> +                     else    /* Default alignment to 4 Bytes */
> +                             align_size = 8;

If not params->bl_len, then 8. If params->bl_len then whatever the user
asked for is what they get.

Then post it on its own so patchwork picks it up and I can grab it
please.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to