Rob, On Monday 04 June 2012 15:03:27 Rob Herring wrote: > On 06/04/2012 07:38 AM, Stefan Roese wrote: > > This patch moves all bootcount implementations into a common > > directory: drivers/bootcount. The generic bootcount driver > > is now usable not only by powerpc platforms, but others as well. > > > > Signed-off-by: Stefan Roese <s...@denx.de> > > Cc: Heiko Schocher <h...@denx.de> > > Cc: Valentin Longchamp <valentin.longch...@keymile.com> > > Cc: Christian Riesch <christian.rie...@omicron.at> > > Cc: Manfred Rudigier <manfred.rudig...@omicron.at> > > Cc: Mike Frysinger <vap...@gentoo.org> > > Cc: Rob Herring <rob.herr...@calxeda.com> > > Cc: Reinhard Meyer <reinhard.me...@emk-elektronik.de> > > --- > > v2: > > - Added CONFIG_BOOTCOUNT_LE to bootcount_davinci.c and enabled it > > > > in calimain.h to select little-endian accessors. > > highbank is also LE. Why don't you use __BYTE_ORDER rather than a new > define?
I just noticed, that highbank only uses one lword as bootcounter storage. So CONFIG_SYS_BOOTCOUNT_SINGLEWORD needs to be set for the "generic" bootcount driver to work here. I'll fix this in v3 after a short delay for further review comments. Thanks, Stefan _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
