Re: [U-Boot] [PATCH] ARM/VExpress: fix timer address for TC2 and other VExpress models

2013-12-03 Thread Andre Przywara

On 11/30/2013 10:20 AM, Ian Campbell wrote:

On Tue, 2013-11-26 at 15:20 +0100, Andre Przywara wrote:

Commit v2013.10-189-gb3a7f22 breaks u-boot on the VExpress TC2, since
the hardcoded value for SP804 timer address is wrong on Versatile
Express boards using the extended memory map.
Replace this value with an existing macro make it work on both sets of
machines.

Signed-off-by: Andre Przywara 


I sent out a similar patch a couple of weeks back:
http://lists.denx.de/pipermail/u-boot/2013-November/167316.html

I don't mind which one goes in, so for this one:
Acked-by: Ian Campbell 


Thanks Ian and sorry for not searching the list more carefully.

Tom, Albert,

can someone please commit this? This is a serious regression against 
2013.10 and reliably hangs u-boot before showing the prompt.


Thanks,
Andre.


---
  include/configs/vexpress_common.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/vexpress_common.h 
b/include/configs/vexpress_common.h
index 6da5e8f..7e78f8a 100644
--- a/include/configs/vexpress_common.h
+++ b/include/configs/vexpress_common.h
@@ -132,7 +132,7 @@
  #define VEXPRESS_FLASHPROG_FLVPPEN(1 << 0)

  #define CONFIG_SYS_TIMER_RATE 100
-#define CONFIG_SYS_TIMER_COUNTER   (0x10011000 + 0x4)
+#define CONFIG_SYS_TIMER_COUNTER   (V2M_TIMER01 + 0x4)
  #define CONFIG_SYS_TIMER_COUNTS_DOWN

  /* SMSC9115 Ethernet from SMSC9118 family */





___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] ARM/VExpress: fix timer address for TC2 and other VExpress models

2013-11-30 Thread Ian Campbell
On Tue, 2013-11-26 at 15:20 +0100, Andre Przywara wrote:
> Commit v2013.10-189-gb3a7f22 breaks u-boot on the VExpress TC2, since
> the hardcoded value for SP804 timer address is wrong on Versatile
> Express boards using the extended memory map.
> Replace this value with an existing macro make it work on both sets of
> machines.
> 
> Signed-off-by: Andre Przywara 

I sent out a similar patch a couple of weeks back:
http://lists.denx.de/pipermail/u-boot/2013-November/167316.html

I don't mind which one goes in, so for this one:
Acked-by: Ian Campbell 

Thanks,
Ian.

> ---
>  include/configs/vexpress_common.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/configs/vexpress_common.h 
> b/include/configs/vexpress_common.h
> index 6da5e8f..7e78f8a 100644
> --- a/include/configs/vexpress_common.h
> +++ b/include/configs/vexpress_common.h
> @@ -132,7 +132,7 @@
>  #define VEXPRESS_FLASHPROG_FLVPPEN   (1 << 0)
>  
>  #define CONFIG_SYS_TIMER_RATE100
> -#define CONFIG_SYS_TIMER_COUNTER (0x10011000 + 0x4)
> +#define CONFIG_SYS_TIMER_COUNTER (V2M_TIMER01 + 0x4)
>  #define CONFIG_SYS_TIMER_COUNTS_DOWN
>  
>  /* SMSC9115 Ethernet from SMSC9118 family */


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] ARM/VExpress: fix timer address for TC2 and other VExpress models

2013-11-26 Thread Andre Przywara
Commit v2013.10-189-gb3a7f22 breaks u-boot on the VExpress TC2, since
the hardcoded value for SP804 timer address is wrong on Versatile
Express boards using the extended memory map.
Replace this value with an existing macro make it work on both sets of
machines.

Signed-off-by: Andre Przywara 
---
 include/configs/vexpress_common.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/vexpress_common.h 
b/include/configs/vexpress_common.h
index 6da5e8f..7e78f8a 100644
--- a/include/configs/vexpress_common.h
+++ b/include/configs/vexpress_common.h
@@ -132,7 +132,7 @@
 #define VEXPRESS_FLASHPROG_FLVPPEN (1 << 0)
 
 #define CONFIG_SYS_TIMER_RATE  100
-#define CONFIG_SYS_TIMER_COUNTER   (0x10011000 + 0x4)
+#define CONFIG_SYS_TIMER_COUNTER   (V2M_TIMER01 + 0x4)
 #define CONFIG_SYS_TIMER_COUNTS_DOWN
 
 /* SMSC9115 Ethernet from SMSC9118 family */
-- 
1.7.12.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot