[U-Boot] [PATCH v3 13/13] fit: make sha256 support optional

Simon Glass sjg at chromium.org
Thu Jul 3 18:14:01 CEST 2014


Hi,

On 3 July 2014 00:03, Heiko Schocher <hs at denx.de> wrote:
>
> Hello Dirk,
>
> Am 03.07.2014 08:27, schrieb dirk.eibach at gdsys.cc:
>
>> From: Dirk Eibach<dirk.eibach at gdsys.cc>
>>
>> sha256 has some beefy memory footprint.
>> Make it optional for constrained systems.
>>
>>
>> Signed-off-by: Dirk Eibach<dirk.eibach at gdsys.cc>
>> ---
>>
>> Changes in v3: None
>> Changes in v2:
>> - make sha256 support optional
>>
>>   include/configs/dlvision-10g.h | 1 +
>>   include/configs/io.h           | 1 +
>>   include/configs/iocon.h        | 1 +
>>   include/configs/neo.h          | 1 +
>>   include/image.h                | 5 +++++
>>   5 files changed, 9 insertions(+)
>
>
> Sorry, some nitpick. As you introduce here the new define "CONFIG_FIT_DISABLE_SHA256",
> can you please add a short description in the README, thanks!

I wonder if it would be better to make the option off by default? You
could perhaps use the existing CONFIG_SHA256 option, and check in
image.h to enable/disable support.

Heiko what do you think?

Regards.
Simon


More information about the U-Boot mailing list