Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262169AbVAKUyY (ORCPT ); Tue, 11 Jan 2005 15:54:24 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262691AbVAKUyY (ORCPT ); Tue, 11 Jan 2005 15:54:24 -0500 Received: from smtp-102-tuesday.noc.nerim.net ([62.4.17.102]:17670 "EHLO mallaury.noc.nerim.net") by vger.kernel.org with ESMTP id S262169AbVAKUyS (ORCPT ); Tue, 11 Jan 2005 15:54:18 -0500 Date: Tue, 11 Jan 2005 21:56:33 +0100 From: Jean Delvare To: Jonas Munsin , Simone Piunno Cc: sensors@Stimpy.netroedge.com, djg@pdp8.net, LKML , Greg KH Subject: Re: 2.6.10-mm2: it87 sensor driver stops CPU fan Message-Id: <20050111215633.3c2c08ea.khali@linux-fr.org> In-Reply-To: <20050111202437.GA2914@nemo.sby.abo.fi> References: <200501102341.44949.pioppo@ferrara.linux.it> <20050111202437.GA2914@nemo.sby.abo.fi> Reply-To: LM Sensors , LKML X-Mailer: Sylpheed version 1.0.0 (GTK+ 1.2.10; i686-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 979 Lines: 26 > Ok, thanks for doing the thinking ;), here is the modified patch > (it87.c_2.6.10-jm3-corrected_manual_pwm_20050111.diff). In addition to > the above change, it also refreshes fan_main_ctrl in the update > routine, as suggested by Jean on IRC. > > - adds manual PWM > - removes buggy "reset" module parameter > - fixes some whitespaces Tested, works as intended for me. Simone, you might revert http://khali.linux-fr.org/devel/i2c/linux-2.6/linux-2.6.10-rc3-i2c-it87-manual-pwm.diff and apply this new patch instead. This won't solve the polarity issue but at least the fans won't stop when you load the it87 driver. We can now move to the second part of the plan. Stay tuned :) -- Jean Delvare http://khali.linux-fr.org/ - 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/