[U-Boot] [PATCH] bootcount: Move bootcount.c from arch/powerpc/lib to lib

[Heiko Schocher]

Hello Stefan,

On 31.05.2012 10:58, Stefan Roese wrote:
> By moving this code to a non-platform specific directory, we can use
> it on other architectures as well (e.g. ARM).
>
> Signed-off-by: Stefan Roese<sr at denx.de>
> ---
>   arch/powerpc/lib/Makefile             |    1 -
>   lib/Makefile                          |    1 +
>   {arch/powerpc/lib =>  lib}/bootcount.c |    0
>   3 files changed, 1 insertion(+), 1 deletion(-)
>   rename {arch/powerpc/lib =>  lib}/bootcount.c (100%)
>
> diff --git a/arch/powerpc/lib/Makefile b/arch/powerpc/lib/Makefile
> index cdd62a2..965f9ea 100644
> --- a/arch/powerpc/lib/Makefile
> +++ b/arch/powerpc/lib/Makefile
> @@ -46,7 +46,6 @@ SOBJS-y	+= reloc.o
>   COBJS-$(CONFIG_BAT_RW) += bat_rw.o
>   COBJS-y	+= board.o
>   COBJS-y	+= bootm.o
> -COBJS-$(CONFIG_BOOTCOUNT_LIMIT) += bootcount.o
>   COBJS-y	+= cache.o
>   COBJS-y	+= extable.o
>   COBJS-y	+= interrupts.o
> diff --git a/lib/Makefile b/lib/Makefile
> index 663d185..9500763 100644
> --- a/lib/Makefile
> +++ b/lib/Makefile
> @@ -34,6 +34,7 @@ COBJS-$(CONFIG_BZIP2) += bzlib_crctable.o
>   COBJS-$(CONFIG_BZIP2) += bzlib_decompress.o
>   COBJS-$(CONFIG_BZIP2) += bzlib_randtable.o
>   COBJS-$(CONFIG_BZIP2) += bzlib_huffman.o
> +COBJS-$(CONFIG_BOOTCOUNT_LIMIT) += bootcount.o
>   COBJS-$(CONFIG_USB_TTY) += circbuf.o
>   COBJS-y += crc7.o
>   COBJS-y += crc16.o
> diff --git a/arch/powerpc/lib/bootcount.c b/lib/bootcount.c
> similarity index 100%
> rename from arch/powerpc/lib/bootcount.c
> rename to lib/bootcount.c

If we do this, we should try to move all bootcounter implementations
to this file, see:

[hs at pollux u-boot]$ grep -lr bootcount_store .
./common/main.c
./arch/powerpc/lib/bootcount.c
./arch/arm/cpu/arm926ejs/at91/cpu.c
./arch/arm/cpu/armv7/highbank/bootcount.c
./arch/arm/cpu/ixp/cpu.c
./arch/blackfin/cpu/bootcount.c
./board/enbw/enbw_cmc/enbw_cmc.c
./board/omicron/calimain/calimain.c
./board/keymile/km_arm/km_arm.c
./include/common.h
[hs at pollux u-boot]$

and should provide a possibility to add board specific code,
for example needed for the

board/keymile/km_arm/km_arm.c boards.

Have you done a "MAKEALL arm"? I think (didn;t tried it) this
would fail with your patch ...

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany