public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <linux@roeck-us.net>
To: Laszlo Papp <lpapp@kde.org>
Cc: LKML <linux-kernel@vger.kernel.org>, lm-sensors@lm-sensors.org
Subject: Re: [lm-sensors] Tachometer speed returned rather than absolute fan speed?
Date: Mon, 10 Mar 2014 08:11:46 -0700	[thread overview]
Message-ID: <20140310151146.GA26903@roeck-us.net> (raw)
In-Reply-To: <CAOMwXhMRtNaxtA7=cc7ur_Qf3iwZAHi=sk-Ujp=hggGA1O5u+Q@mail.gmail.com>

On Mon, Mar 10, 2014 at 01:50:01PM +0000, Laszlo Papp wrote:
> On Mon, Mar 10, 2014 at 1:26 PM, Guenter Roeck <linux@roeck-us.net> wrote:
> > On 03/10/2014 02:59 AM, Laszlo Papp wrote:
> >>>
> >>> The reason is (most likely) that your fan input does not have a pull-up
> >>> resistor. Per datasheet, the fan inputs need a 10kOhm pull-up resistor. I
> >>> confirmed this with my test board - with the pull-up resistor, inputs read
> >>> 0, Without pull-up, the reported value is 1, which translates to 30 RPM. You
> >>> might also need the 10 uF capacitor on the FB pin.
> >>
> >>
> >> We have both of them in place as per requirement for the chip as far
> >> as I understand the datasheet. The circuit would not work properly
> >> without it.
> >>>
> >>> sudo i2cdump -y -f 1 0x48
> >>> No size specified (using byte-data access)
> >>>       0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f    0123456789abcdef
> >>> 00: 00 00 0a 0a ff ff 00 00 00 00 00 00 01 01 00 00    ..??........??..
> >>> 10: 01 01 01 01 1f 1f 02 02 91 91 91 91 91 91 91 91    ????????????????
> >>> 20: 00 00 0a 0a ff ff 00 00 00 00 00 00 01 01 00 00    ..??........??..
> >>
> >>
> >> /usr/sbin/i2cdump -y -f 1 0x48
> >> No size specified (using byte-data access)
> >>       0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f    0123456789abcdef
> >> 00: 00 00 3a 3a fc fc b4 b4 00 00 00 00 01 01 01 01    ..::????....????
> >> 10: 09 09 01 01 1c 1c 02 02 91 91 91 91 91 91 91 91    ????????????????
> >> 20: 00 00 3a 3a fc fc b4 b4 00 00 00 00 01 01 01 01    ..::????....????
> >> 30: 09 09 01 01 1c 1c 02 02 91 91 91 91 91 91 91 91    ????????????????
> >> 40: 00 00 3a 3a fc fc b4 b4 00 00 00 00 01 01 01 01    ..::????....????
> >> 50: 08 08 01 01 1c 1c 02 02 91 91 91 91 91 91 91 91    ????????????????
> >> 60: 00 00 3a 3a fc fc b4 b4 00 00 00 00 01 01 01 01    ..::????....????
> >> 70: 08 08 01 01 1c 1c 02 02 91 91 91 91 91 91 91 91    ????????????????
> >> 80: 00 00 3a 3a fc fc b4 b4 00 00 00 00 01 01 01 01    ..::????....????
> >> 90: 08 08 01 01 1c 1c 02 02 91 91 91 91 91 91 91 91    ????????????????
> >> a0: 00 00 3a 3a fc fc b4 b4 00 00 00 00 01 01 01 01    ..::????....????
> >> b0: 08 08 01 01 1c 1c 02 02 91 91 91 91 91 91 91 91    ????????????????
> >> c0: 00 00 3a 3a fc fc b4 b4 00 00 00 00 01 01 01 01    ..::????....????
> >> d0: 08 08 01 01 1c 1c 02 02 91 91 91 91 91 91 91 91    ????????????????
> >> e0: 00 00 3a 3a fc fc b4 b4 00 00 00 00 01 01 01 01    ..::????....????
> >> f0: 08 08 01 01 1c 1c 02 02 91 91 91 91 91 91 91 91    ????????????????
> >>
> >>> groeck@desktop:~$ sensors max6651-i2c-1-48
> >>> max6651-i2c-1-48
> >>> Adapter: i2c-diolan-u2c at bus 003 device 002
> >>> fan1:          30 RPM  (div = 4)
> >>> fan2:           0 RPM
> >>> fan3:          30 RPM
> >>> fan4:          30 RPM
> >>
> >>
> >> sensors max6651-i2c-1-48
> >> max6651-i2c-1-48
> >> Adapter: DaVinci I2C adapter
> >> fan1:          30 RPM  (div = 4)
> >> fan2:          30 RPM
> >> fan3:         420 RPM
> >> fan4:          30 RPM
> >>
> >>> Only fan2 (TACH1 chip input) has the resistor installed.
> >>
> >>
> >> We have it both on fan1 and fan4.
> >>
> >> If I use our scope for checking the tachometer, I see level zero on
> >> the scope, but 1 when checking the open collector level, since that is
> >> the purpose of the pull-up resistor.
> >>
> > With the pull-up installed, you should always see a high voltage on the
> > scope or with a volt meter. The scope would not see or be able to know
> > that the pin is OC if the pull-up resistor is there.
> >
> > If it is truly OC, it would be floating and might show low. If you can,
> > I would suggest to manually add a 10k resistor against VCC to the open
> > connector and see what happens.
> >
> > Another way to find out for sure might be to disconnect power and
> > measure resistance between the tachometer pin and VCC.
> 
> Perhaps I was not clear. So, we do have both the resistors and
> capacitor as required. I double checked the schematics as well as the
> pins coming from the fan and that was zero. The tacho0 input, however,
> was 1.
> 

How can the pin coming from the fan be zero, and the tach0 input be 1
at the same time ? Those should be the same ?

Anyway, I don't think I can help you further. All I can say is that
the chip operates as expected on my test board.

Guenter

  reply	other threads:[~2014-03-10 15:11 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-07 14:48 Tachometer speed returned rather than absolute fan speed? Laszlo Papp
2014-03-07 15:08 ` Laszlo Papp
2014-03-07 15:25 ` Jean Delvare
2014-03-07 15:31   ` Laszlo Papp
2014-03-07 15:37     ` Jean Delvare
2014-03-07 15:47       ` Laszlo Papp
2014-03-07 18:17         ` Guenter Roeck
2014-03-08 23:50           ` [lm-sensors] " Guenter Roeck
2014-03-09  6:36             ` Laszlo Papp
2014-03-09  8:04               ` Guenter Roeck
2014-03-09 12:44                 ` Laszlo Papp
2014-03-10  9:59                 ` Laszlo Papp
2014-03-10 13:26                   ` Guenter Roeck
2014-03-10 13:50                     ` Laszlo Papp
2014-03-10 15:11                       ` Guenter Roeck [this message]
2014-03-10 18:00                         ` Laszlo Papp
2014-04-08  9:19             ` Pavel Machek
2014-03-10  7:11   ` George Spelvin

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=20140310151146.GA26903@roeck-us.net \
    --to=linux@roeck-us.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.org \
    --cc=lpapp@kde.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox