Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-17 Thread Rafael J. Wysocki
On Wed, Jun 17, 2015 at 4:38 AM, Viresh Kumar viresh.ku...@linaro.org wrote: On 16-06-15, 23:21, Rafael J. Wysocki wrote: Does your ACK also apply to patches [1-2/3] in this series? His RBY tag is already present for 1/3 :) Yes, it is, sorry for overlooking that. -- To unsubscribe from this

Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-16 Thread Rafael J. Wysocki
On Tuesday, June 16, 2015 02:23:23 PM Rob Herring wrote: On Mon, Jun 15, 2015 at 9:54 PM, Viresh Kumar viresh.ku...@linaro.org wrote: On 16-06-15, 06:01, Viresh Kumar wrote: On 16 June 2015 at 05:05, Rob Herring robherri...@gmail.com wrote: +- opp-suspend: Phandle of the OPP to set while

Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-16 Thread Rob Herring
On Mon, Jun 15, 2015 at 9:54 PM, Viresh Kumar viresh.ku...@linaro.org wrote: On 16-06-15, 06:01, Viresh Kumar wrote: On 16 June 2015 at 05:05, Rob Herring robherri...@gmail.com wrote: +- opp-suspend: Phandle of the OPP to set while device is suspended. + I would just do a bool property

Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-16 Thread Viresh Kumar
On 16-06-15, 23:21, Rafael J. Wysocki wrote: Does your ACK also apply to patches [1-2/3] in this series? His RBY tag is already present for 1/3 :) @Rob: Please Ack 2/3 as well :) -- viresh -- To unsubscribe from this list: send the line unsubscribe devicetree in the body of a message to

Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-15 Thread Viresh Kumar
On 16-06-15, 06:01, Viresh Kumar wrote: On 16 June 2015 at 05:05, Rob Herring robherri...@gmail.com 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

Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-15 Thread Rafael J. Wysocki
On Saturday, June 13, 2015 02:10:23 PM Viresh Kumar wrote: 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

Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-15 Thread Rob Herring
On Thu, Jun 4, 2015 at 11:20 AM, Viresh Kumar viresh.ku...@linaro.org wrote: 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:

Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-15 Thread Viresh Kumar
On 16 June 2015 at 05:05, Rob Herring robherri...@gmail.com 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. But I would like to explain why I did it this way. Nishanth

Re: [PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-13 Thread Viresh Kumar
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 viresh.ku...@linaro.org Date: Wed, 20 May 2015 08:27:49 +0530 Subject:

[PATCH V7 3/3] OPP: Add binding for 'opp-suspend'

2015-06-04 Thread Viresh Kumar
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 n...@ti.com Suggested-by: Nishanth Menon n...@ti.com Signed-off-by: