From: maxime.ripard@free-electrons.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [linux-sunxi] [PATCH] ARM: dts: sunxi: Raise minimum CPU voltage for sun7i-a20 to a level all boards can supply
Date: Mon, 3 Aug 2015 11:16:24 +0200 [thread overview]
Message-ID: <20150803091624.GZ2564@lukather> (raw)
In-Reply-To: <CAGRGNgV6b9yt2HnB2x2Wyc05KEahoX-Y+mVhxsryztcfA36H1g@mail.gmail.com>
On Mon, Aug 03, 2015 at 09:35:51AM +1000, Julian Calaby wrote:
> Hi Timo,
>
> On Mon, Aug 3, 2015 at 5:23 AM, Timo Sigurdsson
> <public_timo.s@silentcreek.de> wrote:
> > sun7i-a20.dtsi contains an cpufreq operating point at 0.9 volts. Most A20 boards
> > (or all?), however, do not allow the voltage to go below 1.0V. Thus, raise the
> > voltage for the lowest operating point to 1.0V so all boards can actually use
> > it.
>
> Surely it wouldn't be added here if some could supply 0.9v.
>
> Is the code that uses this smart enough to sensibly switch between two
> operating points with the same frequency and different voltages? If
> so, maybe just add a 144MHz @ 1.0v operating point?
And how would it choose between the two exactly ? Switch to the 144MHz
@ 0.9V and see if it works ? If it doesn't you might have screwed your
system already, and might not be able to recover from it at all.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150803/08e0c339/attachment.sig>
WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
To: Julian Calaby <julian.calaby-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: public_timo.s-fWgRPtSzPNU3WX+qO2AYSQ@public.gmane.org,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Ian Campbell
<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"Mailing List,
Arm"
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
linux-sunxi <linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org>,
Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>
Subject: Re: [PATCH] ARM: dts: sunxi: Raise minimum CPU voltage for sun7i-a20 to a level all boards can supply
Date: Mon, 3 Aug 2015 11:16:24 +0200 [thread overview]
Message-ID: <20150803091624.GZ2564@lukather> (raw)
In-Reply-To: <CAGRGNgV6b9yt2HnB2x2Wyc05KEahoX-Y+mVhxsryztcfA36H1g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
[-- Attachment #1: Type: text/plain, Size: 1038 bytes --]
On Mon, Aug 03, 2015 at 09:35:51AM +1000, Julian Calaby wrote:
> Hi Timo,
>
> On Mon, Aug 3, 2015 at 5:23 AM, Timo Sigurdsson
> <public_timo.s-fWgRPtSzPNU3WX+qO2AYSQ@public.gmane.org> wrote:
> > sun7i-a20.dtsi contains an cpufreq operating point at 0.9 volts. Most A20 boards
> > (or all?), however, do not allow the voltage to go below 1.0V. Thus, raise the
> > voltage for the lowest operating point to 1.0V so all boards can actually use
> > it.
>
> Surely it wouldn't be added here if some could supply 0.9v.
>
> Is the code that uses this smart enough to sensibly switch between two
> operating points with the same frequency and different voltages? If
> so, maybe just add a 144MHz @ 1.0v operating point?
And how would it choose between the two exactly ? Switch to the 144MHz
@ 0.9V and see if it works ? If it doesn't you might have screwed your
system already, and might not be able to recover from it at all.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: Julian Calaby <julian.calaby@gmail.com>
Cc: public_timo.s@silentcreek.de, Rob Herring <robh+dt@kernel.org>,
Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>,
Russell King <linux@arm.linux.org.uk>,
devicetree <devicetree@vger.kernel.org>,
"Mailing List, Arm" <linux-arm-kernel@lists.infradead.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
linux-sunxi <linux-sunxi@googlegroups.com>,
Chen-Yu Tsai <wens@csie.org>
Subject: Re: [linux-sunxi] [PATCH] ARM: dts: sunxi: Raise minimum CPU voltage for sun7i-a20 to a level all boards can supply
Date: Mon, 3 Aug 2015 11:16:24 +0200 [thread overview]
Message-ID: <20150803091624.GZ2564@lukather> (raw)
In-Reply-To: <CAGRGNgV6b9yt2HnB2x2Wyc05KEahoX-Y+mVhxsryztcfA36H1g@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1040 bytes --]
On Mon, Aug 03, 2015 at 09:35:51AM +1000, Julian Calaby wrote:
> Hi Timo,
>
> On Mon, Aug 3, 2015 at 5:23 AM, Timo Sigurdsson
> <public_timo.s@silentcreek.de> wrote:
> > sun7i-a20.dtsi contains an cpufreq operating point at 0.9 volts. Most A20 boards
> > (or all?), however, do not allow the voltage to go below 1.0V. Thus, raise the
> > voltage for the lowest operating point to 1.0V so all boards can actually use
> > it.
>
> Surely it wouldn't be added here if some could supply 0.9v.
>
> Is the code that uses this smart enough to sensibly switch between two
> operating points with the same frequency and different voltages? If
> so, maybe just add a 144MHz @ 1.0v operating point?
And how would it choose between the two exactly ? Switch to the 144MHz
@ 0.9V and see if it works ? If it doesn't you might have screwed your
system already, and might not be able to recover from it at all.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2015-08-03 9:16 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-02 19:23 [PATCH] ARM: dts: sunxi: Raise minimum CPU voltage for sun7i-a20 to a level all boards can supply Timo Sigurdsson
2015-08-02 19:23 ` Timo Sigurdsson
2015-08-02 19:23 ` Timo Sigurdsson
2015-08-02 23:35 ` [linux-sunxi] " Julian Calaby
2015-08-02 23:35 ` Julian Calaby
2015-08-02 23:35 ` Julian Calaby
2015-08-03 2:37 ` Chen-Yu Tsai
2015-08-03 2:37 ` Chen-Yu Tsai
2015-08-03 2:37 ` Chen-Yu Tsai
2015-08-03 4:22 ` [linux-sunxi] " Julian Calaby
2015-08-03 4:22 ` Julian Calaby
2015-08-03 4:22 ` Julian Calaby
2015-08-03 4:26 ` [linux-sunxi] " Chen-Yu Tsai
2015-08-03 4:26 ` Chen-Yu Tsai
2015-08-03 4:26 ` Chen-Yu Tsai
2015-08-03 9:03 ` [linux-sunxi] " Timo Sigurdsson
2015-08-03 9:03 ` Timo Sigurdsson
2015-08-03 9:03 ` Timo Sigurdsson
2015-08-03 9:34 ` [linux-sunxi] " Maxime Ripard
2015-08-03 9:34 ` Maxime Ripard
2015-08-03 9:34 ` Maxime Ripard
2015-08-03 9:36 ` [linux-sunxi] " Julian Calaby
2015-08-03 9:36 ` Julian Calaby
2015-08-03 9:36 ` Julian Calaby
2015-08-04 8:51 ` Timo Sigurdsson
2015-08-04 8:51 ` Timo Sigurdsson
2015-08-04 8:51 ` Timo Sigurdsson
2015-08-03 9:28 ` [linux-sunxi] " Maxime Ripard
2015-08-03 9:28 ` Maxime Ripard
2015-08-03 9:28 ` Maxime Ripard
2015-08-03 8:37 ` [linux-sunxi] " Timo Sigurdsson
2015-08-03 8:37 ` Timo Sigurdsson
2015-08-03 8:37 ` Timo Sigurdsson
2015-08-03 9:23 ` [linux-sunxi] " Maxime Ripard
2015-08-03 9:23 ` Maxime Ripard
2015-08-03 9:23 ` Maxime Ripard
2015-08-03 9:16 ` Maxime Ripard [this message]
2015-08-03 9:16 ` [linux-sunxi] " Maxime Ripard
2015-08-03 9:16 ` Maxime Ripard
2015-08-03 9:13 ` Maxime Ripard
2015-08-03 9:13 ` Maxime Ripard
2015-08-03 9:13 ` Maxime Ripard
2015-08-04 8:38 ` Timo Sigurdsson
2015-08-04 8:38 ` Timo Sigurdsson
2015-08-04 8:38 ` Timo Sigurdsson
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=20150803091624.GZ2564@lukather \
--to=maxime.ripard@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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.