public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] socfpga: missing DECLARE_GLOBAL_DATA_PTR in timer.c
@ 2015-04-08 12:12 Pavel Machek
  2015-04-08 12:43 ` Stefan Roese
  0 siblings, 1 reply; 2+ messages in thread
From: Pavel Machek @ 2015-04-08 12:12 UTC (permalink / raw)
  To: u-boot


for A in `find . -name "*.c"`; do if grep DECLARE_GLOBAL_DATA_PTR $A; then echo -n; else echo $A; fi; done

reveals a few files that probably should have DECLARE_GLOBAL_DATA_PTR.

This fixes it for arch/arm/cpu/armv7/socfpga/timer.c .

Signed-off-by: Pavel Machek <pavel@denx.de>

index 253cde3..2dd8376 100644
--- a/arch/arm/cpu/armv7/socfpga/timer.c
+++ b/arch/arm/cpu/armv7/socfpga/timer.c
@@ -8,6 +8,8 @@
 #include <asm/io.h>
 #include <asm/arch/timer.h>
 
+DECLARE_GLOBAL_DATA_PTR;
+
 #define TIMER_LOAD_VAL		0xFFFFFFFF
 
 static const struct socfpga_timer *timer_base = (void *)CONFIG_SYS_TIMERBASE;

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [U-Boot] socfpga: missing DECLARE_GLOBAL_DATA_PTR in timer.c
  2015-04-08 12:12 [U-Boot] socfpga: missing DECLARE_GLOBAL_DATA_PTR in timer.c Pavel Machek
@ 2015-04-08 12:43 ` Stefan Roese
  0 siblings, 0 replies; 2+ messages in thread
From: Stefan Roese @ 2015-04-08 12:43 UTC (permalink / raw)
  To: u-boot

On 08.04.2015 14:12, Pavel Machek wrote:
>
> for A in `find . -name "*.c"`; do if grep DECLARE_GLOBAL_DATA_PTR $A; then echo -n; else echo $A; fi; done
>
> reveals a few files that probably should have DECLARE_GLOBAL_DATA_PTR.
>
> This fixes it for arch/arm/cpu/armv7/socfpga/timer.c .
>
> Signed-off-by: Pavel Machek <pavel@denx.de>
>
> index 253cde3..2dd8376 100644
> --- a/arch/arm/cpu/armv7/socfpga/timer.c
> +++ b/arch/arm/cpu/armv7/socfpga/timer.c
> @@ -8,6 +8,8 @@
>   #include <asm/io.h>
>   #include <asm/arch/timer.h>
>
> +DECLARE_GLOBAL_DATA_PTR;
> +

Hmmm, "gd" is not used in this file at all. So not sure if we should add 
DECLARE_GLOBAL_DATA_PTR here.

Thanks,
Stefan

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-04-08 12:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-08 12:12 [U-Boot] socfpga: missing DECLARE_GLOBAL_DATA_PTR in timer.c Pavel Machek
2015-04-08 12:43 ` Stefan Roese

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox