From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Thu, 31 May 2012 12:42:12 +0200
Subject: [U-Boot] [PATCH] bootcount: Move bootcount.c from
 arch/powerpc/lib to lib
In-Reply-To: <1338454713-16211-1-git-send-email-sr@denx.de>
References: <1338454713-16211-1-git-send-email-sr@denx.de>
Message-ID: <4FC74B04.4070409@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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@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