[U-Boot] [RFC PATCH 32/44] image: Support signing of images

Simon Glass sjg at chromium.org
Sat Jan 5 22:50:12 CET 2013


Hi Marek,

On Sat, Jan 5, 2013 at 12:19 AM, Marek Vasut <marex at denx.de> wrote:
> Dear Simon Glass,
>
> [...]
>
>> +     int string_size;
>> +     int ret;
>> +
>> +     /*
>> +      * Get the current string size, before we update the FIT and add
>> +      * more
>> +      */
>> +     string_size = fdt_size_dt_strings(fit);
>> +
>> +     ret = fdt_setprop(fit, noffset, FIT_VALUE_PROP, value, value_len);
>> +     ret |= fdt_setprop_string(fit, noffset, "signer-name", "mkimage");
>> +     ret |= fdt_setprop_string(fit, noffset, "signer-version",
>> +                               PLAIN_VERSION);
>
> Can you really be ORR'ing a signed variable such as "ret" ?

It's pretty ugly, and it means that I don't get a proper error
message, so I will change it, particularly as this is only used by
mkimage and code size is not a concern.

Thanks very much for reviewing these patches, and for comments. The
image code is actually mostly pretty nice so it hasn't been difficult
to work with.

Regards,
Simon

> [...]
> Best regards,
> Marek Vasut


More information about the U-Boot mailing list