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

Tom Rini trini at ti.com
Thu Jul 3 21:17:59 CEST 2014


On Thu, Jul 03, 2014 at 09:14:01AM -0700, Simon Glass wrote:
> 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?

I think I like this unless there's a problem...

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140703/6ae56289/attachment.pgp>


More information about the U-Boot mailing list