From patchwork Tue Jun 16 02:54:25 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Viresh Kumar X-Patchwork-Id: 6612841 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 263B1C0020 for ; Tue, 16 Jun 2015 02:57:32 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 268FE207B5 for ; Tue, 16 Jun 2015 02:57:31 +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 2E0F2207B6 for ; Tue, 16 Jun 2015 02:57:30 +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 1Z4h1c-0000DV-7G; Tue, 16 Jun 2015 02:54:56 +0000 Received: from mail-pa0-f46.google.com ([209.85.220.46]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1Z4h1X-0000Bt-Ne for linux-arm-kernel@lists.infradead.org; Tue, 16 Jun 2015 02:54:52 +0000 Received: by pacgb13 with SMTP id gb13so3294973pac.1 for ; Mon, 15 Jun 2015 19:54:30 -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=hXwFWEJBwUPdIejyo2Hd6JQAlYhAqS3l79J1nTq80vA=; b=K1VLe/vQCEx3zf4S4ZSvcFUb7nDJBFTqFSXdB3D+H7IjFMYCI9W8j2xMJU1WJL+lCo Zv2LIHbCOy4cjHvGzYZIi9GgLwSir5mCHfJfJUlSgDwmWtvg0Z/672bCShscS+W647HF BJ1CTtPcg8QLVk21+38AXWSexS91773nShIr7BTgSKnj8DlqH3eyG/0qwP6V0hHZy4XX v3FvCoTTFxiHY61pbV9omVuTQk7IZ9qCTW9/42LUQ/yg6otly6JQLX9jjquY8+Egg85G lTE9HVqB4KYRhcIysaMBrxjQX0LT1RfOLhI/HjSmCg5UmpVbTIBrDOEfA8QBgoWlNWMm wsYw== X-Gm-Message-State: ALoCoQlVlgEMgZq61vbcQz/6qKJCK96RPWff1PtRN89/CGs+F6VgtjClwS0EBupUgJ3KXoOerWo5 X-Received: by 10.68.57.170 with SMTP id j10mr54477104pbq.150.1434423269161; Mon, 15 Jun 2015 19:54:29 -0700 (PDT) Received: from localhost ([122.167.70.98]) by mx.google.com with ESMTPSA id ni1sm13653969pdb.32.2015.06.15.19.54.27 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Mon, 15 Jun 2015 19:54:28 -0700 (PDT) Date: Tue, 16 Jun 2015 08:24:25 +0530 From: Viresh Kumar To: Rob Herring Subject: Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend' Message-ID: <20150616025425.GB26592@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-20150615_195451_837272_53800E81 X-CRM114-Status: GOOD ( 16.79 ) X-Spam-Score: -0.7 (/) Cc: Nishanth Menon , "devicetree@vger.kernel.org" , Abhilash Kesavan , "linaro-kernel@lists.linaro.org" , Thomas Abraham , Kevin Hilman , "linux-pm@vger.kernel.org" , Viswanath Puttagunta , Stephen Boyd , Santosh Shilimkar , Rafael Wysocki , Olof Johansson , Mark Brown , Mike Turquette , Sudeep Holla , Grant Likely , Arnd Bergmann , Thomas Petazzoni , "linux-arm-kernel@lists.infradead.org" , Lucas Stach 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.7 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, 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 16-06-15, 06:01, Viresh Kumar wrote: > On 16 June 2015 at 05:05, Rob Herring wrote: > >> +- opp-suspend: Phandle of the OPP to set while device is suspended. > >> + > > > > I would just do a bool property in the OPP you want. > > I will do that if you want, no big deal. In order to not waste any time, here is the change you suggested. Ack the one (original change or this one) you like :) ------------------8<---------------------- 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 Acked-by: Rob Herring --- V7->V8: - opp-suspend is moved to OPPs instead of the table, and is bool now. Documentation/devicetree/bindings/power/opp.txt | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/Documentation/devicetree/bindings/power/opp.txt b/Documentation/devicetree/bindings/power/opp.txt index 2938c52dbf84..2d4291127003 100644 --- a/Documentation/devicetree/bindings/power/opp.txt +++ b/Documentation/devicetree/bindings/power/opp.txt @@ -113,6 +113,9 @@ properties. frequency for a short duration of time limited by the device's power, current and thermal limits. +- opp-suspend: Marks the OPP to be used during device suspend. Only one OPP in + the table should have this. + - status: Marks the node enabled/disabled. Example 1: Single cluster Dual-core ARM cortex A9, switch DVFS states together. @@ -152,6 +155,7 @@ Example 1: Single cluster Dual-core ARM cortex A9, switch DVFS states together. opp-microvolt = <970000 975000 985000>; opp-microamp = <70000>; clock-latency-ns = <300000>; + opp-suspend; }; opp01 { opp-hz = <1100000000>; @@ -230,6 +234,7 @@ independently. opp-microvolt = <970000 975000 985000>; opp-microamp = <70000>; clock-latency-ns = <300000>; + opp-suspend; }; opp01 { opp-hz = <1100000000>; @@ -305,6 +310,7 @@ DVFS state together. opp-microvolt = <970000 975000 985000>; opp-microamp = <70000>; clock-latency-ns = <300000>; + opp-suspend; }; opp01 { opp-hz = <1100000000>; @@ -330,6 +336,7 @@ DVFS state together. opp-microvolt = <1045000 1050000 1055000>; opp-microamp = <95000>; clock-latency-ns = <400000>; + opp-suspend; }; opp11 { opp-hz = <1400000000>;