On 09/03/2012 03:22 PM, Peter De Schrijver wrote:
> On Wed, Jul 25, 2012 at 12:46:02PM +0200, Daniel Lezcano wrote:
>> The tegra3 and big.LITTLE architecture have different cpu latencies.
>> This API allows to specify a different cpu latency for a specific cpu.
>>
>> With the previous patches, we use the per cpuidle device states pointer,
>> this function overrides this pointer.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
>> ---
>> drivers/cpuidle/cpuidle.c | 17 +++++++++++++++++
>> include/linux/cpuidle.h | 10 +++++++---
>> 2 files changed, 24 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c
>> index 199878a..3b21b68 100644
>> --- a/drivers/cpuidle/cpuidle.c
>> +++ b/drivers/cpuidle/cpuidle.c
>> @@ -456,6 +456,23 @@ void cpuidle_unregister_device(struct cpuidle_device *dev)
>>
>> EXPORT_SYMBOL_GPL(cpuidle_unregister_device);
>>
>> +int cpuidle_register_states(struct cpuidle_device *dev,
>> + struct cpuidle_state *states,
>> + int state_count)
>> +{
>> + if (!dev || !states)
>> + return -EINVAL;
>> +
>> + if (state_count <= 0)
>> + return -EINVAL;
>> +
>> + dev->states = states;
>> + dev->state_count = state_count;
>> +
>> + return 0;
>> +}
>> +EXPORT_SYMBOL_GPL(cpuidle_register_state);
>> +
>> #ifdef CONFIG_SMP
>>
>
> Looks good... apart from the fact that the function definition says
> cpuidle_register_stateS and the exported symbol is cpuidle_register_state...
Ok, fixed. 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
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
