[PATCH] ARM: EXYNOS4: CPUIDLE Support
Kukjin Kim
kgene.kim at samsung.com
Wed Mar 16 02:01:53 EDT 2011
Kyungmin Park wrote:
>
> On Tue, Mar 15, 2011 at 10:57 PM, Kukjin Kim <kgene.kim at samsung.com>
wrote:
> > Jaecheol Lee wrote:
> >>
> >> This patch supports cpuidle framework for EXYNOS4210. Currently,
> >> Only one idle state is possible to use, but more idle states can
> >> be added following by this patch.
> >>
> >> Signed-off-by: Jaecheol Lee <jc.lee at samsung.com>
> >> ---
> >> arch/arm/mach-exynos4/Makefile | 1 +
> >> arch/arm/mach-exynos4/cpuidle.c | 87
> >> +++++++++++++++++++++++++++++++++++++++
> >> 2 files changed, 88 insertions(+), 0 deletions(-)
> >> create mode 100644 arch/arm/mach-exynos4/cpuidle.c
> >>
> >> diff --git a/arch/arm/mach-exynos4/Makefile b/arch/arm/mach-
> > exynos4/Makefile
> >> index 461ae57..bf206fa 100644
> >> --- a/arch/arm/mach-exynos4/Makefile
> >> +++ b/arch/arm/mach-exynos4/Makefile
> >> @@ -16,6 +16,7 @@ obj-$(CONFIG_CPU_EXYNOS4210) += cpu.o init.o
> > clock.o
> >> irq-combiner.o
> >> obj-$(CONFIG_CPU_EXYNOS4210) += cpu.o init.o clock.o irq-combiner.o
> >> obj-$(CONFIG_CPU_EXYNOS4210) += setup-i2c0.o time.o gpiolib.o
irq-eint.o
> >> dma.o
> >> obj-$(CONFIG_CPU_FREQ) += cpufreq.o
> >> +obj-$(CONFIG_CPU_IDLE) += cpuidle.o
> >>
> >> obj-$(CONFIG_SMP) += platsmp.o headsmp.o
> >> obj-$(CONFIG_LOCAL_TIMERS) += localtimer.o
> >> diff --git a/arch/arm/mach-exynos4/cpuidle.c b/arch/arm/mach-
> >> exynos4/cpuidle.c
> >> new file mode 100644
> >> index 0000000..a543a18
> >> --- /dev/null
> >> +++ b/arch/arm/mach-exynos4/cpuidle.c
> >> @@ -0,0 +1,87 @@
> >> +/* linux/arch/arm/mach-exynos4/cpuidle.c
> >> + *
> >> + * Copyright (c) 2011 Samsung Electronics Co., Ltd.
> >> + * http://www.samsung.com
> >> + *
> >> + * This program is free software; you can redistribute it and/or
modify
> >> + * it under the terms of the GNU General Public License version 2 as
> >> + * published by the Free Software Foundation.
> >> +*/
> >> +
> >> +#include <linux/kernel.h>
> >> +#include <linux/init.h>
> >> +#include <linux/cpuidle.h>
> >> +#include <linux/io.h>
> >> +
> >> +#include <asm/proc-fns.h>
> >> +
> >> +static int exynos4_enter_idle(struct cpuidle_device *dev,
> >> + struct cpuidle_state *state);
> >> +
> >> +static struct cpuidle_state exynos4_cpuidle_set[] = {
> >> + [0] = {
> >> + .enter = exynos4_enter_idle,
> >> + .exit_latency = 1,
> >> + .target_residency = 100000,
> >> + .flags = CPUIDLE_FLAG_TIME_VALID,
> >> + .name = "IDLE",
> >> + .desc = "ARM clock gating(WFI)",
> >> + },
> >> +};
> >> +
> >> +static DEFINE_PER_CPU(struct cpuidle_device, exynos4_cpuidle_device);
> >> +
> >> +static struct cpuidle_driver exynos4_idle_driver = {
> >> + .name = "exynos4_idle",
> >> + .owner = THIS_MODULE,
> >> +};
> >> +
> >> +static int exynos4_enter_idle(struct cpuidle_device *dev,
> >> + struct cpuidle_state *state)
> >> +{
> >> + struct timeval before, after;
> >> + int idle_time;
> >> +
> >> + local_irq_disable();
> >> + do_gettimeofday(&before);
> >> +
> >> + cpu_do_idle();
> >> +
> >> + do_gettimeofday(&after);
> >> + local_irq_enable();
> >> + idle_time = (after.tv_sec - before.tv_sec) * USEC_PER_SEC +
> >> + (after.tv_usec - before.tv_usec);
> >> +
> >> + return idle_time;
> >> +}
> >> +
> >> +static int __init exynos4_init_cpuidle(void)
> >> +{
> >> + int i, max_cpuidle_state, cpu_id;
> >> + struct cpuidle_device *device;
> >> +
> >> + cpuidle_register_driver(&exynos4_idle_driver);
> >> +
> >> + for_each_cpu(cpu_id, cpu_online_mask) {
> >> + device = &per_cpu(exynos4_cpuidle_device, cpu_id);
> >> + device->cpu = cpu_id;
> >> +
> >> + device->state_count = (sizeof(exynos4_cpuidle_set) /
> >> + sizeof(struct
> > cpuidle_state));
> >> +
> >> + max_cpuidle_state = device->state_count;
> >> +
> >> + for (i = 0; i < max_cpuidle_state; i++) {
> >> + memcpy(&device->states[i],
&exynos4_cpuidle_set[i],
> >> + sizeof(struct cpuidle_state));
> >> + }
> >> +
> >> + if (cpuidle_register_device(device)) {
> >> + printk(KERN_ERR "CPUidle register device
> > failed\n,");
> >> + return -EIO;
> >> + }
> >> + }
> >> + return 0;
> >> +}
> >> +
> >> +device_initcall(exynos4_init_cpuidle);
> >> --
> >> 1.7.1
> >
> > Cc'ed Kyungmin Park.
> >
> > Hmm...I'm thinking that we need more discussion on this CPUIDLE later,
> > about method, IDLE states and so on.
>
> It's not good decision. As cpufreq, support the basic cpuidle at
> mainline kernel, and add it more states at later.
> Except the state name "IDLE", I hope it will be changed to "C1" for
> support cpuidle tool.
>
> Mr. Lee will support the C states soon.
>
> I'm okay with Mr. Lee patch.
Yeah, if required, we can update it later.
Ok, his code looks ok to me.
Will apply.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
More information about the linux-arm-kernel
mailing list