Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756849Ab1BQTXU (ORCPT ); Thu, 17 Feb 2011 14:23:20 -0500 Received: from zone0.gcu-squad.org ([212.85.147.21]:6724 "EHLO services.gcu-squad.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752604Ab1BQTXT (ORCPT ); Thu, 17 Feb 2011 14:23:19 -0500 Date: Thu, 17 Feb 2011 20:23:07 +0100 From: Jean Delvare To: Guenter Roeck Cc: Clemens Ladisch , lm-sensors@lm-sensors.org, "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] hwmon: (k10temp) add support for AMD Family 12h/14h CPUs Message-ID: <20110217202307.7281f403@endymion.delvare> In-Reply-To: <20110217183934.GB20387@ericsson.com> References: <4D5CDAD0.7030006@ladisch.de> <20110217172639.GA19982@ericsson.com> <20110217190838.381962f6@endymion.delvare> <20110217183934.GB20387@ericsson.com> X-Mailer: Claws Mail 3.7.5 (GTK+ 2.20.1; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1549 Lines: 43 On Thu, 17 Feb 2011 10:39:34 -0800, Guenter Roeck wrote: > On Thu, Feb 17, 2011 at 01:08:38PM -0500, Jean Delvare wrote: > > On Thu, 17 Feb 2011 09:26:39 -0800, Guenter Roeck wrote: > > > On Thu, Feb 17, 2011 at 03:22:40AM -0500, Clemens Ladisch wrote: > > > > Add the PCI ID to support the internal temperature sensor of the > > > > AMD "Llano" and "Brazos" processor families. > > > > > > > > Signed-off-by: Clemens Ladisch > > > > > > Applied, thanks. > > > > > > Since it meets the requirements for a stable patch, I applied it to the > > > patch list for 2.6.38 and added Cc: stable@kernel.org. > > > > Can we have a sensors-detect patch too? > > > Prepared that, but I am getting the following from the ddr3 patch. > > prog/detect/sensors-detect > Useless use of private variable in void context at prog/detect/sensors-detect line 5287. > > Code is > } elsif ($device_type => 9 && $device_type <= 11) { > > Should that be > } elsif ($device_type >= 9 && $device_type <= 11) { > instead ? Yes, you're totally right. > Odd, I didn't notice that before when I tried the ddr3 patch. This is the part I modified from Clemens' original patch. What's really odd is that I didn't notice the warning on my end. I see it now... Sorry, I've just fixed it. -- Jean Delvare -- 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/