mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Quentin Schulz <quentin.schulz@free-electrons.com>
To: dmitry.torokhov@gmail.com, robh+dt@kernel.org,
	mark.rutland@arm.com, maxime.ripard@free-electrons.com,
	wens@csie.org, lee.jones@linaro.org, linux@armlinux.org.uk,
	jic23@kernel.org, knaack.h@gmx.de, lars@metafoo.de,
	pmeerw@pmeerw.net
Cc: Quentin Schulz <quentin.schulz@free-electrons.com>,
	thomas.petazzoni@free-electrons.com, linux-input@vger.kernel.org,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org,
	linux-sunxi@googlegroups.com, icenowy@aosc.xyz
Subject: [PATCH v3 02/11] ARM: sun8i: a33: add all operating points
Date: Tue, 21 Mar 2017 16:36:02 +0100	[thread overview]
Message-ID: <20170321153611.16228-3-quentin.schulz@free-electrons.com> (raw)
In-Reply-To: <20170321153611.16228-1-quentin.schulz@free-electrons.com>

This adds almost all operating points allowed for the A33 as defined by
fex files available at:
https://github.com/linux-sunxi/sunxi-boards/tree/master/sys_config/a33

There are more possible frequencies in this patch than there are in the
fex files because the fex files only give an interval of possible
frequencies for a given voltage. All supported frequencies are defined
in the original driver code in Allwinner vendor tree.

There are two missing frequencies though: 1104MHz and 1200MHz which
require the CPU to have 1.32V supplied, which is higher than the default
voltage.

Without all A33 boards defining the CPU regulator, we cannot have these
two frequencies as it would cause the CPU to try to run a higher
frequency without "overvolting" which is very likely to crash the CPU.

Therefore, these two frequencies must be enabled on a per-board basis.

Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>
---

v2:
  - updated commit log,
  - removed 1.1GHz as it requires overvolting the CPU which need the CPU
  regulator to be set which is set on a per-board basis,

 arch/arm/boot/dts/sun8i-a33.dtsi | 54 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 54 insertions(+)

diff --git a/arch/arm/boot/dts/sun8i-a33.dtsi b/arch/arm/boot/dts/sun8i-a33.dtsi
index 39b2d78..b1cdff66 100644
--- a/arch/arm/boot/dts/sun8i-a33.dtsi
+++ b/arch/arm/boot/dts/sun8i-a33.dtsi
@@ -49,18 +49,72 @@
 		compatible = "operating-points-v2";
 		opp-shared;
 
+		opp@120000000 {
+			opp-hz = /bits/ 64 <120000000>;
+			opp-microvolt = <1040000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
+		opp@240000000 {
+			opp-hz = /bits/ 64 <240000000>;
+			opp-microvolt = <1040000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
+		opp@312000000 {
+			opp-hz = /bits/ 64 <312000000>;
+			opp-microvolt = <1040000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
+		opp@408000000 {
+			opp-hz = /bits/ 64 <408000000>;
+			opp-microvolt = <1040000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
+		opp@480000000 {
+			opp-hz = /bits/ 64 <480000000>;
+			opp-microvolt = <1040000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
+		opp@504000000 {
+			opp-hz = /bits/ 64 <504000000>;
+			opp-microvolt = <1040000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
+		opp@600000000 {
+			opp-hz = /bits/ 64 <600000000>;
+			opp-microvolt = <1040000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
 		opp@648000000 {
 			opp-hz = /bits/ 64 <648000000>;
 			opp-microvolt = <1040000>;
 			clock-latency-ns = <244144>; /* 8 32k periods */
 		};
 
+		opp@720000000 {
+			opp-hz = /bits/ 64 <720000000>;
+			opp-microvolt = <1100000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
 		opp@816000000 {
 			opp-hz = /bits/ 64 <816000000>;
 			opp-microvolt = <1100000>;
 			clock-latency-ns = <244144>; /* 8 32k periods */
 		};
 
+		opp@912000000 {
+			opp-hz = /bits/ 64 <912000000>;
+			opp-microvolt = <1200000>;
+			clock-latency-ns = <244144>; /* 8 32k periods */
+		};
+
 		opp@1008000000 {
 			opp-hz = /bits/ 64 <1008000000>;
 			opp-microvolt = <1200000>;
-- 
2.9.3

  parent reply	other threads:[~2017-03-21 15:36 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-21 15:36 [PATCH v3 00/11] add thermal throttling to Allwinner A33 SoC Quentin Schulz
2017-03-21 15:36 ` [PATCH v3 01/11] ARM: sun8i: a33: add operating-points-v2 property to all nodes Quentin Schulz
2017-03-22  3:32   ` Chen-Yu Tsai
2017-03-22  7:16     ` Maxime Ripard
2017-03-21 15:36 ` Quentin Schulz [this message]
2017-03-22  3:31   ` [PATCH v3 02/11] ARM: sun8i: a33: add all operating points Chen-Yu Tsai
2017-03-22  7:16     ` Maxime Ripard
2017-03-21 15:36 ` [PATCH v3 03/11] ARM: dts: sun8i: sina33: add cpu-supply Quentin Schulz
2017-03-22  3:33   ` Chen-Yu Tsai
2017-03-22  7:17     ` Maxime Ripard
2017-03-21 15:36 ` [PATCH v3 04/11] Documentation: DT: bindings: mfd: add A33 GPADC binding Quentin Schulz
2017-03-23 16:45   ` Lee Jones
2017-03-29  0:31   ` Rob Herring
2017-03-21 15:36 ` [PATCH v3 05/11] Documentation: DT: bindings: input: touschcreen: remove sun4i documentation Quentin Schulz
2017-03-29  7:58   ` Lee Jones
2017-04-03 20:09     ` Dmitry Torokhov
2017-03-21 15:36 ` [PATCH v3 06/11] iio: adc: sun4i-gpadc-iio: move code used in MFD probing to new function Quentin Schulz
2017-03-21 15:36 ` [PATCH v3 07/11] iio: adc: sun4i-gpadc-iio: add support for A33 thermal sensor Quentin Schulz
2017-03-25 17:48   ` Jonathan Cameron
2017-03-21 15:36 ` [PATCH v3 08/11] ARM: dtsi: sun8i: a33: add " Quentin Schulz
2017-03-21 15:36 ` [PATCH v3 09/11] ARM: dtsi: sun8i: a33: add CPU thermal throttling Quentin Schulz
2017-03-21 15:36 ` [PATCH v3 10/11] ARM: sun8i: a33: Add devfreq-based GPU cooling Quentin Schulz
2017-03-21 15:36 ` [PATCH v3 11/11] ARM: sun8i: sina33: add highest OPP of CPUs Quentin Schulz
2017-04-04  8:31 ` [PATCH v3 00/11] add thermal throttling to Allwinner A33 SoC Lee Jones
2017-04-05  9:10   ` Quentin Schulz
2017-04-05  9:57     ` Lee Jones
2017-04-05 11:23       ` Quentin Schulz

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=20170321153611.16228-3-quentin.schulz@free-electrons.com \
    --to=quentin.schulz@free-electrons.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=icenowy@aosc.xyz \
    --cc=jic23@kernel.org \
    --cc=knaack.h@gmx.de \
    --cc=lars@metafoo.de \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sunxi@googlegroups.com \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=maxime.ripard@free-electrons.com \
    --cc=pmeerw@pmeerw.net \
    --cc=robh+dt@kernel.org \
    --cc=thomas.petazzoni@free-electrons.com \
    --cc=wens@csie.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®