Thread (1 message) 1 message, 1 author, 2012-03-15

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

From: Kevin Hilman <hidden>
Date: 2012-03-15 14:25:43
Also in: linux-omap

Grazvydas Ignotas [off-list ref] writes:
On Wed, Mar 14, 2012 at 7:22 PM, Kevin Hilman [off-list ref] wrote:
quoted
Grazvydas Ignotas [off-list ref] writes:

[...]
quoted
quoted
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help