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 15:11:46 +0000 [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
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
WARNING: multiple messages have this Message-ID (diff)
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
next prev parent reply other threads:[~2014-03-10 15:11 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-07 14:48 [lm-sensors] Tachometer speed returned rather than absolute fan speed? Laszlo Papp
2014-03-07 14:48 ` Laszlo Papp
2014-03-07 15:08 ` [lm-sensors] " Laszlo Papp
2014-03-07 15:08 ` Laszlo Papp
2014-03-07 15:25 ` [lm-sensors] " Jean Delvare
2014-03-07 15:25 ` Jean Delvare
2014-03-07 15:31 ` [lm-sensors] " Laszlo Papp
2014-03-07 15:31 ` Laszlo Papp
2014-03-07 15:37 ` [lm-sensors] " Jean Delvare
2014-03-07 15:37 ` Jean Delvare
2014-03-07 15:47 ` [lm-sensors] " Laszlo Papp
2014-03-07 15:47 ` Laszlo Papp
2014-03-07 18:17 ` [lm-sensors] " Guenter Roeck
2014-03-07 18:17 ` Guenter Roeck
2014-03-08 23:50 ` [lm-sensors] " Guenter Roeck
2014-03-08 23:50 ` Guenter Roeck
2014-03-09 6:36 ` Laszlo Papp
2014-03-09 6:36 ` Laszlo Papp
2014-03-09 8:04 ` Guenter Roeck
2014-03-09 8:04 ` Guenter Roeck
2014-03-09 12:44 ` Laszlo Papp
2014-03-09 12:44 ` Laszlo Papp
2014-03-10 9:59 ` Laszlo Papp
2014-03-10 9:59 ` Laszlo Papp
2014-03-10 13:26 ` Guenter Roeck
2014-03-10 13:26 ` Guenter Roeck
2014-03-10 13:50 ` Laszlo Papp
2014-03-10 13:50 ` Laszlo Papp
2014-03-10 15:11 ` Guenter Roeck [this message]
2014-03-10 15:11 ` Guenter Roeck
2014-03-10 18:00 ` Laszlo Papp
2014-03-10 18:00 ` Laszlo Papp
2014-04-08 9:19 ` Pavel Machek
2014-04-08 9:19 ` Pavel Machek
2014-03-10 7:11 ` George Spelvin
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 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.