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

[Tom Rini <trini@ti.com>]

On Sat, Aug 03, 2013 at 06:41:32AM +0200, Heiko Schocher wrote:
> 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@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 ...

Oh hell, Lars, is that right and you guys have a different clock
frequency than the reference platforms?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130805/b9367a27/attachment.pgp>