Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-22 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 03/21/2012 10:54 PM, Kevin Hilman wrote: Daniel Lezcanodaniel.lezc...@linaro.org writes: On 03/21/2012 02:43 PM, Jean Pihet wrote: On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On

[RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not change the behavior of the driver itself. A couple a things call my intention. Why the cpuidle device is set for cpu0 only and why the WFI is not used ? Daniel Lezcano (7): ARM: OMAP4: cpuidle -

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Shilimkar, Santosh
On Wed, Mar 21, 2012 at 2:57 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not change the behavior of the driver itself. Thanks. Will have a look at your series. A couple a things call my

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
On 03/21/2012 10:36 AM, Shilimkar, Santosh wrote: On Wed, Mar 21, 2012 at 2:57 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not change the behavior of the driver itself. Thanks. Will have a

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Santosh Shilimkar
On Wednesday 21 March 2012 03:21 PM, Daniel Lezcano wrote: On 03/21/2012 10:36 AM, Shilimkar, Santosh wrote: On Wed, Mar 21, 2012 at 2:57 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Santosh Shilimkar
Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not change the behavior of the driver itself. A couple a things call my intention. Why the cpuidle device is set for cpu0 only

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
On 03/21/2012 10:56 AM, Santosh Shilimkar wrote: On Wednesday 21 March 2012 03:21 PM, Daniel Lezcano wrote: On 03/21/2012 10:36 AM, Shilimkar, Santosh wrote: On Wed, Mar 21, 2012 at 2:57 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: This patchset is a proposition to improve a bit the

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
On 03/21/2012 11:07 AM, Santosh Shilimkar wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not change the behavior of the driver itself. A couple a things call my

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Shilimkar, Santosh
On Wed, Mar 21, 2012 at 4:13 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: On 03/21/2012 10:56 AM, Santosh Shilimkar wrote: On Wednesday 21 March 2012 03:21 PM, Daniel Lezcano wrote: On 03/21/2012 10:36 AM, Shilimkar, Santosh wrote: On Wed, Mar 21, 2012 at 2:57 PM, Daniel Lezcano

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
On 03/21/2012 11:49 AM, Shilimkar, Santosh wrote: On Wed, Mar 21, 2012 at 4:13 PM, Daniel Lezcano daniel.lezc...@linaro.org wrote: On 03/21/2012 10:56 AM, Santosh Shilimkar wrote: On Wednesday 21 March 2012 03:21 PM, Daniel Lezcano wrote: On 03/21/2012 10:36 AM, Shilimkar, Santosh wrote:

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Jean Pihet
Hi Santosh, Daniel, On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not change the behavior

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Jean Pihet
On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not change the behavior of the driver itself.

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
On 03/21/2012 02:19 PM, Jean Pihet wrote: Hi Santosh, Daniel, On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit the code. The changes are code

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
On 03/21/2012 02:43 PM, Jean Pihet wrote: On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Shilimkar, Santosh
On Wed, Mar 21, 2012 at 6:49 PM, Jean Pihet jean.pi...@newoldbits.com wrote: Hi Santosh, Daniel, On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
On 03/21/2012 02:43 PM, Jean Pihet wrote: On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit the code. The changes are code cleanup and does not

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 03/21/2012 02:43 PM, Jean Pihet wrote: On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Daniel Lezcano
On 03/21/2012 10:54 PM, Kevin Hilman wrote: Daniel Lezcanodaniel.lezc...@linaro.org writes: On 03/21/2012 02:43 PM, Jean Pihet wrote: On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: