* [PATCH v2] cpuidle: Add Kconfig.arm and move calxeda, kirkwood and zynq
@ 2013-07-03 8:49 kpark3469
2013-07-14 23:26 ` Rafael J. Wysocki
0 siblings, 1 reply; 6+ messages in thread
From: kpark3469 @ 2013-07-03 8:49 UTC (permalink / raw)
To: linux-pm, daniel.lezcano; +Cc: keun-o.park, rjw
From: Sahara <keun-o.park@windriver.com>
This patch adds Kconfig.arm for cpuidle drivers for ARM. And, this
also moves calxeda, kirkwood and zynq to Kconfig.arm. Like cpufreq
menu, "CPU Idle" menu is added to drivers/cpuidle/Kconfig.
Signed-off-by: Sahara <keun-o.park@windriver.com>
---
drivers/cpuidle/Kconfig | 19 +++++++------------
drivers/cpuidle/Kconfig.arm | 21 +++++++++++++++++++++
drivers/cpuidle/Makefile | 8 +++++---
3 files changed, 33 insertions(+), 15 deletions(-)
create mode 100644 drivers/cpuidle/Kconfig.arm
diff --git a/drivers/cpuidle/Kconfig b/drivers/cpuidle/Kconfig
index 81de5d9..b3fb81d 100644
--- a/drivers/cpuidle/Kconfig
+++ b/drivers/cpuidle/Kconfig
@@ -1,5 +1,6 @@
+menu "CPU Idle"
-menuconfig CPU_IDLE
+config CPU_IDLE
bool "CPU idle PM support"
default y if ACPI || PPC_PSERIES
select CPU_IDLE_GOV_LADDER if (!NO_HZ && !NO_HZ_IDLE)
@@ -29,19 +30,13 @@ config CPU_IDLE_GOV_MENU
bool "Menu governor (for tickless system)"
default y
-config CPU_IDLE_CALXEDA
- bool "CPU Idle Driver for Calxeda processors"
- depends on ARCH_HIGHBANK
- help
- Select this to enable cpuidle on Calxeda processors.
-
-config CPU_IDLE_ZYNQ
- bool "CPU Idle Driver for Xilinx Zynq processors"
- depends on ARCH_ZYNQ
- help
- Select this to enable cpuidle on Xilinx Zynq processors.
+menu "ARM CPU Idle Drivers"
+depends on ARM
+source "drivers/cpuidle/Kconfig.arm"
+endmenu
endif
config ARCH_NEEDS_CPU_IDLE_COUPLED
def_bool n
+endmenu
diff --git a/drivers/cpuidle/Kconfig.arm b/drivers/cpuidle/Kconfig.arm
new file mode 100644
index 0000000..df69259
--- /dev/null
+++ b/drivers/cpuidle/Kconfig.arm
@@ -0,0 +1,21 @@
+#
+# ARM CPU Idle drivers
+#
+
+config ARM_HIGHBANK_CPUIDLE
+ bool "CPU Idle Driver for Calxeda processors"
+ depends on ARCH_HIGHBANK
+ help
+ Select this to enable cpuidle on Calxeda processors.
+
+config ARM_KIRKWOOD_CPUIDLE
+ bool "CPU Idle Driver for Marvell Kirkwood SoCs"
+ depends on ARCH_KIRKWOOD
+ help
+ This adds the CPU Idle driver for Marvell Kirkwood SoCs.
+
+config ARM_ZYNQ_CPUIDLE
+ bool "CPU Idle Driver for Xilinx Zynq processors"
+ depends on ARCH_ZYNQ
+ help
+ Select this to enable cpuidle on Xilinx Zynq processors.
diff --git a/drivers/cpuidle/Makefile b/drivers/cpuidle/Makefile
index 8767a7b..77cf767 100644
--- a/drivers/cpuidle/Makefile
+++ b/drivers/cpuidle/Makefile
@@ -5,6 +5,8 @@
obj-y += cpuidle.o driver.o governor.o sysfs.o governors/
obj-$(CONFIG_ARCH_NEEDS_CPU_IDLE_COUPLED) += coupled.o
-obj-$(CONFIG_CPU_IDLE_CALXEDA) += cpuidle-calxeda.o
-obj-$(CONFIG_ARCH_KIRKWOOD) += cpuidle-kirkwood.o
-obj-$(CONFIG_CPU_IDLE_ZYNQ) += cpuidle-zynq.o
+##################################################################################
+# ARM SoC drivers
+obj-$(CONFIG_ARM_HIGHBANK_CPUIDLE) += cpuidle-calxeda.o
+obj-$(CONFIG_ARM_KIRKWOOD_CPUIDLE) += cpuidle-kirkwood.o
+obj-$(CONFIG_ARM_ZYNQ_CPUIDLE) += cpuidle-zynq.o
--
1.7.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] cpuidle: Add Kconfig.arm and move calxeda, kirkwood and zynq
2013-07-03 8:49 [PATCH v2] cpuidle: Add Kconfig.arm and move calxeda, kirkwood and zynq kpark3469
@ 2013-07-14 23:26 ` Rafael J. Wysocki
2013-07-15 14:18 ` Daniel Lezcano
0 siblings, 1 reply; 6+ messages in thread
From: Rafael J. Wysocki @ 2013-07-14 23:26 UTC (permalink / raw)
To: kpark3469, daniel.lezcano; +Cc: linux-pm, keun-o.park
On Wednesday, July 03, 2013 05:49:55 PM kpark3469@gmail.com wrote:
> From: Sahara <keun-o.park@windriver.com>
>
> This patch adds Kconfig.arm for cpuidle drivers for ARM. And, this
> also moves calxeda, kirkwood and zynq to Kconfig.arm. Like cpufreq
> menu, "CPU Idle" menu is added to drivers/cpuidle/Kconfig.
>
> Signed-off-by: Sahara <keun-o.park@windriver.com>
Daniel: Yes, no, maybe?
Rafael
> ---
> drivers/cpuidle/Kconfig | 19 +++++++------------
> drivers/cpuidle/Kconfig.arm | 21 +++++++++++++++++++++
> drivers/cpuidle/Makefile | 8 +++++---
> 3 files changed, 33 insertions(+), 15 deletions(-)
> create mode 100644 drivers/cpuidle/Kconfig.arm
>
> diff --git a/drivers/cpuidle/Kconfig b/drivers/cpuidle/Kconfig
> index 81de5d9..b3fb81d 100644
> --- a/drivers/cpuidle/Kconfig
> +++ b/drivers/cpuidle/Kconfig
> @@ -1,5 +1,6 @@
> +menu "CPU Idle"
>
> -menuconfig CPU_IDLE
> +config CPU_IDLE
> bool "CPU idle PM support"
> default y if ACPI || PPC_PSERIES
> select CPU_IDLE_GOV_LADDER if (!NO_HZ && !NO_HZ_IDLE)
> @@ -29,19 +30,13 @@ config CPU_IDLE_GOV_MENU
> bool "Menu governor (for tickless system)"
> default y
>
> -config CPU_IDLE_CALXEDA
> - bool "CPU Idle Driver for Calxeda processors"
> - depends on ARCH_HIGHBANK
> - help
> - Select this to enable cpuidle on Calxeda processors.
> -
> -config CPU_IDLE_ZYNQ
> - bool "CPU Idle Driver for Xilinx Zynq processors"
> - depends on ARCH_ZYNQ
> - help
> - Select this to enable cpuidle on Xilinx Zynq processors.
> +menu "ARM CPU Idle Drivers"
> +depends on ARM
> +source "drivers/cpuidle/Kconfig.arm"
> +endmenu
>
> endif
>
> config ARCH_NEEDS_CPU_IDLE_COUPLED
> def_bool n
> +endmenu
> diff --git a/drivers/cpuidle/Kconfig.arm b/drivers/cpuidle/Kconfig.arm
> new file mode 100644
> index 0000000..df69259
> --- /dev/null
> +++ b/drivers/cpuidle/Kconfig.arm
> @@ -0,0 +1,21 @@
> +#
> +# ARM CPU Idle drivers
> +#
> +
> +config ARM_HIGHBANK_CPUIDLE
> + bool "CPU Idle Driver for Calxeda processors"
> + depends on ARCH_HIGHBANK
> + help
> + Select this to enable cpuidle on Calxeda processors.
> +
> +config ARM_KIRKWOOD_CPUIDLE
> + bool "CPU Idle Driver for Marvell Kirkwood SoCs"
> + depends on ARCH_KIRKWOOD
> + help
> + This adds the CPU Idle driver for Marvell Kirkwood SoCs.
> +
> +config ARM_ZYNQ_CPUIDLE
> + bool "CPU Idle Driver for Xilinx Zynq processors"
> + depends on ARCH_ZYNQ
> + help
> + Select this to enable cpuidle on Xilinx Zynq processors.
> diff --git a/drivers/cpuidle/Makefile b/drivers/cpuidle/Makefile
> index 8767a7b..77cf767 100644
> --- a/drivers/cpuidle/Makefile
> +++ b/drivers/cpuidle/Makefile
> @@ -5,6 +5,8 @@
> obj-y += cpuidle.o driver.o governor.o sysfs.o governors/
> obj-$(CONFIG_ARCH_NEEDS_CPU_IDLE_COUPLED) += coupled.o
>
> -obj-$(CONFIG_CPU_IDLE_CALXEDA) += cpuidle-calxeda.o
> -obj-$(CONFIG_ARCH_KIRKWOOD) += cpuidle-kirkwood.o
> -obj-$(CONFIG_CPU_IDLE_ZYNQ) += cpuidle-zynq.o
> +##################################################################################
> +# ARM SoC drivers
> +obj-$(CONFIG_ARM_HIGHBANK_CPUIDLE) += cpuidle-calxeda.o
> +obj-$(CONFIG_ARM_KIRKWOOD_CPUIDLE) += cpuidle-kirkwood.o
> +obj-$(CONFIG_ARM_ZYNQ_CPUIDLE) += cpuidle-zynq.o
>
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] cpuidle: Add Kconfig.arm and move calxeda, kirkwood and zynq
2013-07-14 23:26 ` Rafael J. Wysocki
@ 2013-07-15 14:18 ` Daniel Lezcano
2013-07-15 23:44 ` Rafael J. Wysocki
0 siblings, 1 reply; 6+ messages in thread
From: Daniel Lezcano @ 2013-07-15 14:18 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: kpark3469, linux-pm, keun-o.park
On 07/15/2013 01:26 AM, Rafael J. Wysocki wrote:
> On Wednesday, July 03, 2013 05:49:55 PM kpark3469@gmail.com wrote:
>> From: Sahara <keun-o.park@windriver.com>
>>
>> This patch adds Kconfig.arm for cpuidle drivers for ARM. And, this
>> also moves calxeda, kirkwood and zynq to Kconfig.arm. Like cpufreq
>> menu, "CPU Idle" menu is added to drivers/cpuidle/Kconfig.
>>
>> Signed-off-by: Sahara <keun-o.park@windriver.com>
>
> Daniel: Yes, no, maybe?
Actually I picked the patch in my tree, I was waiting for the -rc1
before sending the pull request.
I have 3 other patches on top of it I want to send for review before
sending the pull request (as you take 1PR/week).
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] cpuidle: Add Kconfig.arm and move calxeda, kirkwood and zynq
2013-07-15 14:18 ` Daniel Lezcano
@ 2013-07-15 23:44 ` Rafael J. Wysocki
2013-07-16 9:46 ` Daniel Lezcano
0 siblings, 1 reply; 6+ messages in thread
From: Rafael J. Wysocki @ 2013-07-15 23:44 UTC (permalink / raw)
To: Daniel Lezcano; +Cc: kpark3469, linux-pm, keun-o.park
On Monday, July 15, 2013 04:18:58 PM Daniel Lezcano wrote:
> On 07/15/2013 01:26 AM, Rafael J. Wysocki wrote:
> > On Wednesday, July 03, 2013 05:49:55 PM kpark3469@gmail.com wrote:
> >> From: Sahara <keun-o.park@windriver.com>
> >>
> >> This patch adds Kconfig.arm for cpuidle drivers for ARM. And, this
> >> also moves calxeda, kirkwood and zynq to Kconfig.arm. Like cpufreq
> >> menu, "CPU Idle" menu is added to drivers/cpuidle/Kconfig.
> >>
> >> Signed-off-by: Sahara <keun-o.park@windriver.com>
> >
> > Daniel: Yes, no, maybe?
>
> Actually I picked the patch in my tree, I was waiting for the -rc1
> before sending the pull request.
>
> I have 3 other patches on top of it I want to send for review before
> sending the pull request (as you take 1PR/week).
OK, I'm dropping it then.
I'll assume you'll handle ARM-specific changes from now on, so I'm going to
ignore them going forward.
Thanks,
Rafael
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] cpuidle: Add Kconfig.arm and move calxeda, kirkwood and zynq
2013-07-15 23:44 ` Rafael J. Wysocki
@ 2013-07-16 9:46 ` Daniel Lezcano
2013-07-16 12:30 ` Rafael J. Wysocki
0 siblings, 1 reply; 6+ messages in thread
From: Daniel Lezcano @ 2013-07-16 9:46 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: kpark3469, linux-pm, keun-o.park
On 07/16/2013 01:44 AM, Rafael J. Wysocki wrote:
> On Monday, July 15, 2013 04:18:58 PM Daniel Lezcano wrote:
>> On 07/15/2013 01:26 AM, Rafael J. Wysocki wrote:
>>> On Wednesday, July 03, 2013 05:49:55 PM kpark3469@gmail.com wrote:
>>>> From: Sahara <keun-o.park@windriver.com>
>>>>
>>>> This patch adds Kconfig.arm for cpuidle drivers for ARM. And, this
>>>> also moves calxeda, kirkwood and zynq to Kconfig.arm. Like cpufreq
>>>> menu, "CPU Idle" menu is added to drivers/cpuidle/Kconfig.
>>>>
>>>> Signed-off-by: Sahara <keun-o.park@windriver.com>
>>>
>>> Daniel: Yes, no, maybe?
>>
>> Actually I picked the patch in my tree, I was waiting for the -rc1
>> before sending the pull request.
>>
>> I have 3 other patches on top of it I want to send for review before
>> sending the pull request (as you take 1PR/week).
>
> OK, I'm dropping it then.
>
> I'll assume you'll handle ARM-specific changes from now on, so I'm going to
> ignore them going forward.
Ok, please ignore my previous PR, it was based on your pm-next branch
which has this patch.
I rebased my branch adding this patch and I will send you another
updated PR.
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] cpuidle: Add Kconfig.arm and move calxeda, kirkwood and zynq
2013-07-16 9:46 ` Daniel Lezcano
@ 2013-07-16 12:30 ` Rafael J. Wysocki
0 siblings, 0 replies; 6+ messages in thread
From: Rafael J. Wysocki @ 2013-07-16 12:30 UTC (permalink / raw)
To: Daniel Lezcano; +Cc: kpark3469, linux-pm, keun-o.park
On Tuesday, July 16, 2013 11:46:08 AM Daniel Lezcano wrote:
> On 07/16/2013 01:44 AM, Rafael J. Wysocki wrote:
> > On Monday, July 15, 2013 04:18:58 PM Daniel Lezcano wrote:
> >> On 07/15/2013 01:26 AM, Rafael J. Wysocki wrote:
> >>> On Wednesday, July 03, 2013 05:49:55 PM kpark3469@gmail.com wrote:
> >>>> From: Sahara <keun-o.park@windriver.com>
> >>>>
> >>>> This patch adds Kconfig.arm for cpuidle drivers for ARM. And, this
> >>>> also moves calxeda, kirkwood and zynq to Kconfig.arm. Like cpufreq
> >>>> menu, "CPU Idle" menu is added to drivers/cpuidle/Kconfig.
> >>>>
> >>>> Signed-off-by: Sahara <keun-o.park@windriver.com>
> >>>
> >>> Daniel: Yes, no, maybe?
> >>
> >> Actually I picked the patch in my tree, I was waiting for the -rc1
> >> before sending the pull request.
> >>
> >> I have 3 other patches on top of it I want to send for review before
> >> sending the pull request (as you take 1PR/week).
> >
> > OK, I'm dropping it then.
> >
> > I'll assume you'll handle ARM-specific changes from now on, so I'm going to
> > ignore them going forward.
>
> Ok, please ignore my previous PR, it was based on your pm-next branch
> which has this patch.
>
> I rebased my branch adding this patch and I will send you another
> updated PR.
OK, thanks a lot!
Rafael
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-07-16 12:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-03 8:49 [PATCH v2] cpuidle: Add Kconfig.arm and move calxeda, kirkwood and zynq kpark3469
2013-07-14 23:26 ` Rafael J. Wysocki
2013-07-15 14:18 ` Daniel Lezcano
2013-07-15 23:44 ` Rafael J. Wysocki
2013-07-16 9:46 ` Daniel Lezcano
2013-07-16 12:30 ` Rafael J. Wysocki
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.