Subject: [PATCH 2/2] cpuidle / calxeda: remove redundant Kconfig option

From: [email-masked] (Rob Herring)
Date: Tue, 19 Mar 2013 10:13:51 -0500

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