Grazvydas Ignotas <notasas@xxxxxxxxx> writes: > On Wed, Mar 14, 2012 at 7:22 PM, Kevin Hilman <khilman@xxxxxx> wrote: >> Grazvydas Ignotas <notasas@xxxxxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
