From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] sunxi: set the default CPUx frequency of H5 to 816MHz
Date: Tue, 31 Oct 2017 14:54:41 +0100 [thread overview]
Message-ID: <20171031135441.wdbromnlischud4t@flea> (raw)
In-Reply-To: <655c14fa3074f1930e6b1d058324af88@aosc.io>
On Tue, Oct 31, 2017 at 04:05:36PM +0800, icenowy at aosc.io wrote:
> 在 2017-10-31 15:57,Jagan Teki 写道:
> > On Tue, Oct 31, 2017 at 5:06 AM, Icenowy Zheng <icenowy@aosc.io> wrote:
> > > Some H5 boards are designed to start at 1.1V CPUx voltage (e.g. Nano
> > > Pi
> > > NEO2), which may not work properly at 1008MHz if the chip's quality is
> > > not so good.
> > >
> > > Lower the default CPUx frequency of H5 to 816MHz.
> > >
> > > Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> > > ---
> > > arch/arm/mach-sunxi/Kconfig | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/arch/arm/mach-sunxi/Kconfig b/arch/arm/mach-sunxi/Kconfig
> > > index 09cfec6f57..1fededd0a3 100644
> > > --- a/arch/arm/mach-sunxi/Kconfig
> > > +++ b/arch/arm/mach-sunxi/Kconfig
> > > @@ -397,9 +397,9 @@ config SYS_CLK_FREQ
> > > default 1008000000 if MACH_SUN5I
> > > default 1008000000 if MACH_SUN6I
> > > default 912000000 if MACH_SUN7I
> > > + default 816000000 if MACH_SUN50I || MACH_SUN50I_H5
> > > default 1008000000 if MACH_SUN8I
> >
> > Even orangepi pc2 has 1.1v after power-on and it's work fine [1] did
> > you find an issue with neo2?
So you have one single model that fails, and you change the default
frequency of all the boards using that SoC?
It seems a bit overkill.
I guess we have two solutions:
1) Change the frequency in that board config
2) Change the voltage in that board config
There's no need to enforce a SoC-wide change.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171031/74509f18/attachment.sig>
next prev parent reply other threads:[~2017-10-31 13:54 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-30 23:36 [U-Boot] [PATCH] sunxi: set the default CPUx frequency of H5 to 816MHz Icenowy Zheng
2017-10-31 7:57 ` Jagan Teki
2017-10-31 8:05 ` icenowy at aosc.io
2017-10-31 8:53 ` Jagan Teki
2017-10-31 9:08 ` Icenowy Zheng
2017-10-31 13:54 ` Maxime Ripard [this message]
2017-11-01 0:31 ` Icenowy Zheng
2017-11-01 2:10 ` [U-Boot] [linux-sunxi] " Ondřej Jirman
2017-11-01 2:20 ` Ondřej Jirman
2017-11-01 9:16 ` Jagan Teki
2017-11-02 13:56 ` [U-Boot] " Maxime Ripard
2017-11-03 8:51 ` Jagan Teki
2017-11-03 16:30 ` Maxime Ripard
2017-11-06 8:10 ` Jagan Teki
2017-12-02 16:27 ` Jagan Teki
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=20171031135441.wdbromnlischud4t@flea \
--to=maxime.ripard@free-electrons.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.