Re: [PATCH][v2] ARM: davinci: cpuidle - remove ops

Hi Daniel,

On 5/21/2012 3:37 AM, Daniel Lezcano wrote:

This patch removes the ops usage because we have the index passed as parameter to the idle function and we can determine if we do WFI or memory retention.

The benefit of this cleanup is the removal of:

• the ops
• the statedata usage because we want to get rid of it in all the drivers
• extra structure definition
• extra functions definition
• remove macro definition using BIT(0)

It also benefits the readability.

Signed-off-by: Daniel Lezcano daniel.lezcano@linaro.org

Looks good to me. If there is no other plan, I will queue these two cpuidle patches for v3.6 through DaVinci tree.

Thanks, Sekhar