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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05/22/2012 07:58 PM, Sekhar Nori wrote:
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@xxxxxxxxxx>

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

Cool !

Just a reminder, I don't have this board, so I was not able to test it.

Thanks.
  -- Daniel


--
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

_______________________________________________
linux-pm mailing list
linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/linux-pm



[Index of Archives]     [Linux ACPI]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [CPU Freq]     [Kernel Newbies]     [Fedora Kernel]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux