All of lore.kernel.org
 help / color / mirror / Atom feed
From: arno@natisbad.org (Arnaud Ebalard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv1 2/3] arm: mvebu: add .dts file for Synology DS213j
Date: Tue, 18 Nov 2014 00:23:53 +0100	[thread overview]
Message-ID: <871tp1pg1i.fsf@natisbad.org> (raw)
In-Reply-To: <546A7997.1040603@killerwolves.net> (Ben Peddell's message of "Tue, 18 Nov 2014 08:41:27 +1000")

Hi Ben,

Ben Peddell <klightspeed@killerwolves.net> writes:

>>> The 5000RPM was interpolated from my measured RPM pulse rate (4200PPM)
>>> of the 80mm fan in my DS211j (which uses the 150-32-35 fan config) at
>>> maximum RPM.
>> 
>> Then, I think I need to change the speed map to have 1900 RPM as max
>> speed. Out of curiosity, how did you measured the RPM pulse rate
>> exactly?
>
> I hooked a logic analyzer to the sense pin and measured the pulse rate.
>
> Looking at the specifications of the fan in this unit (an EC7025L12ER),
> it has a maximum fan speed of 3000RPM, so it looks like it uses two
> pulses per revolution and not one as I had assumed when I created the
> speed map.

Thanks for this feedback.

Cheers,

a+

WARNING: multiple messages have this Message-ID (diff)
From: arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org (Arnaud Ebalard)
To: Ben Peddell <klightspeed-aslSrjg9ejhWX4hkXwHRhw@public.gmane.org>
Cc: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>,
	Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>,
	Gregory Clement
	<gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	Sebastian Hesselbarth
	<sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
	Ian Campbell
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCHv1 2/3] arm: mvebu: add .dts file for Synology DS213j
Date: Tue, 18 Nov 2014 00:23:53 +0100	[thread overview]
Message-ID: <871tp1pg1i.fsf@natisbad.org> (raw)
In-Reply-To: <546A7997.1040603-aslSrjg9ejhWX4hkXwHRhw@public.gmane.org> (Ben Peddell's message of "Tue, 18 Nov 2014 08:41:27 +1000")

Hi Ben,

Ben Peddell <klightspeed-aslSrjg9ejhWX4hkXwHRhw@public.gmane.org> writes:

>>> The 5000RPM was interpolated from my measured RPM pulse rate (4200PPM)
>>> of the 80mm fan in my DS211j (which uses the 150-32-35 fan config) at
>>> maximum RPM.
>> 
>> Then, I think I need to change the speed map to have 1900 RPM as max
>> speed. Out of curiosity, how did you measured the RPM pulse rate
>> exactly?
>
> I hooked a logic analyzer to the sense pin and measured the pulse rate.
>
> Looking at the specifications of the fan in this unit (an EC7025L12ER),
> it has a maximum fan speed of 3000RPM, so it looks like it uses two
> pulses per revolution and not one as I had assumed when I created the
> speed map.

Thanks for this feedback.

Cheers,

a+
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2014-11-17 23:23 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-16  0:04 [PATCHv1 0/3] arm: mvebu: add Synology DS213j and DS414 .dts files Arnaud Ebalard
2014-11-16  0:04 ` Arnaud Ebalard
2014-11-16  0:04 ` [PATCHv1 1/3] arm: mvebu: add uartX labels for Armada SoC serial nodes Arnaud Ebalard
2014-11-16  0:04   ` Arnaud Ebalard
2014-11-16  1:09   ` Andrew Lunn
2014-11-16  1:09     ` Andrew Lunn
2014-11-16 12:34     ` Arnaud Ebalard
2014-11-16 12:34       ` Arnaud Ebalard
2014-11-16  0:04 ` [PATCHv1 2/3] arm: mvebu: add .dts file for Synology DS213j Arnaud Ebalard
2014-11-16  0:04   ` Arnaud Ebalard
2014-11-16  2:00   ` Andrew Lunn
2014-11-16  2:00     ` Andrew Lunn
2014-11-16  2:31     ` Andrew Lunn
2014-11-16  2:31       ` Andrew Lunn
2014-11-16 13:14     ` Arnaud Ebalard
2014-11-16 13:14       ` Arnaud Ebalard
2014-11-16 23:23       ` Ben Peddell
2014-11-16 23:23         ` Ben Peddell
2014-11-17  8:25         ` Arnaud Ebalard
2014-11-17  8:25           ` Arnaud Ebalard
2014-11-17 22:41           ` Ben Peddell
2014-11-17 22:41             ` Ben Peddell
2014-11-17 23:23             ` Arnaud Ebalard [this message]
2014-11-17 23:23               ` Arnaud Ebalard
2014-11-16  0:04 ` [PATCHv1 3/3] arm: mvebu: add .dts file for Synology DS414 Arnaud Ebalard
2014-11-16  0:04   ` Arnaud Ebalard
2014-11-16  2:15   ` Andrew Lunn
2014-11-16  2:15     ` Andrew Lunn
2014-11-16 13:17     ` Arnaud Ebalard
2014-11-16 13:17       ` Arnaud Ebalard
2014-11-16  2:20 ` [PATCHv1 0/3] arm: mvebu: add Synology DS213j and DS414 .dts files Andrew Lunn
2014-11-16  2:20   ` Andrew Lunn
2014-11-16 13:24   ` Arnaud Ebalard
2014-11-16 13:24     ` Arnaud Ebalard

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=871tp1pg1i.fsf@natisbad.org \
    --to=arno@natisbad.org \
    --cc=linux-arm-kernel@lists.infradead.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.