Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756642Ab1ESH0z (ORCPT ); Thu, 19 May 2011 03:26:55 -0400 Received: from ch1ehsobe003.messaging.microsoft.com ([216.32.181.183]:53447 "EHLO ch1outboundpool.messaging.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755520Ab1ESH0x (ORCPT ); Thu, 19 May 2011 03:26:53 -0400 X-SpamScore: -8 X-BigFish: VPS-8(zz1432N98dKzz1202hzz8275bhz32i668h839h61h) X-Spam-TCS-SCL: 0:0 X-Forefront-Antispam-Report: KIP:(null);UIP:(null);IPVD:NLI;H:ausb3twp01.amd.com;RD:none;EFVD:NLI X-WSS-ID: 0LLFM0O-01-CLZ-02 X-M-MSG: Date: Thu, 19 May 2011 09:26:46 +0200 From: Andreas Herrmann To: Matthew Garrett CC: , , , , Subject: Re: [PATCH 1/5] acpi-cpufreq: Add support for modern AMD CPUs Message-ID: <20110519072646.GB19481@alberich.amd.com> References: <1305651819-25660-1-git-send-email-mjg@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <1305651819-25660-1-git-send-email-mjg@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-OriginatorOrg: amd.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2021 Lines: 45 On Tue, May 17, 2011 at 01:03:35PM -0400, Matthew Garrett wrote: > The programming model for P-states on modern AMD CPUs is very similar to > that of Intel and VIA. It makes sense to consolidate this support into one > driver rather than duplicating functionality between two of them. This > patch adds support for AMDs with hardware P-state control to acpi-cpufreq. I think it's a good idea to move the stuff into the more general driver. (And this way powernow-k8 becomes what it is supposed to be -- relevant for K8 CPUs only.) From a first glance most stuff looks fine. (but detailed review and testing still outstanding.) > Signed-off-by: Matthew Garrett > --- > arch/x86/include/asm/cpufeature.h | 1 + > arch/x86/include/asm/msr-index.h | 2 + > arch/x86/kernel/cpu/cpufreq/acpi-cpufreq.c | 43 ++++++++++++++++++++++++---- > arch/x86/kernel/cpu/scattered.c | 1 + > 4 files changed, 41 insertions(+), 6 deletions(-) > > diff --git a/arch/x86/include/asm/cpufeature.h b/arch/x86/include/asm/cpufeature.h > index 91f3e087..fee7089 100644 > --- a/arch/x86/include/asm/cpufeature.h > +++ b/arch/x86/include/asm/cpufeature.h > @@ -174,6 +174,7 @@ > #define X86_FEATURE_PLN (7*32+ 5) /* Intel Power Limit Notification */ > #define X86_FEATURE_PTS (7*32+ 6) /* Intel Package Thermal Status */ > #define X86_FEATURE_DTS (7*32+ 7) /* Digital Thermal Sensor */ > +#define X86_FEATURE_POWERNOW (7*32+ 8) /* AMD frequency scaling */ I think it shouldn't be named like that. I'd rather call it HW_PSTATE. The more general term POWERNOW is also associated with the old FID/VID control method of changing frequencies on AMD CPUs. Also AMD documentation refers to this feature as HwPstate. Regards, Andreas -- 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/