From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Fitzhardinge Subject: Re: [PATCH 4/8] acpi-cpufreq: remove ACPI from speedstep-centrino driver Date: Wed, 02 Aug 2006 09:39:25 -0700 Message-ID: <44D0D53D.3010108@goop.org> References: <44CE5015.2090200@linux.intel.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <44CE5015.2090200@linux.intel.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: cpufreq-bounces@lists.linux.org.uk Errors-To: cpufreq-bounces+glkc-cpufreq=m.gmane.org+glkc-cpufreq=m.gmane.org@lists.linux.org.uk Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Alexey Starikovskiy Cc: cpufreq@lists.linux.org.uk, Dave Jones Alexey Starikovskiy wrote: > speedstep-centrino.c | 270 > +++++++-------------------------------------------- > 1 file changed, 40 insertions(+), 230 deletions(-) > > remove ACPI from speedstep-centrino driver > > Signed-off: Denis Sadykov > Signed-off-by: Venkatesh Pallipadi > Signed-off-by: Alexey Starikovskiy > > Index: linux-2.6.17/arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c > =================================================================== > --- > linux-2.6.17.orig/arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c > 2006-07-11 20:13:44.000000000 +0000 > +++ linux-2.6.17/arch/i386/kernel/cpu/cpufreq/speedstep-centrino.c > 2006-07-13 18:54:07.000000000 +0000 > @@ -2,36 +2,36 @@ > * cpufreq driver for Enhanced SpeedStep, as found in Intel's Pentium > * M (part of the Centrino chipset). > * > - * Since the original Pentium M, most new Intel CPUs support Enhanced > - * SpeedStep. > - * > * Despite the "SpeedStep" in the name, this is almost entirely unlike > * traditional SpeedStep. > * > * Modelled on speedstep.c > * > * Copyright (C) 2003 Jeremy Fitzhardinge > + * > + * WARNING WARNING WARNING > + * > + * This driver manipulates the PERF_CTL MSR, which is only somewhat > + * documented. While it seems to work on my laptop, it has not been > + * tested anywhere else, and it may not work for you, do strange > + * things or simply crash. > */ > [...] > #define PFX "speedstep-centrino: " > -#define MAINTAINER "cpufreq@lists.linux.org.uk" > +#define MAINTAINER "Jeremy Fitzhardinge " You're reverting a change I made recently. Did you mean to? J