All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 01/10] am33xx: Move V_OSCK/V_SCLK to <asm/arch-am33xx/clocks_am33xx.h>
Date: Mon, 5 Aug 2013 11:53:17 -0400	[thread overview]
Message-ID: <20130805155317.GO5164@bill-the-cat> (raw)
In-Reply-To: <20130805154029.GN5164@bill-the-cat>

On Mon, Aug 05, 2013 at 11:40:29AM -0400, Tom Rini wrote:
> On Mon, Aug 05, 2013 at 05:26:56PM +0200, Lars Poeschel wrote:
> > Am Montag, 5. August 2013, 17:17:32 schrieb Tom Rini:
> > > 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?
> > 
> > Sorry, but yes, thats right. pcm051 is delivered with 25000000hz crystal. 
> > Main reason was to be able to boot from ethernet right from the ROM 
> > bootloader.
> 
> Alright thanks, I'll go re-work that part of the series.

Dropped from v2.

-- 
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/b6c6e249/attachment.pgp>

  reply	other threads:[~2013-08-05 15:53 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-02 20:26 [U-Boot] [PATCH 00/10] Introduce common config file for TI ARMv7 platforms Tom Rini
2013-08-02 20:26 ` [U-Boot] [PATCH 01/10] am33xx: Move V_OSCK/V_SCLK to <asm/arch-am33xx/clocks_am33xx.h> Tom Rini
2013-08-03  4:41   ` Heiko Schocher
2013-08-05 15:17     ` Tom Rini
2013-08-05 15:26       ` Lars Poeschel
2013-08-05 15:40         ` Tom Rini
2013-08-05 15:53           ` Tom Rini [this message]
2013-08-02 20:26 ` [U-Boot] [PATCH 02/10] am33xx: CONFIG_DMA_COHERENT defines are unused, remove Tom Rini
2013-08-06 18:25   ` Dan Murphy
2013-08-02 20:26 ` [U-Boot] [PATCH 03/10] am33xx: Stop using PHYS_DRAM_1 define Tom Rini
2013-08-02 20:26 ` [U-Boot] [PATCH 04/10] am335x_evm: Use default baud rate table Tom Rini
2013-08-02 20:26 ` [U-Boot] [PATCH 05/10] arm: spl: For Falcon Mode, set a default machid of ~0 Tom Rini
2013-08-05  3:53   ` Heiko Schocher
2013-08-06 18:28   ` Dan Murphy
2013-08-06 19:10     ` Tom Rini
2013-08-06 20:03       ` Dan Murphy
2013-08-02 20:26 ` [U-Boot] [PATCH 06/10] am335x_evm: Bring in 'boot_fdt' logic from i.MX Tom Rini
2013-08-02 20:26 ` [U-Boot] [PATCH 07/10] TI:am33xx: Create common config files for TI ARMv7 platforms, and AM33xx Tom Rini
2013-08-06 18:33   ` Dan Murphy
2013-08-06 19:11     ` Tom Rini
2013-08-02 20:26 ` [U-Boot] [PATCH 08/10] TI:omap5: Convert to ti_armv7_common.h Tom Rini
2013-08-06 18:37   ` Dan Murphy
2013-08-06 19:13     ` Tom Rini
2013-08-06 19:24       ` Dan Murphy
2013-08-06 18:38   ` Dan Murphy
2013-08-02 20:26 ` [U-Boot] [PATCH 09/10] TI:armv7: Enable CONFIG_CMD_SPI Tom Rini
2013-08-06 18:34   ` Dan Murphy
2013-08-06 18:50     ` Jagan Teki
2013-08-06 18:55       ` Jagan Teki
2013-08-06 19:03         ` Tom Rini
2013-08-06 19:10           ` Jagan Teki
2013-08-02 20:26 ` [U-Boot] [PATCH 10/10] TI:armv7: Enable CONFIG_CMD_GPIO Tom Rini
2013-08-06 18:34   ` Dan Murphy
2013-08-03  4:51 ` [U-Boot] [PATCH 00/10] Introduce common config file for TI ARMv7 platforms Heiko Schocher
2013-08-05 15:39   ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130805155317.GO5164@bill-the-cat \
    --to=trini@ti.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.