Re: [PATCH 0/4] cpuidle - support multiple drivers at a time

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

On 10/04/2012 04:16 PM, Peter De Schrijver wrote:
> On Tue, Sep 25, 2012 at 12:43:50AM +0200, Daniel Lezcano wrote:
>> The discussion about having different cpus on the system with
>> different latencies bring us to a first attemp by adding a
>> pointer in the cpuidle_device to the states array.
>>
>> But as Rafael suggested, it would make more sense to create a
>> driver per cpu [1].
>>
>> This patch adds support for multiple cpuidle drivers.
>>
>> It creates a per cpu cpuidle driver pointer.
>>
>> In order to not break the different drivers, the function cpuidle_register_driver
>> assign for each cpu, the driver.
>>
>> The multiple driver support is optional and if it is not set, the cpuide driver
>> core code remains the same (except some code reorganisation).
>>
>> I did the following tests compiled, booted, tested without/with CONFIG_CPU_IDLE,
>> with/without CONFIG_CPU_IDLE_MULTIPLE_DRIVERS.
>>
>> Tested on Core2 Duo T9500 with acpi_idle [and intel_idle]
>> Tested on ARM Dual Cortex-A9 U8500 (aka Snowball)
>>
> 
> Tested on Cardhu (Tegra30).
> 
> Besides the possibly problem with the sysfs format:
> 
> Acked-by: Peter De Schrijver <pdeschrijver@nvidia.com>

Thanks a lot Peter for testing and reviewing the patchset.

  -- 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

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html