public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stelian Pop <stelian@popies.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] AT91: Enable PLLB for USB
Date: Tue, 21 Oct 2008 23:20:44 +0200	[thread overview]
Message-ID: <1224624044.7536.12.camel@galileo> (raw)
In-Reply-To: <3efb10970810211054t5627284fkf62b2971bf451ca2@mail.gmail.com>

Le mardi 21 octobre 2008 ? 19:54 +0200, Remy Bohmer a ?crit :
> Hello Stelian,

Hi Remy,

> Are all your USB stick problems solved by now? Including that stick
> that did not work?

Nope. I still have one (kinda old) stick which does not work, on any
AT91 board. But I suspect the issues are no longer in the board/cpu
code, but in the core USB layer.

> I will try your patches tomorrow, but I have one question:
> > #define AT91_MAIN_CLOCK                18432000        /* 18.432 MHz crystal */
> >  #define AT91_MASTER_CLOCK      100000000       /* peripheral */
> >  #define AT91_CPU_CLOCK         200000000       /* cpu */
> 
> Are such nice rounded values possible with that unrounded crystal?
> 
> Looking at sam9261 I see:
> #define AT91_MAIN_CLOCK         198656000       /* from 18.432 MHz crystal */

This one is clearly wrong. 18.432 MHz is 18432000 Hz.

> #define AT91_MASTER_CLOCK       99328000        /* peripheral = main / 2

And the master clock is not main / 2, but PLLA / 2, where PLLA is
initialized by the bootstrap to (main * 0x60) / 9.

This gives:
	PLLA = 18432000 * 0x60 / 9 = 196608000 Hz
	MASTER = 196608000 / 2 = 98304000 Hz.

The defines in the header files contain rounded values, I don't think
it's too important to be precise here.

Stelian.
-- 
Stelian Pop <stelian@popies.net>

  reply	other threads:[~2008-10-21 21:20 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-21 16:07 [U-Boot] [PATCH ARM/AT91 0/4] AT91CAP9/AT91SAM9 bugfixes for 2008.12 Stelian Pop
2008-10-21 16:07 ` [U-Boot] [PATCH] AT91: Use AT91_CPU_CLOCK in displays Stelian Pop
2008-11-06 19:29   ` Jean-Christophe PLAGNIOL-VILLARD
2008-11-06 21:55     ` Stelian Pop
2008-11-06 22:51       ` Jean-Christophe PLAGNIOL-VILLARD
2008-10-21 16:07 ` [U-Boot] [PATCH] AT91: Enable PLLB for USB Stelian Pop
2008-10-21 17:54   ` Remy Bohmer
2008-10-21 21:20     ` Stelian Pop [this message]
2008-11-06 19:27   ` Jean-Christophe PLAGNIOL-VILLARD
2008-11-06 21:55     ` Stelian Pop
2008-11-06 22:52       ` Jean-Christophe PLAGNIOL-VILLARD
2008-10-21 16:08 ` [U-Boot] [PATCH] AT91: Replace AT91_BASE_EMAC by the board specific values Stelian Pop
2008-11-06 19:28   ` Jean-Christophe PLAGNIOL-VILLARD
2008-10-21 16:08 ` [U-Boot] [PATCH] AT91: Replace (undefined) AT91_ID_US* " Stelian Pop
2008-11-06 19:27   ` Jean-Christophe PLAGNIOL-VILLARD

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=1224624044.7536.12.camel@galileo \
    --to=stelian@popies.net \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox