All of lore.kernel.org
 help / color / mirror / Atom feed
From: klightspeed@killerwolves.net (Ben Peddell)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv1 2/3] arm: mvebu: add .dts file for Synology DS213j
Date: Mon, 17 Nov 2014 09:23:13 +1000	[thread overview]
Message-ID: <546931E1.5070909@killerwolves.net> (raw)
In-Reply-To: <87d28nb83u.fsf@natisbad.org>

On 16/11/2014 11:14 PM, Arnaud Ebalard wrote:
>>> +	gpio-fan-100-32-35 {
>> There is some logic to this naming, but i don't remember what it is.
>> Did you check this is appropriate?
> The 32-35 is for the MPP i.e. fan uses mpp32 to mpp35 (3 MPP for speed,
> one for alarm). I don't know what the 100 part means. What I did is I
> googled to see which fan supported kirkwood models have and noticed
> that the DS212j uses the exact same fan, i.e. Y.S.Tech FD129225LL-N so
> I opted for the same naming.
>
> On that aspect, you may be able to explain why the speed-map for this
> fan goes up to 5000 RPM. This fan is supposed to operate at 1900RPM.
> Note that it does not matter in practice.

On the Kirkwood Diskstations, the first number (100, 120 or 150) refers
to the middle of the three fan speed resistors, the second number (15 or
32) refers to the first fan control pin, and the third number (18 or 35)
refers to the first fan alarm pin.

i.e. the 100-32-35 uses fan speed resistors of 33, 100 and 150 ohms,
uses pins 32-34 for fan control, and uses pin 35 for the fan alarm.

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.

-- 
Ben Peddell
 * ICT - School Support Officer
   - St Mary's School Bowen
   - St Catherine's School Proserpine
   - St John Bosco's School Collinsville
 * Mobile: 0431 800 195

*** Disclaimer ***
This email is only for the addressees, so please don't disclose it
to others without prior agreement.  The opinions expressed are mine,
and not my employer's.  My employer shall not be liable.  I am not
a lawyer.  I check my attachments for viruses, but I can't guarantee
my email will arrive unaltered.

#include "http://klightspeed.killerwolves.net/stddisclaimer.txt"

WARNING: multiple messages have this Message-ID (diff)
From: Ben Peddell <klightspeed-aslSrjg9ejhWX4hkXwHRhw@public.gmane.org>
To: Arnaud Ebalard <arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org>,
	Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
Cc: 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: Mon, 17 Nov 2014 09:23:13 +1000	[thread overview]
Message-ID: <546931E1.5070909@killerwolves.net> (raw)
In-Reply-To: <87d28nb83u.fsf-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org>

On 16/11/2014 11:14 PM, Arnaud Ebalard wrote:
>>> +	gpio-fan-100-32-35 {
>> There is some logic to this naming, but i don't remember what it is.
>> Did you check this is appropriate?
> The 32-35 is for the MPP i.e. fan uses mpp32 to mpp35 (3 MPP for speed,
> one for alarm). I don't know what the 100 part means. What I did is I
> googled to see which fan supported kirkwood models have and noticed
> that the DS212j uses the exact same fan, i.e. Y.S.Tech FD129225LL-N so
> I opted for the same naming.
>
> On that aspect, you may be able to explain why the speed-map for this
> fan goes up to 5000 RPM. This fan is supposed to operate at 1900RPM.
> Note that it does not matter in practice.

On the Kirkwood Diskstations, the first number (100, 120 or 150) refers
to the middle of the three fan speed resistors, the second number (15 or
32) refers to the first fan control pin, and the third number (18 or 35)
refers to the first fan alarm pin.

i.e. the 100-32-35 uses fan speed resistors of 33, 100 and 150 ohms,
uses pins 32-34 for fan control, and uses pin 35 for the fan alarm.

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.

-- 
Ben Peddell
 * ICT - School Support Officer
   - St Mary's School Bowen
   - St Catherine's School Proserpine
   - St John Bosco's School Collinsville
 * Mobile: 0431 800 195

*** Disclaimer ***
This email is only for the addressees, so please don't disclose it
to others without prior agreement.  The opinions expressed are mine,
and not my employer's.  My employer shall not be liable.  I am not
a lawyer.  I check my attachments for viruses, but I can't guarantee
my email will arrive unaltered.

#include "http://klightspeed.killerwolves.net/stddisclaimer.txt"

--
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-16 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 [this message]
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
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=546931E1.5070909@killerwolves.net \
    --to=klightspeed@killerwolves.net \
    --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.