From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [lm-sensors] [Patch] Adding threshold support to coretemp Date: Tue, 14 Dec 2010 10:24:35 -0800 Message-ID: <20101214182435.GA32358@ericsson.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Received: from imr4.ericy.com ([198.24.6.8]:41733 "EHLO imr4.ericy.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757116Ab0LNSZc (ORCPT ); Tue, 14 Dec 2010 13:25:32 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: "R, Durgadoss" Cc: "Yu, Fenghua" , Chen Gong , "lenb@kernel.org" , "khali@linux-fr.org" , "linux-acpi@vger.kernel.org" , "lm-sensors@lm-sensors.org" On Tue, Dec 14, 2010 at 06:55:35AM -0500, R, Durgadoss wrote: > Hi Jean/Fenghua, > > I am submitting a patch to enable core thermal threshold > Support to coretemp.c. There are two core thermal thresholds > available through sysfs interfaces temp1_core_thresh[0/1]. > > The expectation is that thresh0 is lesser than the current temperature > and thresh1 is higher than the current temperature. Whenever the current > temperature crosses these limits, an interrupt is generated. > This interrupt is handles by the user space to do power > Management via CPU throttling, etc.. > > This patch is generated against stable Linux-2.6 kernel. > > Kindly review and merge. > ------------------------------------------------------------------ > From: Durgadoss R > > Date: Tue, 14 Dec 2010 05:10:27 +0530 > Subject: [PATCH] Adding_threshold_support_to_coretemp > > This patch adds the core thermal threshold support to coretemp.c. > These thresholds can be read/written using the sysfs interface > temp1_core_thresh[0/1]. These can be used to generate interrupts, > to do dynamic power management. > > Signed-off-by: Durgadoss R > I am not inclined to look further into this patch until the ABI questions are resolved. As written, the patch gets my NACK simply because it introduces random driver specific new ABI attributes. Guenter