Subject: [GIT PULL] cpufreq: OMAP updates for v3.4

From: [email-masked] (Kevin Hilman)
Date: Thu, 15 Mar 2012 07:25:43 -0700

Grazvydas Ignotas <[email-masked]> writes:

> On Wed, Mar 14, 2012 at 7:22 PM, Kevin Hilman <[email-masked]> wrote:
>> Grazvydas Ignotas <[email-masked]> writes:
>>
>> [...]
>>
>>>> Russell King (1):
>>>> ? ? ?cpufreq: OMAP driver depends CPUfreq tables
>>>
>>> It seems this one got messed up, it says "default ARCH_OMAP2PLUS"
>>> instead of depends.
>>
>> That was intentional. ?What do you think is messed up about it?
>>
>> It's the same as saying:
>>
>> ? ? depends on ARCH_OMAP2PLUS
>> ? ? default y
>
> Hm, somehow I've never encountered this syntax, but it more looks like
> it means "default y if ARCH_OMAP2PLUS" instead of that.
>
> Checked this with menuconfig now:
>   default ARCH_OMAP2PLUS
> resolves to:
>   ARCH_HAS_CPUFREQ [=y] && CPU_FREQ [=y] && ARM [=y]
>
> ..which might cause build break when OMAP is not selected, and
>
>   depends on ARCH_OMAP2PLUS
>   default y
> resolves to:
>   ARCH_HAS_CPUFREQ [=y] && CPU_FREQ [=y] && ARM [=y] && ARCH_OMAP2PLUS [=y]
>
> ..which should be randconfig friendly.

Yeah, you're right.  Feel free to send a patch using your comments above
to describe that the current form is not randconfig friendly.

Kevin