[U-Boot] [PATCH] arm: mvebu: timer.c: Explicitly move "init_done" var to data section

As reported by Simon Guinot, commit ade741b3 "arm: mvebu: Call timer_init early before PHY and DDR init" breaks Kirkwood platforms. As the static variable "init_done" is not available at that early boot time. This patch moves it to explicitly to the data section, making it available at that time.
Signed-off-by: Stefan Roese sr@denx.de Reported-by: Simon Guinot simon.guinot@sequanux.org Cc: Luka Perkov luka.perkov@sartura.hr --- arch/arm/mach-mvebu/timer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-mvebu/timer.c b/arch/arm/mach-mvebu/timer.c index c516c41..5449a89 100644 --- a/arch/arm/mach-mvebu/timer.c +++ b/arch/arm/mach-mvebu/timer.c @@ -41,7 +41,7 @@ #define timestamp gd->arch.tbl #define lastdec gd->arch.lastinc
-static int init_done; +static int init_done __attribute__((section(".data"))) = 0;
/* Timer reload and current value registers */ struct kwtmr_val {

On Thu, Sep 03, 2015 at 12:47:07PM +0200, Stefan Roese wrote:
As reported by Simon Guinot, commit ade741b3 "arm: mvebu: Call timer_init early before PHY and DDR init" breaks Kirkwood platforms. As the static variable "init_done" is not available at that early boot time. This patch moves it to explicitly to the data section, making it available at that time.
Signed-off-by: Stefan Roese sr@denx.de Reported-by: Simon Guinot simon.guinot@sequanux.org Cc: Luka Perkov luka.perkov@sartura.hr
Tested-by: Simon Guinot simon.guinot@sequanux.org
Thanks,
Simon
arch/arm/mach-mvebu/timer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-mvebu/timer.c b/arch/arm/mach-mvebu/timer.c index c516c41..5449a89 100644 --- a/arch/arm/mach-mvebu/timer.c +++ b/arch/arm/mach-mvebu/timer.c @@ -41,7 +41,7 @@ #define timestamp gd->arch.tbl #define lastdec gd->arch.lastinc
-static int init_done; +static int init_done __attribute__((section(".data"))) = 0;
/* Timer reload and current value registers */ struct kwtmr_val { -- 2.5.1
U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
participants (2)
-
Simon Guinot
-
Stefan Roese