Re: [PATCH 2/2] ARM: at91: cpuidle: Move driver to drivers/cpuidle

On 10/14/2013 12:20 PM, Bartlomiej Zolnierkiewicz wrote:

Hi Daniel,

On Sunday, October 13, 2013 12:17:17 PM Daniel Lezcano wrote:

...

On 10/13/2013 11:19 AM, Thomas Petazzoni wrote:

...

Dear Daniel Lezcano,

On Sat, 12 Oct 2013 17:51:39 +0200, Daniel Lezcano wrote:

...

+config ARM_AT91_CPUIDLE

•    bool "Cpu Idle Driver for the AT91 processors"
  

Nitpick: indentation is done with spaces here, while it should be one tab.

Ah, right. Fixed for the V2.

Could you also please add 'default y' to preserve the old behavior? (the at91 cpuidle support was always built-in for CONFIG_CPU_IDLE=y)

Yes, sure.

Thanks for the review.

-- Daniel