Message ID | 1441277227-1656-1-git-send-email-sr@denx.de |
---|---|
State | Accepted |
Delegated to: | Stefan Roese |
Headers | show |
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
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 {
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(-)