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.
Thanks,
Rafael
> 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; }
>
--
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
