Re: [PATCH][V2] cpuidle: simplify multiple driver support

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

On 05/28/2013 11:28 PM, Rafael J. Wysocki wrote:
> On Tuesday, May 28, 2013 11:08:13 PM Daniel Lezcano wrote:
>> On 05/22/2013 03:05 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 and also the cpuidle_[un]register functions,
>>> added recently in the cpuidle framework.
>>>
>>> The current code, a bit poor in comments, has been commented and simplified.
>>>
>>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>>> ---
>>>
>>> [V2]:
>>>  - fixed bad refcount check
>>>  - inverted clockevent notify off order at unregister time
>> Hi Rafael,
>>
>> could you consider this patch for merging ? I took into account
>> Francesco's comments.
> I'm waiting for Len to have a look at it which may take a few days more.

Ok, is it possible to queue the patch for 3.11 now ?

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