[U-Boot] [PATCH 01/10] am33xx: Move V_OSCK/V_SCLK to <asm/arch-am33xx/clocks_am33xx.h>

[Heiko Schocher]

Hello Tom,

Am 02.08.2013 22:26, schrieb Tom Rini:
> This detail belongs in the arch header file, given how we are structured
> today at least.
>
> Signed-off-by: Tom Rini<trini at ti.com>
> ---
>   arch/arm/include/asm/arch-am33xx/clocks_am33xx.h |    4 ++++
>   include/configs/igep0033.h                       |    4 ----
>   include/configs/pcm051.h                         |    4 ----
>   include/configs/ti814x_evm.h                     |    4 ----
>   4 files changed, 4 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm/include/asm/arch-am33xx/clocks_am33xx.h b/arch/arm/include/asm/arch-am33xx/clocks_am33xx.h
> index 80e1899..3becb98 100644
> --- a/arch/arm/include/asm/arch-am33xx/clocks_am33xx.h
> +++ b/arch/arm/include/asm/arch-am33xx/clocks_am33xx.h
> @@ -11,6 +11,10 @@
>   #ifndef _CLOCKS_AM33XX_H_
>   #define _CLOCKS_AM33XX_H_
>
> +/* Clock Defines */
> +#define V_OSCK				24000000  /* Clock output from T2 */
> +#define V_SCLK				(V_OSCK)
> +

Hmm.. look at the chunk for the pcm051 board ...

>   /* MAIN PLL Fdll = 550 MHz, by default */
>   #ifndef CONFIG_SYS_MPUCLK
>   #define CONFIG_SYS_MPUCLK	550
[...]
> diff --git a/include/configs/pcm051.h b/include/configs/pcm051.h
> index 9b16c47..7073501 100644
> --- a/include/configs/pcm051.h
> +++ b/include/configs/pcm051.h
> @@ -102,10 +102,6 @@
>   		"fi;" \
>   	"fi;" \
>
> -/* Clock Defines */
> -#define V_OSCK				25000000  /* Clock output from T2 */
> -#define V_SCLK				(V_OSCK)
> -

... this defines 25000000 not 24000000 for V_OSCK ...

>   #define CONFIG_CMD_ECHO
>
>   /* max number of command args */
> diff --git a/include/configs/ti814x_evm.h b/include/configs/ti814x_evm.h
> index eac5ad0..ceee43b 100644
> --- a/include/configs/ti814x_evm.h
> +++ b/include/configs/ti814x_evm.h
> @@ -100,10 +100,6 @@
>   		"fi;" \
>   	"fi;" \
>
> -/* Clock Defines */
> -#define V_OSCK			24000000	/* Clock output from T2 */
> -#define V_SCLK			(V_OSCK>>  1)
> -
>   #define CONFIG_CMD_ECHO
>
>   /* max number of command args */

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany