From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [lm-sensors] Patch[2/2] Adding Interrupt Handling Support to Coretemp Date: Wed, 8 Dec 2010 07:06:50 -0800 Message-ID: <20101208150650.GA28273@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]:42761 "EHLO imr4.ericy.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754832Ab0LHPHf (ORCPT ); Wed, 8 Dec 2010 10:07:35 -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" , "linux-acpi@vger.kernel.org" , "Zhang, Rui" , "lm-sensors@lm-sensors.org" , "lenb@kernel.org" On Wed, Dec 08, 2010 at 04:17:27AM -0500, R, Durgadoss wrote: > Hi Fenghua, > > This patch, the second one in the series adds the interrupt > Handling support for the thermal thresholds. When an interrupt > occurs, this is notified to the user space(so that the user space > can take some action) by a netlink event. > > This patch is generated against the stable Linux-2.6 tree. > > This patch depends on the "Adding notification to thermal > Framework" patch, which can be downloaded from here: > https://patchwork.kernel.org/patch/282042/ > > Kindly review and merge. > ------------------------------------------------------------------ > From: Durgadoss R > > Date: Tue, 7 Dec 2010 04:10:41 +0530 > Subject: [PATCH 2/2] Adding_Threshold_Interrupt_Handling_to_Coretemp > > This patch adds the interrupt handling for the core/package > thermal thresholds in coretemp. The interrupts are routed to coretemp > via therm_throt.c. Whenever an interrupt occurs, a notification is sent > in the form of a netlink event to the user space. > > Signed-off-by: Durgadoss R > After browsing through this and the previous patch, I think the functionality should reside outside and be independent of hwmon. arch/x86/kernel/cpu or drivers/platform/x86 might be better places for it, though I am not really in a position to make that judgement. Guenter