lkml.org 
[lkml]   [2007]   [Jun]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] powernow-k8: depend on acpi-processor for SMP systems
    On Thu 2007-05-17 06:24:22, Ed Sweetman wrote:
    > Pavel Machek wrote:
    > >Hi!
    > >
    > >
    > >>powernow-k8 uses PSB BIOS tables to read frequency info on UP systems, but
    > >>on SMP it requires the acpi-processor driver. Kconfig should be updated
    > >>accordingly to avoid the issues that users are running into.
    > >>
    > >>http://bugzilla.kernel.org/show_bug.cgi?id=8075
    > >>https://bugs.gentoo.org/show_bug.cgi?id=178585
    > >>
    > >>Signed-off-by: Daniel Drake <dsd@gentoo.org>
    > >>
    > >>Index: linux/arch/i386/kernel/cpu/cpufreq/Kconfig
    > >>===================================================================
    > >>--- linux.orig/arch/i386/kernel/cpu/cpufreq/Kconfig
    > >>+++ linux/arch/i386/kernel/cpu/cpufreq/Kconfig
    > >>@@ -81,6 +81,7 @@ config X86_POWERNOW_K7_ACPI
    > >> config X86_POWERNOW_K8
    > >> tristate "AMD Opteron/Athlon64 PowerNow!"
    > >> select CPU_FREQ_TABLE
    > >>+ select ACPI_PROCESSOR if SMP
    > >> depends on EXPERIMENTAL
    > >> help
    > >> This adds the CPUFreq driver for mobile AMD Opteron/Athlon64
    > >> processors.
    > >>
    > >
    > >It is still possible to run SMP kernel on UP machine -- so this sounds
    > >wrong.
    >
    > It's possible to run a kernel with drivers for hardware you dont have.
    > But not having the dependencies correct for those drivers for that
    > hardware you dont have just because it's possible you can run a kernel
    > with those drivers without that hardware isn't correct. Got it?

    Breaking working configs is not fine, got it? We are having the thing
    called "stable release" here. Feel free to fix code to decide during
    runtime.

    Pavel

    --
    (english) http://www.livejournal.com/~pavelmachek
    (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-06-04 12:55    [W:7.686 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site