From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend' Date: Tue, 16 Jun 2015 23:21:39 +0200 Message-ID: <4228747.vs1U0RqIQB@vostro.rjw.lan> References: <20150616025425.GB26592@linux> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: Sender: linux-pm-owner@vger.kernel.org To: Rob Herring Cc: Viresh Kumar , "linaro-kernel@lists.linaro.org" , "linux-pm@vger.kernel.org" , Arnd Bergmann , Nishanth Menon , Mark Brown , Mike Turquette , Stephen Boyd , Grant Likely , Olof Johansson , Sudeep Holla , "devicetree@vger.kernel.org" , Viswanath Puttagunta , Lucas Stach , Thomas Petazzoni , "linux-arm-kernel@lists.infradead.org" , Thomas Abraham , Abhilash Kesavan , Kevin Hilman , Santosh List-Id: devicetree@vger.kernel.org On Tuesday, June 16, 2015 02:23:23 PM Rob Herring wrote: > On Mon, Jun 15, 2015 at 9:54 PM, Viresh Kumar wrote: > > 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 > > --- > > V7->V8: > > - opp-suspend is moved to OPPs instead of the table, and is bool now. > > Acked-by: Rob Herring Thanks! Does your ACK also apply to patches [1-2/3] in this series? Rafael From mboxrd@z Thu Jan 1 00:00:00 1970 From: rjw@rjwysocki.net (Rafael J. Wysocki) Date: Tue, 16 Jun 2015 23:21:39 +0200 Subject: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend' In-Reply-To: References: <20150616025425.GB26592@linux> Message-ID: <4228747.vs1U0RqIQB@vostro.rjw.lan> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tuesday, June 16, 2015 02:23:23 PM Rob Herring wrote: > On Mon, Jun 15, 2015 at 9:54 PM, Viresh Kumar wrote: > > 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 > > --- > > V7->V8: > > - opp-suspend is moved to OPPs instead of the table, and is bool now. > > Acked-by: Rob Herring Thanks! Does your ACK also apply to patches [1-2/3] in this series? Rafael