* linux-next: build warning after merge of the cpufreq tree
@ 2011-04-27 1:10 Stephen Rothwell
2011-04-27 1:32 ` Dave Jones
0 siblings, 1 reply; 6+ messages in thread
From: Stephen Rothwell @ 2011-04-27 1:10 UTC (permalink / raw)
To: Dave Jones; +Cc: linux-next, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 856 bytes --]
Hi Dave,
After merging the cpufreq tree, today's linux-next build (powerpc
ppc64_defconfig) produced this warning:
drivers/cpufreq/Kconfig:67:warning: choice value used outside its choice group
drivers/cpufreq/Kconfig:75:warning: choice value used outside its choice group
drivers/cpufreq/Kconfig:84:warning: choice value used outside its choice group
drivers/cpufreq/Kconfig:93:warning: choice value used outside its choice group
drivers/cpufreq/Kconfig:105:warning: choice value used outside its choice group
I am not sure what has caused this. The only change in the cpufreq tree
to that file is commit e002ba3328a2 ("[CPUFREQ] Move x86 cpufreq drivers
to drivers/cpufreq/") which rearranged a few things but nothing obvious.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: linux-next: build warning after merge of the cpufreq tree 2011-04-27 1:10 linux-next: build warning after merge of the cpufreq tree Stephen Rothwell @ 2011-04-27 1:32 ` Dave Jones 2011-04-27 1:54 ` Stephen Rothwell 0 siblings, 1 reply; 6+ messages in thread From: Dave Jones @ 2011-04-27 1:32 UTC (permalink / raw) To: Stephen Rothwell; +Cc: linux-next, linux-kernel On Wed, Apr 27, 2011 at 11:10:24AM +1000, Stephen Rothwell wrote: > Hi Dave, > > After merging the cpufreq tree, today's linux-next build (powerpc > ppc64_defconfig) produced this warning: > > drivers/cpufreq/Kconfig:67:warning: choice value used outside its choice group > drivers/cpufreq/Kconfig:75:warning: choice value used outside its choice group > drivers/cpufreq/Kconfig:84:warning: choice value used outside its choice group > drivers/cpufreq/Kconfig:93:warning: choice value used outside its choice group > drivers/cpufreq/Kconfig:105:warning: choice value used outside its choice group > > I am not sure what has caused this. The only change in the cpufreq tree > to that file is commit e002ba3328a2 ("[CPUFREQ] Move x86 cpufreq drivers > to drivers/cpufreq/") which rearranged a few things but nothing obvious. As the commit message suggests, per-arch cpufreq drivers are moving to drivers/cpufreq/ So far, I've only moved the x86 ones. As a side-effect of this, the source "drivers/cpufreq/Kconfig" is no longer a per-arch thing, as it now gets included from kernel/power/Kconfig. I suspect the warnings are happening because on the unconverted archs, drivers/cpufreq/Kconfig is now getting included twice. If you kill the 'source' line in kernel/power/Kconfig, does this make things go back to normal ? (Check that cpufreq still shows up in the resulting config afterwards) If so, I'll make that change after all archs have moved their drivers. Dave ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: build warning after merge of the cpufreq tree 2011-04-27 1:32 ` Dave Jones @ 2011-04-27 1:54 ` Stephen Rothwell 2011-04-27 2:41 ` Dave Jones 2011-04-29 20:12 ` Dave Jones 0 siblings, 2 replies; 6+ messages in thread From: Stephen Rothwell @ 2011-04-27 1:54 UTC (permalink / raw) To: Dave Jones; +Cc: linux-next, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1113 bytes --] Hi Dave, On Tue, 26 Apr 2011 21:32:17 -0400 Dave Jones <davej@redhat.com> wrote: > > As the commit message suggests, per-arch cpufreq drivers are moving to drivers/cpufreq/ > So far, I've only moved the x86 ones. As a side-effect of this, the > source "drivers/cpufreq/Kconfig" is no longer a per-arch thing, as it now gets > included from kernel/power/Kconfig. > > I suspect the warnings are happening because on the unconverted archs, drivers/cpufreq/Kconfig > is now getting included twice. If you kill the 'source' line in kernel/power/Kconfig, > does this make things go back to normal ? (Check that cpufreq still shows up in > the resulting config afterwards) Removing the source line in kernel/power/Kconfig makes the warnings go away and the only change in the generated config file is this: @@ -465,10 +465,6 @@ # CONFIG_PM_RUNTIME is not set CONFIG_PM=y # CONFIG_PM_DEBUG is not set - -# -# CPU Frequency scaling -# CONFIG_SECCOMP=y CONFIG_ISA_DMA_API=y -- Cheers, Stephen Rothwell sfr@canb.auug.org.au http://www.canb.auug.org.au/~sfr/ [-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: build warning after merge of the cpufreq tree 2011-04-27 1:54 ` Stephen Rothwell @ 2011-04-27 2:41 ` Dave Jones 2011-04-29 20:12 ` Dave Jones 1 sibling, 0 replies; 6+ messages in thread From: Dave Jones @ 2011-04-27 2:41 UTC (permalink / raw) To: Stephen Rothwell; +Cc: linux-next, linux-kernel On Wed, Apr 27, 2011 at 11:54:27AM +1000, Stephen Rothwell wrote: > Removing the source line in kernel/power/Kconfig makes the warnings go > away and the only change in the generated config file is this: > > @@ -465,10 +465,6 @@ > # CONFIG_PM_RUNTIME is not set > CONFIG_PM=y > # CONFIG_PM_DEBUG is not set > - > -# > -# CPU Frequency scaling > -# > CONFIG_SECCOMP=y > CONFIG_ISA_DMA_API=y Ok, thanks for testing. I'll fix something up. Dave ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: build warning after merge of the cpufreq tree 2011-04-27 1:54 ` Stephen Rothwell 2011-04-27 2:41 ` Dave Jones @ 2011-04-29 20:12 ` Dave Jones 2011-05-02 5:06 ` Stephen Rothwell 1 sibling, 1 reply; 6+ messages in thread From: Dave Jones @ 2011-04-29 20:12 UTC (permalink / raw) To: Stephen Rothwell; +Cc: linux-next, linux-kernel On Wed, Apr 27, 2011 at 11:54:27AM +1000, Stephen Rothwell wrote: > Hi Dave, > > On Tue, 26 Apr 2011 21:32:17 -0400 Dave Jones <davej@redhat.com> wrote: > > > > As the commit message suggests, per-arch cpufreq drivers are moving to drivers/cpufreq/ > > So far, I've only moved the x86 ones. As a side-effect of this, the > > source "drivers/cpufreq/Kconfig" is no longer a per-arch thing, as it now gets > > included from kernel/power/Kconfig. > > > > I suspect the warnings are happening because on the unconverted archs, drivers/cpufreq/Kconfig > > is now getting included twice. If you kill the 'source' line in kernel/power/Kconfig, > > does this make things go back to normal ? (Check that cpufreq still shows up in > > the resulting config afterwards) > > Removing the source line in kernel/power/Kconfig makes the warnings go > away and the only change in the generated config file is this: I've moved this work onto a separate move-drivers branch. Could you track that in next too please? (Apply after fixes & next) I'm hoping this branch will be fairly short-lived, but it depends how quickly other architecture maintainers move their drivers over too. thanks, Dave ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: build warning after merge of the cpufreq tree 2011-04-29 20:12 ` Dave Jones @ 2011-05-02 5:06 ` Stephen Rothwell 0 siblings, 0 replies; 6+ messages in thread From: Stephen Rothwell @ 2011-05-02 5:06 UTC (permalink / raw) To: Dave Jones; +Cc: linux-next, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1986 bytes --] Hi Dave, On Fri, 29 Apr 2011 16:12:42 -0400 Dave Jones <davej@redhat.com> wrote: > > I've moved this work onto a separate move-drivers branch. Could you track that in next too please? > (Apply after fixes & next) > > I'm hoping this branch will be fairly short-lived, but it depends how quickly > other architecture maintainers move their drivers over too. I have added that new branch starting today. I called it cpufreq-move. Thanks for adding your subsystem tree as a participant of linux-next. As you may know, this is not a judgment of your code. The purpose of linux-next is for integration testing and to lower the impact of conflicts between subsystems in the next merge window. You will need to ensure that the patches/commits in your tree/series have been: * submitted under GPL v2 (or later) and include the Contributor's Signed-off-by, * posted to the relevant mailing list, * reviewed by you (or another maintainer of your subsystem tree), * successfully unit tested, and * destined for the current or next Linux merge window. Basically, this should be just what you would send to Linus (or ask him to fetch). It is allowed to be rebased if you deem it necessary. -- Cheers, Stephen Rothwell sfr@canb.auug.org.au Legal Stuff: By participating in linux-next, your subsystem tree contributions are public and will be included in the linux-next trees. You may be sent e-mail messages indicating errors or other issues when the patches/commits from your subsystem tree are merged and tested in linux-next. These messages may also be cross-posted to the linux-next mailing list, the linux-kernel mailing list, etc. The linux-next tree project and IBM (my employer) make no warranties regarding the linux-next project, the testing procedures, the results, the e-mails, etc. If you don't agree to these ground rules, let me know and I'll remove your tree from participation in linux-next. [-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-05-02 5:06 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-04-27 1:10 linux-next: build warning after merge of the cpufreq tree Stephen Rothwell 2011-04-27 1:32 ` Dave Jones 2011-04-27 1:54 ` Stephen Rothwell 2011-04-27 2:41 ` Dave Jones 2011-04-29 20:12 ` Dave Jones 2011-05-02 5:06 ` Stephen Rothwell
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).