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. -- Gilles Chanteperdrix.
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 Xenomaiemail@example.com https://mail.gna.org/listinfo/xenomai-core