From mboxrd@z Thu Jan 1 00:00:00 1970 From: robherring2@gmail.com (Rob Herring) Date: Tue, 19 Mar 2013 10:13:51 -0500 Subject: [PATCH 2/2] cpuidle / calxeda: remove redundant Kconfig option In-Reply-To: <514877B5.6080200@linaro.org> References: <1363080476-26555-1-git-send-email-daniel.lezcano@linaro.org> <1363080476-26555-2-git-send-email-daniel.lezcano@linaro.org> <51485876.307@gmail.com> <514877B5.6080200@linaro.org> Message-ID: <514880AF.7010404@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 03/19/2013 09:35 AM, Daniel Lezcano wrote: > On 03/19/2013 01:22 PM, Rob Herring wrote: >> On 03/12/2013 04:27 AM, Daniel Lezcano wrote: >>> When the CPU_IDLE and the ARCH_HIGHBANK options are set it is >>> pointless to define a new option CPU_IDLE_CALXEDA because it >>> is redundant. >>> >>> The Makefile drivers directory contains a condition to compile >>> the cpuidle drivers: >>> >>> obj-$(CONFIG_CPU_IDLE) += cpuidle/ >>> >>> Hence, if CPU_IDLE is not set we won't enter this directory. >>> >>> This patch removes the useless Kconfig option and replaces the >>> condition in the Makefile by CONFIG_ARCH_HIGHBANK. >> >> If I have multiple platforms including highbank compiled in, but want to >> disable cpuidle just for highbank, then you can't disable it at compile >> time. > > Could you elaborate a bit ? Is it today possible to have multiple > platforms in a single kernel ? You've heard of single zImage, right? Vexpress, mvebu, highbank, imx, omap2+, socfpga, vt8500 off the top of my head as of 3.8. Rob