From: "Alexander Shiyan" <shc_work@mail.ru>
To: "Jean-Christophe PLAGNIOL-VILLARD" <plagnioj@jcrosoft.com>
Cc: barebox@lists.infradead.org
Subject: Re[2]: [PATCH v2 3/9] ARM: clps711x: Adds config option for CPU PLL multiplier
Date: Wed, 13 Feb 2013 16:23:25 +0400 [thread overview]
Message-ID: <1360758205.278703573@f188.mail.ru> (raw)
In-Reply-To: <20130213120224.GM19322@game.jcrosoft.org>
...
> > diff --git a/arch/arm/boards/clep7212/lowlevel.c b/arch/arm/boards/clep7212/lowlevel.c
> > index b7d6d1d..fcf8285 100644
> > --- a/arch/arm/boards/clep7212/lowlevel.c
> > +++ b/arch/arm/boards/clep7212/lowlevel.c
> > @@ -14,9 +14,13 @@
> >
> > #include <mach/clps711x.h>
> >
> > +#if (CONFIG_CLPS711X_CPU_PLL_MULT < 20) || (CONFIG_CLPS711X_CPU_PLL_MULT > 50)
> > +# error "CPU PLL multiplier out of range"
> > +#endif
> > +
> > void __naked __bare_init barebox_arm_reset_vector(void)
> > {
> > arm_cpu_lowlevel_init();
> >
> > - clps711x_barebox_entry();
> > + clps711x_barebox_entry(CONFIG_CLPS711X_CPU_PLL_MULT);
> > }
> > diff --git a/arch/arm/mach-clps711x/Kconfig b/arch/arm/mach-clps711x/Kconfig
> > index f0adeda..d2873b4 100644
> > --- a/arch/arm/mach-clps711x/Kconfig
> > +++ b/arch/arm/mach-clps711x/Kconfig
> > @@ -10,6 +10,19 @@ config MACH_CLEP7212
> >
> > endchoice
> >
> > +menu "CLPS711X specific settings"
> > +
> > +config CLPS711X_CPU_PLL_MULT
> > + int "CPU PLL multiplier (20-50)"
> > + range 20 50
> > + default "40"
> > + help
> > + Define CPU PLL multiplier. PLL is calculated by formula:
> > + PLL Frequency = (PLL Multiplier / 2) * 3686400 Hz
> > + Default value is 40, for achieve 73 MHz.
> > +
> Nack NO KConfig for such sensitve value
>
> if it's soc detect it I do not have the datasheet but I'm sure you can
There are two possible clock sources: 3.6864 MHz crystal or 13 MHz
clock. Only source can be detected. When crystal is chosen, CPU starts at
low speed, so we should setup PLL register to increase speed.
Detect more is impossible.
Datasheet, if you interest, here www.cirrus.com/en/pubs/manual/ep73xx_um-f.pdf
But I am doubt if you find more.
Thanks!
---
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2013-02-13 12:23 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-13 11:41 [PATCH v2 1/9] ARM: clps711x: Rework lowlevel initialization code Alexander Shiyan
2013-02-13 11:41 ` [PATCH v2 2/9] ARM: clps711x: Move basic lowlevel initialization in common CLPS711X location Alexander Shiyan
2013-02-13 11:41 ` [PATCH v2 3/9] ARM: clps711x: Adds config option for CPU PLL multiplier Alexander Shiyan
2013-02-13 12:02 ` Jean-Christophe PLAGNIOL-VILLARD
2013-02-13 12:23 ` Alexander Shiyan [this message]
2013-02-13 13:36 ` [SPAM] Re[2]: " Jean-Christophe PLAGNIOL-VILLARD
2013-02-13 17:06 ` Sascha Hauer
2013-02-13 17:20 ` Re[2]: " Alexander Shiyan
2013-02-13 17:40 ` Sascha Hauer
2013-02-13 18:01 ` Jean-Christophe PLAGNIOL-VILLARD
2013-02-13 18:06 ` Re[2]: " Alexander Shiyan
2013-02-13 11:41 ` [PATCH v2 4/9] ARM: clps711x: Remove unused "start" declaration from reset.c Alexander Shiyan
2013-02-13 11:41 ` [PATCH v2 5/9] ARM: clps711x: Mark private functions that not will be used outside as static Alexander Shiyan
2013-02-13 11:41 ` [PATCH v2 6/9] ARM: clep7212: Fix NULL pointer exception if MMU is enabled Alexander Shiyan
2013-02-13 11:41 ` [PATCH v2 7/9] ARM: clps711x: Move memory initialization in common CLPS711X location Alexander Shiyan
2013-02-13 11:41 ` [PATCH v2 8/9] ARM: clep7212: Update default environment Alexander Shiyan
2013-02-13 11:41 ` [PATCH v2 9/9] ARM: clps711x: Update defconfig Alexander Shiyan
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=1360758205.278703573@f188.mail.ru \
--to=shc_work@mail.ru \
--cc=barebox@lists.infradead.org \
--cc=plagnioj@jcrosoft.com \
/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