[V3 patch 09/19] ARM: OMAP3: cpuidle: use init/exit common routine

Kevin Hilman khilman at linaro.org
Fri Apr 19 18:19:01 EDT 2013


Daniel Lezcano <daniel.lezcano at linaro.org> writes:

> Remove the duplicated code and use the cpuidle common code for initialization.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano at linaro.org>

Reviewed-by: Kevin Hilman <khilman at linaro.org>

> ---
>  arch/arm/mach-omap2/cpuidle34xx.c |   18 ++----------------
>  1 file changed, 2 insertions(+), 16 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/cpuidle34xx.c b/arch/arm/mach-omap2/cpuidle34xx.c
> index 027a787..cca045c 100644
> --- a/arch/arm/mach-omap2/cpuidle34xx.c
> +++ b/arch/arm/mach-omap2/cpuidle34xx.c
> @@ -26,6 +26,7 @@
>  #include <linux/cpuidle.h>
>  #include <linux/export.h>
>  #include <linux/cpu_pm.h>
> +#include <asm/cpuidle.h>
>  
>  #include "powerdomain.h"
>  #include "clockdomain.h"
> @@ -259,8 +260,6 @@ static int omap3_enter_idle_bm(struct cpuidle_device *dev,
>  	return ret;
>  }
>  
> -static DEFINE_PER_CPU(struct cpuidle_device, omap3_idle_dev);
> -
>  static struct cpuidle_driver omap3_idle_driver = {
>  	.name             = "omap3_idle",
>  	.owner            = THIS_MODULE,
> @@ -336,8 +335,6 @@ static struct cpuidle_driver omap3_idle_driver = {
>   */
>  int __init omap3_idle_init(void)
>  {
> -	struct cpuidle_device *dev;
> -
>  	mpu_pd = pwrdm_lookup("mpu_pwrdm");
>  	core_pd = pwrdm_lookup("core_pwrdm");
>  	per_pd = pwrdm_lookup("per_pwrdm");
> @@ -346,16 +343,5 @@ int __init omap3_idle_init(void)
>  	if (!mpu_pd || !core_pd || !per_pd || !cam_pd)
>  		return -ENODEV;
>  
> -	cpuidle_register_driver(&omap3_idle_driver);
> -
> -	dev = &per_cpu(omap3_idle_dev, smp_processor_id());
> -	dev->cpu = 0;
> -
> -	if (cpuidle_register_device(dev)) {
> -		printk(KERN_ERR "%s: CPUidle register device failed\n",
> -		       __func__);
> -		return -EIO;
> -	}
> -
> -	return 0;
> +	return cpuidle_register(&omap3_idle_driver, NULL);
>  }



More information about the linux-arm-kernel mailing list