| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CAL_Jsq+FBNLi8viCENmutrfWiefN0QBEKSCAY_KUi94ebthFCg@mail.gmail.com>
Date: Thu, 30 Jul 2015 08:23:59 -0500
From: Rob Herring <robherring2@...il.com>
To: Viresh Kumar <viresh.kumar@...aro.org>
Cc: Rafael Wysocki <rjw@...ysocki.net>,
Stephen Boyd <sboyd@...eaurora.org>,
"linaro-kernel@...ts.linaro.org" <linaro-kernel@...ts.linaro.org>,
"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
Arnd Bergmann <arnd.bergmann@...aro.org>,
Nishanth Menon <nm@...com>, Mark Brown <broonie@...nel.org>,
Michael Turquette <mturquette@...libre.com>,
Sudeep Holla <Sudeep.Holla@....com>,
Viswanath Puttagunta <viswanath.puttagunta@...aro.org>,
Lucas Stach <l.stach@...gutronix.de>,
Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
Thomas Abraham <ta.omasab@...il.com>,
Abhilash Kesavan <kesavan.abhilash@...il.com>,
Kevin Hilman <khilman@...aro.org>,
Santosh Shilimkar <santosh.shilimkar@...cle.com>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
Ian Campbell <ijc+devicetree@...lion.org.uk>,
Kumar Gala <galak@...eaurora.org>,
open list <linux-kernel@...r.kernel.org>,
Mark Rutland <mark.rutland@....com>,
Pawel Moll <pawel.moll@....com>,
"Rafael J. Wysocki" <rafael.j.wysocki@...el.com>,
Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH V3 01/16] PM / OPP: Update bindings to make opp-hz a 64
bit value
On Wed, Jul 29, 2015 at 5:52 AM, Viresh Kumar <viresh.kumar@...aro.org> wrote:
> With a 32 bit value, the maximum frequency that the bindings can
> support is ~ 4 GHz. And that might fall short of what newer systems may
> have.
4GHz should be fast enough for anyone...
> Allow opp-hz to be a 64 bit big-endian value.
>
> Cc: Rob Herring <rob.herring@...aro.org>
Acked-by: Rob Herring <robh@...nel.org>
> Cc: devicetree@...r.kernel.org
> Suggested-by: Stephen Boyd <sboyd@...eaurora.org>
> Suggested-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>
> Signed-off-by: Viresh Kumar <viresh.kumar@...aro.org>
> ---
> Documentation/devicetree/bindings/power/opp.txt | 40 ++++++++++++-------------
> 1 file changed, 20 insertions(+), 20 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/power/opp.txt b/Documentation/devicetree/bindings/power/opp.txt
> index 0d5e7c978121..0cb44dc21f97 100644
> --- a/Documentation/devicetree/bindings/power/opp.txt
> +++ b/Documentation/devicetree/bindings/power/opp.txt
> @@ -88,7 +88,7 @@ This defines voltage-current-frequency combinations along with other related
> properties.
>
> Required properties:
> -- opp-hz: Frequency in Hz
> +- opp-hz: Frequency in Hz, expressed as a 64-bit big-endian integer.
>
> Optional properties:
> - opp-microvolt: voltage in micro Volts.
> @@ -158,20 +158,20 @@ Example 1: Single cluster Dual-core ARM cortex A9, switch DVFS states together.
> opp-shared;
>
> opp00 {
> - opp-hz = <1000000000>;
> + opp-hz = /bits/ 64 <1000000000>;
> opp-microvolt = <970000 975000 985000>;
> opp-microamp = <70000>;
> clock-latency-ns = <300000>;
> opp-suspend;
> };
> opp01 {
> - opp-hz = <1100000000>;
> + opp-hz = /bits/ 64 <1100000000>;
> opp-microvolt = <980000 1000000 1010000>;
> opp-microamp = <80000>;
> clock-latency-ns = <310000>;
> };
> opp02 {
> - opp-hz = <1200000000>;
> + opp-hz = /bits/ 64 <1200000000>;
> opp-microvolt = <1025000>;
> clock-latency-ns = <290000>;
> turbo-mode;
> @@ -237,20 +237,20 @@ independently.
> */
>
> opp00 {
> - opp-hz = <1000000000>;
> + opp-hz = /bits/ 64 <1000000000>;
> opp-microvolt = <970000 975000 985000>;
> opp-microamp = <70000>;
> clock-latency-ns = <300000>;
> opp-suspend;
> };
> opp01 {
> - opp-hz = <1100000000>;
> + opp-hz = /bits/ 64 <1100000000>;
> opp-microvolt = <980000 1000000 1010000>;
> opp-microamp = <80000>;
> clock-latency-ns = <310000>;
> };
> opp02 {
> - opp-hz = <1200000000>;
> + opp-hz = /bits/ 64 <1200000000>;
> opp-microvolt = <1025000>;
> opp-microamp = <90000;
> lock-latency-ns = <290000>;
> @@ -313,20 +313,20 @@ DVFS state together.
> opp-shared;
>
> opp00 {
> - opp-hz = <1000000000>;
> + opp-hz = /bits/ 64 <1000000000>;
> opp-microvolt = <970000 975000 985000>;
> opp-microamp = <70000>;
> clock-latency-ns = <300000>;
> opp-suspend;
> };
> opp01 {
> - opp-hz = <1100000000>;
> + opp-hz = /bits/ 64 <1100000000>;
> opp-microvolt = <980000 1000000 1010000>;
> opp-microamp = <80000>;
> clock-latency-ns = <310000>;
> };
> opp02 {
> - opp-hz = <1200000000>;
> + opp-hz = /bits/ 64 <1200000000>;
> opp-microvolt = <1025000>;
> opp-microamp = <90000>;
> clock-latency-ns = <290000>;
> @@ -339,20 +339,20 @@ DVFS state together.
> opp-shared;
>
> opp10 {
> - opp-hz = <1300000000>;
> + opp-hz = /bits/ 64 <1300000000>;
> opp-microvolt = <1045000 1050000 1055000>;
> opp-microamp = <95000>;
> clock-latency-ns = <400000>;
> opp-suspend;
> };
> opp11 {
> - opp-hz = <1400000000>;
> + opp-hz = /bits/ 64 <1400000000>;
> opp-microvolt = <1075000>;
> opp-microamp = <100000>;
> clock-latency-ns = <400000>;
> };
> opp12 {
> - opp-hz = <1500000000>;
> + opp-hz = /bits/ 64 <1500000000>;
> opp-microvolt = <1010000 1100000 1110000>;
> opp-microamp = <95000>;
> clock-latency-ns = <400000>;
> @@ -379,7 +379,7 @@ Example 4: Handling multiple regulators
> opp-shared;
>
> opp00 {
> - opp-hz = <1000000000>;
> + opp-hz = /bits/ 64 <1000000000>;
> opp-microvolt = <970000>, /* Supply 0 */
> <960000>, /* Supply 1 */
> <960000>; /* Supply 2 */
> @@ -392,7 +392,7 @@ Example 4: Handling multiple regulators
> /* OR */
>
> opp00 {
> - opp-hz = <1000000000>;
> + opp-hz = /bits/ 64 <1000000000>;
> opp-microvolt = <970000 975000 985000>, /* Supply 0 */
> <960000 965000 975000>, /* Supply 1 */
> <960000 965000 975000>; /* Supply 2 */
> @@ -405,7 +405,7 @@ Example 4: Handling multiple regulators
> /* OR */
>
> opp00 {
> - opp-hz = <1000000000>;
> + opp-hz = /bits/ 64 <1000000000>;
> opp-microvolt = <970000 975000 985000>, /* Supply 0 */
> <960000 965000 975000>, /* Supply 1 */
> <960000 965000 975000>; /* Supply 2 */
> @@ -437,12 +437,12 @@ Example 5: Multiple OPP tables
> opp-shared;
>
> opp00 {
> - opp-hz = <600000000>;
> + opp-hz = /bits/ 64 <600000000>;
> ...
> };
>
> opp01 {
> - opp-hz = <800000000>;
> + opp-hz = /bits/ 64 <800000000>;
> ...
> };
> };
> @@ -453,12 +453,12 @@ Example 5: Multiple OPP tables
> opp-shared;
>
> opp10 {
> - opp-hz = <1000000000>;
> + opp-hz = /bits/ 64 <1000000000>;
> ...
> };
>
> opp11 {
> - opp-hz = <1100000000>;
> + opp-hz = /bits/ 64 <1100000000>;
> ...
> };
> };
> --
> 2.4.0
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists