From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753647AbdBFPZk (ORCPT ); Mon, 6 Feb 2017 10:25:40 -0500 Received: from lelnx194.ext.ti.com ([198.47.27.80]:19920 "EHLO lelnx194.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751715AbdBFPZi (ORCPT ); Mon, 6 Feb 2017 10:25:38 -0500 Message-ID: <58989547.9030002@ti.com> Date: Mon, 6 Feb 2017 09:24:55 -0600 From: Dave Gerlach User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Lukasz Majewski CC: =?windows-1252?Q?Beno=EEt_Cousson?= , , , , Kishon Vijay Abraham I , "Menon, Nishanth" Subject: Re: [PATCH 2/2] DTS: dra7: Add "opp_high" opp node to support 1.5 GHz SoC operation References: <1486135491-14223-1-git-send-email-lukma@denx.de> <1486135491-14223-2-git-send-email-lukma@denx.de> <20170206152450.3739af18@jawa> In-Reply-To: <20170206152450.3739af18@jawa> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 02/06/2017 08:24 AM, Lukasz Majewski wrote: > Hi Dave, > >> On 02/03/2017 09:24 AM, Lukasz Majewski wrote: >>> The "opp_high" property brings support for 1.5 GHz CPU frequency >>> for TI's am57xx line of processors. >>> >>> Signed-off-by: Lukasz Majewski >>> --- >>> arch/arm/boot/dts/dra7.dtsi | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >> >> This can't be blindly enabled for all am57xx SoCs, there is a >> forthcoming ti-cpufreq [1] driver that is needed that will decided >> which OPPs can and cannot be enabled for the specific variant in use. > > Do you know if there is ongoing work on using Avs0 voltage (encoded in > efuse) to control the per SoC PMIC voltage value when switching to > opp_high (1.5 GHz)? Yes, Viresh Kumar introduced multi-regulator support in the OPP core recently [1] which I wrote a test driver for here [2] which hooks into the OPP framework to do both ABB regulator scaling and also programs the AVS0 voltages. The current implementation I have cleaned up will depend on the ti-cpufreq driver so once that is merged I will send the AVS0 and ABB support driver upstream as well. If you are curious you can look at current internal implementation here [3]. Regards, Dave [1] http://www.spinics.net/lists/devicetree/msg153212.html [2] https://www.spinics.net/lists/kernel/msg2384336.html [3] http://git.ti.com/ti-linux-kernel/ti-linux-kernel/blobs/ti-lsk-linux-4.9.y/drivers/base/power/opp/ti-opp-supply.c > > >> >> Regards, >> Dave >> >> [1] http://www.spinics.net/lists/devicetree/msg158967.html >> >>> diff --git a/arch/arm/boot/dts/dra7.dtsi >>> b/arch/arm/boot/dts/dra7.dtsi index a860a56..5e678cd 100644 >>> --- a/arch/arm/boot/dts/dra7.dtsi >>> +++ b/arch/arm/boot/dts/dra7.dtsi >>> @@ -108,6 +108,11 @@ >>> opp-hz = /bits/ 64 <1176000000>; >>> opp-microvolt = <1160000 885000 1160000>; >>> }; >>> + >>> + opp_high@1500000000 { >>> + opp-hz = /bits/ 64 <1500000000>; >>> + opp-microvolt = <1210000 950000 1250000>; >>> + }; >>> }; >>> >>> /* >>> >> > > > > > Best regards, > > Lukasz Majewski > > -- > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de >