Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-13 Thread Stefan Roese
On 08/11/2012 06:20 PM, Mike Frysinger wrote: On Tuesday 05 June 2012 02:37:55 Stefan Roese wrote: --- /dev/null +++ b/drivers/bootcount/Makefile +COBJS-$(CONFIG_BFIN_CPU)+= bootcount_blackfin.o needs to be CONFIG_BLACKFIN Okay. +all:$(LIB) unused rule - delete Okay.

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-13 Thread Andreas Bießmann
Dear Stefan Roese, On 05.06.2012 08:37, 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 s...@denx.de Cc:

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-13 Thread Stefan Roese
Hi Andreas, On 08/13/2012 03:11 PM, Andreas Bießmann wrote: +LIB := $(obj)libbootcount.o + +COBJS-y += bootcount.o +COBJS-$(CONFIG_AT91SAM9XE) += bootcount_at91.o I tend to NAK this. Before it was available to all at91 processors (keep in mind nearly

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-13 Thread Andreas Bießmann
Hi Stefan, On 13.08.2012 15:37, Stefan Roese wrote: Hi Andreas, On 08/13/2012 03:11 PM, Andreas Bießmann wrote: +LIB:= $(obj)libbootcount.o + +COBJS-y+= bootcount.o +COBJS-$(CONFIG_AT91SAM9XE) += bootcount_at91.o I tend to NAK this. Before it was

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-13 Thread Stefan Roese
Hi Andreas, On 08/13/2012 03:48 PM, Andreas Bießmann wrote: +COBJS-y += bootcount.o +COBJS-$(CONFIG_AT91SAM9XE)+= bootcount_at91.o I tend to NAK this. Before it was available to all at91 processors (keep in mind nearly all at91 have this gpbr register). Now

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-13 Thread Andreas Bießmann
Hi Stefan, On 13.08.2012 16:51, Stefan Roese wrote: Hi Andreas, On 08/13/2012 03:48 PM, Andreas Bießmann wrote: +COBJS-y += bootcount.o +COBJS-$(CONFIG_AT91SAM9XE) += bootcount_at91.o I tend to NAK this. Before it was available to all at91 processors (keep

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-13 Thread Christian Riesch
Hi Stefan, On Monday, August 13, 2012, 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 s...@denx.de

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-11 Thread Mike Frysinger
On Tuesday 05 June 2012 02:37:55 Stefan Roese wrote: --- /dev/null +++ b/drivers/bootcount/Makefile +COBJS-$(CONFIG_BFIN_CPU) += bootcount_blackfin.o needs to be CONFIG_BLACKFIN +all: $(LIB) unused rule - delete --- /dev/null +++ b/include/bootcount.h +#ifdef

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-10 Thread Wolfgang Denk
Dear Stefan Roese, In message 1338878275-1918-1-git-send-email...@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

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

2012-06-05 Thread Stefan Roese
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 s...@denx.de Cc: Heiko Schocher h...@denx.de Cc: Valentin Longchamp

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-06-05 Thread Rob Herring
On 06/05/2012 01:37 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 s...@denx.de Cc: Heiko Schocher