From: "Heiko Stübner" <heiko@sntech.de>
To: Kever Yang <kever.yang@rock-chips.com>
Cc: Chris Zhong <zyw@rock-chips.com>,
dianders@chromium.org, Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org, Russell King <linux@arm.linux.org.uk>,
Pawel Moll <pawel.moll@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Linus Walleij <linus.walleij@linaro.org>,
linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org,
Rob Herring <robh+dt@kernel.org>,
Kumar Gala <galak@codeaurora.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/2] Increase the maximum cpu frequency of rk3288
Date: Tue, 25 Nov 2014 17:23:59 +0100 [thread overview]
Message-ID: <3297542.UgVU3NH89O@diego> (raw)
In-Reply-To: <5474AAB4.2060301@rock-chips.com>
Hi Chris, Kever,
Am Mittwoch, 26. November 2014, 00:13:40 schrieb Kever Yang:
> On 11/25/2014 05:37 PM, Chris Zhong wrote:
> > The maximum cpu frequency of rk3288 can up to 1.8Ghz, but the vdd_cpu need
> > set to 1.4v. I've tested these patches on rk3288 evb board.
>
> I'm not sure why you need this patch, I think we have a discuss
> for the cpu operating point before.
> In this case:
> 1. rk3288.dtsi is for all the rk3288 Soc based system, you may need
> a separate opp table in rk3288-evb-rk808.dts;
> 2. 1.4V may beyond the supported voltage range too much,
> and it's not a good idea to add it to rk3288.dtsi as a safe voltage.
> 3. Do you have a stress/heavy load test on evb with 1.4v at 1.8GHz?
That is how I remember it as well ... the default opp table is supposed to
contain only the safest of values, while individual boards can then increase
those in conjunction with their thermal design.
As the evb itself does only have the naked soc without any heat-sink or
similar, I'm not even sure if these additional frequencies should be enabled
there at all.
Heiko
next prev parent reply other threads:[~2014-11-25 16:23 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-25 9:37 [PATCH 0/2] Increase the maximum cpu frequency of rk3288 Chris Zhong
2014-11-25 9:37 ` [PATCH 1/2] ARM: dts: rockchip: change the max voltage of cpu to 1.4v Chris Zhong
2014-11-25 9:37 ` [PATCH 2/2] ARM: dts: rk3288: change the max clk of cpu to 1.8Ghz Chris Zhong
2014-11-25 16:13 ` [PATCH 0/2] Increase the maximum cpu frequency of rk3288 Kever Yang
2014-11-25 16:23 ` Heiko Stübner [this message]
2014-11-25 17:37 ` Doug Anderson
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=3297542.UgVU3NH89O@diego \
--to=heiko@sntech.de \
--cc=devicetree@vger.kernel.org \
--cc=dianders@chromium.org \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=kever.yang@rock-chips.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=zyw@rock-chips.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;
as well as URLs for NNTP newsgroup(s).