[U-Boot] [PATCH v5] Consolidate bootcount code into drivers/bootcount

Wolfgang Denk wd at denx.de
Sat Sep 1 14:26:22 CEST 2012


Dear Stefan Roese,

In message <1345175741-18016-1-git-send-email-sr at denx.de> you 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>
> Tested-by: Valentin Longchamp <valentin.longchamp at keymile.com>
> Tested-by: Christian Riesch <christian.riesch at omicron.at>
> Acked-by: Rob Herring <rob.herring at calxeda.com>
> ---
> v5:
> - Removed bootcount target from arch/blackfin/cpu/Makefile
>   as pointer out by Mike (thanks)
> 
> v4:
> - Rebased against TOT
> - Addressed Mike's comments
> 
> v3:
> - Moved le-/be-accessors into header so that they now can be
>   used by all bootcount drivers.
> - Changed CONFIG_BOOTCOUNT_LE to CONFIG_SYS_BOOTCOUNT_LE
> - Enabled CONFIG_SYS_BOOTCOUNT_LE in highbank
> - Enabled CONFIG_SYS_BOOTCOUNT_SINGLEWORD in highbank
> 
> v2:
> - Added CONFIG_BOOTCOUNT_LE to bootcount_davinci.c and enabled it
>   in calimain.h to select little-endian accessors.
> 
>  Makefile                                           |  1 +
>  arch/arm/cpu/arm926ejs/at91/cpu.c                  | 26 --------
>  arch/arm/cpu/armv7/highbank/Makefile               |  2 +-
>  arch/arm/cpu/armv7/highbank/bootcount.c            | 36 -----------
>  arch/arm/cpu/ixp/cpu.c                             | 22 -------
>  arch/blackfin/cpu/Makefile                         |  1 -
>  arch/powerpc/lib/Makefile                          |  1 -
>  board/enbw/enbw_cmc/enbw_cmc.c                     | 29 ---------
>  board/keymile/km_arm/km_arm.c                      | 51 ---------------
>  board/omicron/calimain/calimain.c                  | 29 ---------
>  drivers/bootcount/Makefile                         | 45 ++++++++++++++
>  .../powerpc/lib => drivers/bootcount}/bootcount.c  | 25 ++++----
>  drivers/bootcount/bootcount_at91.c                 | 43 +++++++++++++
>  .../bootcount/bootcount_blackfin.c                 |  0
>  drivers/bootcount/bootcount_davinci.c              | 49 +++++++++++++++
>  drivers/bootcount/bootcount_ram.c                  | 72 ++++++++++++++++++++++
>  include/bootcount.h                                | 51 +++++++++++++++
>  include/configs/calimain.h                         |  1 +
>  include/configs/enbw_cmc.h                         |  1 +
>  include/configs/highbank.h                         |  2 +
>  include/configs/km/km_arm.h                        |  2 +
>  21 files changed, 282 insertions(+), 207 deletions(-)
>  delete mode 100644 arch/arm/cpu/armv7/highbank/bootcount.c
>  create mode 100644 drivers/bootcount/Makefile
>  rename {arch/powerpc/lib => drivers/bootcount}/bootcount.c (82%)
>  create mode 100644 drivers/bootcount/bootcount_at91.c
>  rename arch/blackfin/cpu/bootcount.c => drivers/bootcount/bootcount_blackfin.c (100%)
>  create mode 100644 drivers/bootcount/bootcount_davinci.c
>  create mode 100644 drivers/bootcount/bootcount_ram.c
>  create mode 100644 include/bootcount.h

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Microsoft Multimedia:
You have nice graphics, sound and animations when the system crashes.


More information about the U-Boot mailing list