From patchwork Sat Jun 13 08:40:23 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Viresh Kumar X-Patchwork-Id: 6602251 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 8B8949F2F4 for ; Sat, 13 Jun 2015 08:44:46 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 96C08206B8 for ; Sat, 13 Jun 2015 08:44:45 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id AE8B6206B6 for ; Sat, 13 Jun 2015 08:44:44 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1Z3gzk-0003rZ-JY; Sat, 13 Jun 2015 08:40:52 +0000 Received: from mail-pa0-f41.google.com ([209.85.220.41]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1Z3gzi-0003q2-4s for linux-arm-kernel@lists.infradead.org; Sat, 13 Jun 2015 08:40:50 +0000 Received: by pabqy3 with SMTP id qy3so35064031pab.3 for ; Sat, 13 Jun 2015 01:40:28 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-type:content-disposition:in-reply-to :user-agent; bh=EEqhDHYW070t9DjZNNXDsgxGF5+K4TpTJzqjoz78jws=; b=Z0XgsE04JTo9gMZwzZOb8uxzzW80EaBbHZU3M/93YSV7DPnllH0ezvJzx1sRPWaNjq qvIuIaJU2+sN54yHCUoKJtD0Jnom2Ba7Gl9hDeTu/GvAx1RRTl9hpcp1NfgqG7duFJUZ abzmo9MMRNSwJoNC4JdouqGH0J+1sRS9xNHjcvxivF+MgbsJ7ZPBuOqQri6kpf4KkdCh u8SHJJ9Ln01p1jBJYMbJ1nlw0hs4JpqSag2EYBVLMdawG7ctOTOGxBi0ja0nooikH6XM U8jUNMpFoQ3m152M8j5sc6cxYtxiNohw1bMmCmBlZmcgvfqzkOZJW8KjWslVbHPIjPt1 IAAg== X-Gm-Message-State: ALoCoQmF1oX4l8FiA94EOq+VFhvn4XWiIr8Ur7i05PX/n+b0R9YZMVZFBSpSjWCpHMADOJR+rY3N X-Received: by 10.70.37.207 with SMTP id a15mr29944929pdk.4.1434184828560; Sat, 13 Jun 2015 01:40:28 -0700 (PDT) Received: from localhost ([122.167.70.98]) by mx.google.com with ESMTPSA id zx1sm5869551pbb.73.2015.06.13.01.40.26 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Sat, 13 Jun 2015 01:40:27 -0700 (PDT) Date: Sat, 13 Jun 2015 14:10:23 +0530 From: Viresh Kumar To: Rafael Wysocki , rob.herring@linaro.org Subject: Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend' Message-ID: <20150613084023.GA4203@linux> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20150613_014050_231816_D9B310B2 X-CRM114-Status: GOOD ( 13.36 ) X-Spam-Score: -0.7 (/) Cc: nm@ti.com, devicetree@vger.kernel.org, kesavan.abhilash@gmail.com, linaro-kernel@lists.linaro.org, ta.omasab@gmail.com, khilman@linaro.org, linux-pm@vger.kernel.org, viswanath.puttagunta@linaro.org, santosh.shilimkar@oracle.com, sboyd@codeaurora.org, olof@lixom.net, broonie@kernel.org, mike.turquette@linaro.org, Sudeep.Holla@arm.com, grant.likely@linaro.org, arnd.bergmann@linaro.org, thomas.petazzoni@free-electrons.com, linux-arm-kernel@lists.infradead.org, l.stach@pengutronix.de X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On 04-06-15, 21:50, Viresh Kumar wrote: > + opp-suspend = <&suspend_opp>; > - opp00 { > + suspend-opp: opp00 { Minor nit, s/suspend-opp/suspend_opp and here is updated patch From: Viresh Kumar Date: Wed, 20 May 2015 08:27:49 +0530 Subject: [PATCH V8] OPP: Add binding for 'opp-suspend' On few platforms, for power efficiency, we want the device to be configured for a specific OPP while we put the device in suspend state. Add an optional property in operating-points-v2 bindings for that. Acked-by: Nishanth Menon Suggested-by: Nishanth Menon Signed-off-by: Viresh Kumar --- Documentation/devicetree/bindings/power/opp.txt | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/power/opp.txt b/Documentation/devicetree/bindings/power/opp.txt index 2938c52dbf84..29f115d26f7d 100644 --- a/Documentation/devicetree/bindings/power/opp.txt +++ b/Documentation/devicetree/bindings/power/opp.txt @@ -74,6 +74,8 @@ This describes the OPPs belonging to a device. This node can have following Missing property means devices have independent clock/voltage/current lines, but they share OPP tables. +- opp-suspend: Phandle of the OPP to set while device is suspended. + * OPP Node @@ -145,9 +147,10 @@ Example 1: Single cluster Dual-core ARM cortex A9, switch DVFS states together. cpu0_opp_table: opp_table0 { compatible = "operating-points-v2"; + opp-suspend = <&suspend_opp>; opp-shared; - opp00 { + suspend_opp: opp00 { opp-hz = <1000000000>; opp-microvolt = <970000 975000 985000>; opp-microamp = <70000>; @@ -219,13 +222,14 @@ independently. cpu_opp_table: opp_table { compatible = "operating-points-v2"; + opp-suspend = <&suspend_opp>; /* * Missing opp-shared property means CPUs switch DVFS states * independently. */ - opp00 { + suspend_opp: opp00 { opp-hz = <1000000000>; opp-microvolt = <970000 975000 985000>; opp-microamp = <70000>; @@ -298,9 +302,10 @@ DVFS state together. cluster0_opp: opp_table0 { compatible = "operating-points-v2"; + opp-suspend = <&suspend_opp0>; opp-shared; - opp00 { + suspend_opp: opp00 { opp-hz = <1000000000>; opp-microvolt = <970000 975000 985000>; opp-microamp = <70000>; @@ -323,9 +328,10 @@ DVFS state together. cluster1_opp: opp_table1 { compatible = "operating-points-v2"; + opp-suspend = <&suspend_opp1>; opp-shared; - opp10 { + suspend_opp: opp10 { opp-hz = <1300000000>; opp-microvolt = <1045000 1050000 1055000>; opp-microamp = <95000>;