[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