From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: Re: [Xenomai-core] [PATCH] detect conflict with INPUT_PCSPKR From: Philippe Gerum In-Reply-To: <17614.15855.660608.773754@domain.hid> References: <44CB2884.80801@domain.hid> <17611.36969.814874.846387@domain.hid> <17611.37633.881446.443324@domain.hid> <44CC5D59.3080405@domain.hid> <17614.12685.956714.372133@domain.hid> <44CE350F.5060303@domain.hid> <17614.15855.660608.773754@domain.hid> Content-Type: text/plain Date: Tue, 01 Aug 2006 15:35:04 +0200 Message-Id: <1154439304.4963.1.camel@domain.hid> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Reply-To: rpm@xenomai.org List-Id: "Xenomai life and development \(bug reports, patches, discussions\)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gilles Chanteperdrix Cc: Jan Kiszka , xenomai-core On Mon, 2006-07-31 at 19:29 +0200, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: > > Gilles Chanteperdrix wrote: > > > Jan Kiszka wrote: > > > > No problem, but only when combining with a > > > > > > > > comment "Switch off CONFIG_INPUT_PCSPKR to use Xenomai" (or so) > > > > depends on !X86_TSC && X86 && INPUT_PCSPKR > > > > > > > > Otherwise the user will be left alone here with a non-selectable Xenomai > > > > option... > > > > > > > > This actually remind me of my suggestion some months ago to add Kconfig > > > > warnings for CONFIG_CPU_FREQ & friends. Might be a good chance to catch > > > > this all. Is scripts/Kconfig.frag the preferred place to add it? > > > > > > > > > Why not? Here is a second patch that follows your suggestions. > > > > No objections! Looks good to me. But shouldn't we add APM as well? > > With APM, as well as with warnings for 2.4. > Ack. Applique directement stp. A+ > plain text document attachment (xeno-kconfig-warnings.diff) > Index: scripts/Kconfig.frag > =================================================================== > --- scripts/Kconfig.frag (revision 1402) > +++ scripts/Kconfig.frag (working copy) > @@ -1,7 +1,20 @@ > > menu "Real-time sub-system" > > +comment "WARNING! You enabled APM, CPU Frequency scaling or use of ACPI" > + depends on APM || CPU_FREQ || ACPI_PROCESSOR > +comment "processor C states as idle handler (ACPI 'processor' option)." > + depends on APM || CPU_FREQ || ACPI_PROCESSOR > +comment "These options are known to cause troubles with Xenomai." > + depends on APM || CPU_FREQ || ACPI_PROCESSOR > + > +comment "NOTE: Xenomai conflicts with PC speaker support." > + depends on !X86_TSC && X86 && INPUT_PCSPKR > +comment "(menu Device Drivers/Input device support/Miscellaneous devices)" > + depends on !X86_TSC && X86 && INPUT_PCSPKR > + > config XENOMAI > + depends on X86_TSC || !X86 || !INPUT_PCSPKR > bool "Xenomai" > default y > select IPIPE > Index: ksrc/arch/i386/Config.in > =================================================================== > --- ksrc/arch/i386/Config.in (revision 1402) > +++ ksrc/arch/i386/Config.in (working copy) > @@ -1,6 +1,12 @@ > mainmenu_option next_comment > comment 'Real-time sub-system' > > +if [ "$CONFIG_APM" != "n" -o "$CONFIG_APM_CPU_IDLE" != "n" -o "$CONFIG_APM_DISPLAY_BLANK" != "n" -o "$CONFIG_ACPI_PROCESSOR" != "n" ]; then > + comment "WARNING! You enabled APM or use of ACPI processor C states as" > + comment "idle handler (ACPI 'processor' option)." > + comment "These options are known to cause troubles with Xenomai" > +fi > + > if [ "$CONFIG_IPIPE" = "n" ]; then > comment "Xenomai depends on Adeos interrupt pipeline" > else > _______________________________________________ > Xenomai-core mailing list > Xenomai-core@domain.hid > https://mail.gna.org/listinfo/xenomai-core -- Philippe.