On 09/19/2012 09:49 PM, Rafael J. Wysocki wrote:
> On Wednesday, September 19, 2012, Daniel Lezcano wrote:
>> These two functions are never used.
> This is not the case. pm_genpd_attach_cpuidle() and
> pm_genpd_detach_cpuidle() use them.
>
> Please be more careful.
Oops, sorry.
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
>> ---
>> drivers/cpuidle/driver.c | 29 +----------------------------
>> include/linux/cpuidle.h | 4 ----
>> 2 files changed, 1 insertions(+), 32 deletions(-)
>>
>> diff --git a/drivers/cpuidle/driver.c b/drivers/cpuidle/driver.c
>> index 58bf3b1..40cd3f3 100644
>> --- a/drivers/cpuidle/driver.c
>> +++ b/drivers/cpuidle/driver.c
>> @@ -16,7 +16,6 @@
>>
>> static struct cpuidle_driver *cpuidle_curr_driver;
>> DEFINE_SPINLOCK(cpuidle_driver_lock);
>> -int cpuidle_driver_refcount;
>>
>> static void __cpuidle_register_driver(struct cpuidle_driver *drv)
>> {
>> @@ -90,34 +89,8 @@ void cpuidle_unregister_driver(struct cpuidle_driver *drv)
>> }
>>
>> spin_lock(&cpuidle_driver_lock);
>> -
>> - if (!WARN_ON(cpuidle_driver_refcount > 0))
>> - cpuidle_curr_driver = NULL;
>> -
>> + cpuidle_curr_driver = NULL;
>> spin_unlock(&cpuidle_driver_lock);
>> }
>>
>> EXPORT_SYMBOL_GPL(cpuidle_unregister_driver);
>> -
>> -struct cpuidle_driver *cpuidle_driver_ref(void)
>> -{
>> - struct cpuidle_driver *drv;
>> -
>> - spin_lock(&cpuidle_driver_lock);
>> -
>> - drv = cpuidle_curr_driver;
>> - cpuidle_driver_refcount++;
>> -
>> - spin_unlock(&cpuidle_driver_lock);
>> - return drv;
>> -}
>> -
>> -void cpuidle_driver_unref(void)
>> -{
>> - spin_lock(&cpuidle_driver_lock);
>> -
>> - if (!WARN_ON(cpuidle_driver_refcount <= 0))
>> - cpuidle_driver_refcount--;
>> -
>> - spin_unlock(&cpuidle_driver_lock);
>> -}
>> diff --git a/include/linux/cpuidle.h b/include/linux/cpuidle.h
>> index 279b1ea..5abf62b 100644
>> --- a/include/linux/cpuidle.h
>> +++ b/include/linux/cpuidle.h
>> @@ -145,8 +145,6 @@ extern void disable_cpuidle(void);
>> extern int cpuidle_idle_call(void);
>> extern int cpuidle_register_driver(struct cpuidle_driver *drv);
>> extern struct cpuidle_driver *cpuidle_get_driver(void);
>> -extern struct cpuidle_driver *cpuidle_driver_ref(void);
>> -extern void cpuidle_driver_unref(void);
>> extern void cpuidle_unregister_driver(struct cpuidle_driver *drv);
>> extern int cpuidle_register_device(struct cpuidle_device *dev);
>> extern void cpuidle_unregister_device(struct cpuidle_device *dev);
>> @@ -169,8 +167,6 @@ static inline int cpuidle_idle_call(void) { return -ENODEV; }
>> static inline int cpuidle_register_driver(struct cpuidle_driver *drv)
>> {return -ENODEV; }
>> static inline struct cpuidle_driver *cpuidle_get_driver(void) {return NULL; }
>> -static inline struct cpuidle_driver *cpuidle_driver_ref(void) {return NULL; }
>> -static inline void cpuidle_driver_unref(void) {}
>> static inline void cpuidle_unregister_driver(struct cpuidle_driver *drv) { }
>> static inline int cpuidle_register_device(struct cpuidle_device *dev)
>> {return -ENODEV; }
>>
>
> _______________________________________________
> linaro-dev mailing list
> linaro-dev@xxxxxxxxxxxxxxxx
> http://lists.linaro.org/mailman/listinfo/linaro-dev
>
--
<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
