All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simone Piunno <pioppo@ferrara.linux.it>
To: "Jean Delvare" <khali@linux-fr.org>
Cc: sensors@stimpy.netroedge.com, "Jonas Munsin" <jmunsin@iki.fi>,
	djg@pdp8.net, "Greg KH" <greg@kroah.com>,
	"LKML" <linux-kernel@vger.kernel.org>
Subject: Re: 2.6.10-mm2: it87 sensor driver stops CPU fan
Date: Tue, 11 Jan 2005 22:04:59 +0100	[thread overview]
Message-ID: <200501112205.02322.pioppo@ferrara.linux.it> (raw)
In-Reply-To: <YN0o4rkI.1105435582.0805630.khali@localhost>

On Tuesday 11 January 2005 10:26, Jean Delvare wrote:

> What you have here is this default configuration, i.e. all fans are
> supposedly off. Of course it isn't the case, I assume that your fans
> are running at full speed when you turn your computer on. 

Yes, thank you for your analisys.

> Additionally, the first fan control output was turned to manual PWM
> control mode, full speed. The driver supposedly doesn't do that, I
> guess you did it yourself through the sysfs interface?

Of course, sorry.

> > > Do you know what kind of it87 chip you do have? There are three of
> > > them, IT8705F, IT8712F and a SIS950 clone (mostly similar to the
> > > IT8705F).
> See /sys/bus/i2c/devices/0-0290/name. If it says it8712 it's an IT8712F,
> if it says it87 it is a less featured IT8705F or clone. After looking at

pioppo@roentgen ~ $ cat /sys/devices/platform/i2c-0/0-0290/name
it87

> 2* I would then add a check to the it87 driver, which completely disables
> the fan speed control interface if the initial configuration looks weird
> (all fans supposedly stopped and polarity set to "active low"). This
> should protect users of the driver who have a faulty BIOS.

If the driver can perform a similar guess, couldn't it also activate a reverse 
polarity mode as well?  I think all systems boot with with full-speed fan, so 
any value you found at loading time should be the full-speed one, shouln't 
it?

BTW:
  I'm writing a report to giga-byte.

Cheers,
  Simone
-- 
http://thisurlenablesemailtogetthroughoverzealousspamfilters.org

  parent reply	other threads:[~2005-01-11 21:03 UTC|newest]

Thread overview: 59+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-08  0:50 2.6.10-mm2: it87 sensor driver stops CPU fan Simone Piunno
2005-01-08  9:34 ` Jean Delvare
2005-05-19  6:25   ` Jean Delvare
2005-01-10 22:41   ` Simone Piunno
2005-05-19  6:25     ` Simone Piunno
2005-01-11  9:26     ` Jean Delvare
2005-05-19  6:25       ` Jean Delvare
2005-01-11 20:24       ` Jonas Munsin
2005-05-19  6:25         ` Jonas Munsin
2005-01-11 20:56         ` Jean Delvare
2005-01-11 22:41         ` Greg KH
2005-05-19  6:25           ` Greg KH
2005-01-11 21:04       ` Simone Piunno [this message]
2005-01-12  9:44         ` Jean Delvare
2005-05-19  6:25           ` Jean Delvare
2005-01-12 22:27       ` Jonas Munsin
2005-05-19  6:25         ` Jonas Munsin
2005-01-13 23:29         ` Greg KH
2005-05-19  6:25           ` Greg KH
2005-01-14 14:40           ` Jean Delvare
2005-05-19  6:25             ` Jean Delvare
2005-01-15 15:30             ` [PATCH 2.6] I2C: Allow it87 pwm reconfiguration Jean Delvare
2005-05-19  6:25               ` Jean Delvare
2005-01-15 17:18               ` Simone Piunno
2005-05-19  6:25                 ` Simone Piunno
2005-01-19 23:23               ` Greg KH
2005-05-19  6:25                 ` Greg KH
2005-01-15 15:54             ` 2.6.10-mm2: it87 sensor driver stops CPU fan Simone Piunno
2005-05-19  6:25               ` Simone Piunno
2005-01-15 16:55               ` Jean Delvare
2005-05-19  6:25                 ` Jean Delvare
2005-01-16 22:32                 ` Simone Piunno
2005-05-19  6:25                   ` Simone Piunno
2005-01-17 19:19                   ` Jean Delvare
2005-05-19  6:25                     ` Jean Delvare
2005-01-17 19:37                     ` 2.6 Series Mem Mgmt Chris Bookholt
2005-01-17 19:56                       ` Brian Gerst
2005-01-08 16:20 ` 2.6.10-mm2: it87 sensor driver stops CPU fan Jean Delvare
2005-01-08 19:23   ` Simone Piunno
2005-05-19  6:25     ` Simone Piunno
2005-01-10 19:23     ` Simone Piunno
2005-05-19  6:25       ` Simone Piunno
2005-01-10 19:34       ` Jean Delvare
2005-01-19 20:19         ` Nicolas Pitre
2005-05-19  6:25           ` Nicolas Pitre
2005-01-19 20:52           ` Jean Delvare
2005-05-19  6:25             ` Jean Delvare
2005-01-19 22:10             ` Nicolas Pitre
2005-05-19  6:25               ` Nicolas Pitre
2005-01-20 11:08               ` Jean Delvare
2005-05-19  6:25                 ` Jean Delvare
2005-01-20 16:01                 ` Nicolas Pitre
2005-05-19  6:25                   ` Nicolas Pitre
2005-01-20 16:28                   ` Jean Delvare
2005-05-19  6:25                     ` Jean Delvare
2005-01-20 21:19                     ` Nicolas Pitre
2005-05-19  6:25                       ` Nicolas Pitre
2005-01-21  6:46                       ` Jean Delvare
2005-05-19  6:25                         ` Jean Delvare

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200501112205.02322.pioppo@ferrara.linux.it \
    --to=pioppo@ferrara.linux.it \
    --cc=djg@pdp8.net \
    --cc=greg@kroah.com \
    --cc=jmunsin@iki.fi \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sensors@stimpy.netroedge.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.