[U-Boot] [PATCH v2] at91: change CONFIG_SYS_HZ to 1000

[Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>]

On 02:58 Wed 25 Feb     , Detlef Vollmann wrote:
>  Change at91 CPUs based on arm926ejs to return milliseconds
>  from get_timer and get_ticks.  Also changes in the value of
>  CONFIG_SYS_HZ to 1000 in all board configs using these CPUs.
>  This will not compile on boards using these CPUs with a
>  different value for CONFIG_SYS_HZ.
> 
> 
> Signed-off-by: Detlef Vollmann <dv@vollmann.ch>
> ---
>  cpu/arm926ejs/at91/timer.c      |   37 +++++++++++++++++++++++++++++++++----
>  include/configs/afeb9260.h      |    2 +-
>  include/configs/at91cap9adk.h   |    2 +-
>  include/configs/at91sam9260ek.h |    2 +-
>  include/configs/at91sam9261ek.h |    2 +-
>  include/configs/at91sam9263ek.h |    2 +-
>  include/configs/at91sam9rlek.h  |    2 +-
>  7 files changed, 39 insertions(+), 10 deletions(-)
> 
> diff --git a/cpu/arm926ejs/at91/timer.c b/cpu/arm926ejs/at91/timer.c
> index fec545b..0a38a4f 100644
> --- a/cpu/arm926ejs/at91/timer.c
> +++ b/cpu/arm926ejs/at91/timer.c
> @@ -30,6 +30,19 @@
>  #include <asm/arch/io.h>
>  
>  /*
> + * This code essentially ignores the settings of AT91_MASTER_CLOCK
> + * and CONFIG_SYS_HZ, so we just bail out if they're not set to
> + * the canonical values.
> + */
> +#if (CONFIG_SYS_HZ != 1000)
> +#error "CONFIG_SYS_HZ must be set to 1000 for this CPU"
> +#endif
no need please remove
> +/* We leave some leeway for the master clock... */
> +#if (AT91_MASTER_CLOCK < 85000000) || (AT91_MASTER_CLOCK > 115000000)
> +#error "AT91_MASTER_CLOCK must be set to 100000000 for this CPU"
> +#endif
why?

Best Regards,
J.