Hi, On 3 July 2014 00:03, Heiko Schocher <h...@denx.de> wrote: > > Hello Dirk, > > Am 03.07.2014 08:27, schrieb dirk.eib...@gdsys.cc: > >> From: Dirk Eibach<dirk.eib...@gdsys.cc> >> >> sha256 has some beefy memory footprint. >> Make it optional for constrained systems. >> >> >> Signed-off-by: Dirk Eibach<dirk.eib...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
