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.
Yes, sure. I could move those inline functions to a header, so that they can be used by the other "drivers" as well. Okay? > Why don't you use __BYTE_ORDER rather than a new > define? Unfortunately not. There are LE platforms that use BE accessors for the bootcounter already (Davinci enbw_cmc). Mostly historical reasons I assume, since the original bootcount implementation was powerpc specific with those be32() functions. Thanks, Stefan _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot