[U-Boot] [PATCH v2] Consolidate bootcount code into drivers/bootcount
Stefan Roese
sr at denx.de
Mon Jun 4 15:42:22 CEST 2012
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 <sr at denx.de>
> > Cc: Heiko Schocher <hs at denx.de>
> > Cc: Valentin Longchamp <valentin.longchamp at keymile.com>
> > Cc: Christian Riesch <christian.riesch at omicron.at>
> > Cc: Manfred Rudigier <manfred.rudigier at omicron.at>
> > Cc: Mike Frysinger <vapier at gentoo.org>
> > Cc: Rob Herring <rob.herring at calxeda.com>
> > Cc: Reinhard Meyer <reinhard.meyer at 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
More information about the U-Boot
mailing list