Re: [PATCH] cpuidle: simplify multiple driver support

[Daniel Lezcano <daniel.lezcano@linaro.org>]

On 04/26/2013 02:07 PM, Rafael J. Wysocki wrote:
> On Friday, April 26, 2013 01:42:09 PM Daniel Lezcano wrote:
>> Commit bf4d1b5ddb78f86078ac6ae0415802d5f0c68f92 brought the multiple driver
>> support. The code added a couple of new API to register the driver per cpu.
>> That led to some code complexity to handle the kernel config options when
>> the multiple driver support is enabled or not, which is not really necessary.
>> The code has to be compatible when the multiple driver support is not enabled,
>> and the multiple driver support has to be compatible with the old api.
>>
>> This patch removes this API, which is not yet used by any driver but needed
>> for the HMP cpuidle drivers which will come soon, and replaces its usage
>> by a cpumask pointer in the cpuidle driver structure telling what cpus are
>> handled by the driver. That let the API cpuidle_[un]register_driver to be used
>> for the multipled driver support.
>>
>> The current code, a bit poor in comments, has been commented and simplified.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Honestly, I'd prefer that to go into 3.11.  At this point I want things to
> stabilize rather than to stir up.

Yes, sure. That makes sense.

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