Hi, On Mon, Jun 4, 2012 at 3:14 PM, Stefan Roese <s...@denx.de> wrote: > 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
Thanks a lot! I will test it on the calimain board. >> > >> > 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? > I don't think this is necessary. As long as a board uses its native endianess to store the boot counter, nothing special is needed. Regards, Christian >> 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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
