From: arno@natisbad.org (Arnaud Ebalard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: mvebu: Add Netgear ReadyNAS 2120 board
Date: Sat, 09 Nov 2013 22:26:12 +0100 [thread overview]
Message-ID: <871u2pmfm3.fsf@natisbad.org> (raw)
In-Reply-To: <20131109210836.GK14890@lunn.ch> (Andrew Lunn's message of "Sat, 9 Nov 2013 22:08:36 +0100")
Hi,
Andrew Lunn <andrew@lunn.ch> writes:
> On Sat, Nov 09, 2013 at 09:27:32PM +0100, Arnaud Ebalard wrote:
>>
>> All hardware parts of the (mv78230 Armada XP based) NETGEAR ReadyNAS
>> 2120 are supported by mainline kernel (USB 3.0 and eSATA rear ports,
>> USB 2.0 front port, Gigabit controller and PHYs for the two rear ports,
>> serial port, LEDs, Buttons, 88SE9170 SATA controllers, three G762 fan
>> controllers, G751 temperature sensor) except for:
>
>> + i2c at 11000 {
>> + compatible = "marvell,mv64xxx-i2c";
>> + clock-frequency = <400000>;
>> + status = "okay";
>> +
>> + /* Rear fan #1 of 3 (Protechnic MGT4012XB-O20,
>> + * 8000RPM) near eSATA port */
>> + g762_fan1: g762 at 3e {
>> + compatible = "gmt,g762";
>> + reg = <0x3e>;
>> + clocks = <&g762_clk>; /* input clock */
>> + fan_gear_mode = <0>;
>> + fan_startv = <1>;
>> + pwm_polarity = <0>;
>> + };
>> +
>> + /* Rear fan #2 of 3 at the center */
>> + g762_fan2: g762 at 48 {
>> + compatible = "gmt,g762";
>> + reg = <0x48>;
>> + clocks = <&g762_clk>; /* input clock */
>> + fan_gear_mode = <0>;
>> + fan_startv = <1>;
>> + pwm_polarity = <0>;
>> + };
>> +
>> + /* Rear fan #3 of 3 */
>> + g762_fan3: g762 at 49 {
>> + compatible = "gmt,g762";
>> + reg = <0x49>;
>> + clocks = <&g762_clk>; /* input clock */
>> + fan_gear_mode = <0>;
>> + fan_startv = <1>;
>> + pwm_polarity = <0>;
>> + };
>> +
>> + g751: g751 at 4c {
>> + compatible = "gmt,g751";
>> + reg = <0x4c>;
>> + };
>
> Hi Arnaud
>
> Looks good to me. However, if for some reason you need to respin, it
> would be nice to add a comment about what the g751 is. All the other
> i2c devices you say what they are. No need to respin just because of
> this.
I'll keep that in mind. Thanks.
GMT G751 is an I2C temperature sensor and thermal watchdog chip. It is
basically a clone of National Semiconductor LM75 chip:
root at thin:/sys# sensors
g762-i2c-0-3e
Adapter: mv64xxx_i2c adapter
fan1: 5461 RPM (div = 1)
g762-i2c-0-48
Adapter: mv64xxx_i2c adapter
fan1: 5461 RPM (div = 1)
g762-i2c-0-49
Adapter: mv64xxx_i2c adapter
fan1: 5461 RPM (div = 1)
g751-i2c-0-4c
Adapter: mv64xxx_i2c adapter
temp1: +30.5?C (high = +80.0?C, hyst = +75.0?C)
armada_thermal-virtual-0
Adapter: Virtual device
temp1: +34.2?C
Cheers,
a+
WARNING: multiple messages have this Message-ID (diff)
From: arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org (Arnaud Ebalard)
To: 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>,
Ezequiel Garcia
<ezequiel.garcia-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Thomas Petazzoni
<thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@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>,
Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH] ARM: mvebu: Add Netgear ReadyNAS 2120 board
Date: Sat, 09 Nov 2013 22:26:12 +0100 [thread overview]
Message-ID: <871u2pmfm3.fsf@natisbad.org> (raw)
In-Reply-To: <20131109210836.GK14890-g2DYL2Zd6BY@public.gmane.org> (Andrew Lunn's message of "Sat, 9 Nov 2013 22:08:36 +0100")
Hi,
Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> writes:
> On Sat, Nov 09, 2013 at 09:27:32PM +0100, Arnaud Ebalard wrote:
>>
>> All hardware parts of the (mv78230 Armada XP based) NETGEAR ReadyNAS
>> 2120 are supported by mainline kernel (USB 3.0 and eSATA rear ports,
>> USB 2.0 front port, Gigabit controller and PHYs for the two rear ports,
>> serial port, LEDs, Buttons, 88SE9170 SATA controllers, three G762 fan
>> controllers, G751 temperature sensor) except for:
>
>> + i2c@11000 {
>> + compatible = "marvell,mv64xxx-i2c";
>> + clock-frequency = <400000>;
>> + status = "okay";
>> +
>> + /* Rear fan #1 of 3 (Protechnic MGT4012XB-O20,
>> + * 8000RPM) near eSATA port */
>> + g762_fan1: g762@3e {
>> + compatible = "gmt,g762";
>> + reg = <0x3e>;
>> + clocks = <&g762_clk>; /* input clock */
>> + fan_gear_mode = <0>;
>> + fan_startv = <1>;
>> + pwm_polarity = <0>;
>> + };
>> +
>> + /* Rear fan #2 of 3 at the center */
>> + g762_fan2: g762@48 {
>> + compatible = "gmt,g762";
>> + reg = <0x48>;
>> + clocks = <&g762_clk>; /* input clock */
>> + fan_gear_mode = <0>;
>> + fan_startv = <1>;
>> + pwm_polarity = <0>;
>> + };
>> +
>> + /* Rear fan #3 of 3 */
>> + g762_fan3: g762@49 {
>> + compatible = "gmt,g762";
>> + reg = <0x49>;
>> + clocks = <&g762_clk>; /* input clock */
>> + fan_gear_mode = <0>;
>> + fan_startv = <1>;
>> + pwm_polarity = <0>;
>> + };
>> +
>> + g751: g751@4c {
>> + compatible = "gmt,g751";
>> + reg = <0x4c>;
>> + };
>
> Hi Arnaud
>
> Looks good to me. However, if for some reason you need to respin, it
> would be nice to add a comment about what the g751 is. All the other
> i2c devices you say what they are. No need to respin just because of
> this.
I'll keep that in mind. Thanks.
GMT G751 is an I2C temperature sensor and thermal watchdog chip. It is
basically a clone of National Semiconductor LM75 chip:
root@thin:/sys# sensors
g762-i2c-0-3e
Adapter: mv64xxx_i2c adapter
fan1: 5461 RPM (div = 1)
g762-i2c-0-48
Adapter: mv64xxx_i2c adapter
fan1: 5461 RPM (div = 1)
g762-i2c-0-49
Adapter: mv64xxx_i2c adapter
fan1: 5461 RPM (div = 1)
g751-i2c-0-4c
Adapter: mv64xxx_i2c adapter
temp1: +30.5°C (high = +80.0°C, hyst = +75.0°C)
armada_thermal-virtual-0
Adapter: Virtual device
temp1: +34.2°C
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
next prev parent reply other threads:[~2013-11-09 21:26 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-09 20:27 [PATCH] ARM: mvebu: Add Netgear ReadyNAS 2120 board Arnaud Ebalard
2013-11-09 20:27 ` Arnaud Ebalard
2013-11-09 21:08 ` Andrew Lunn
2013-11-09 21:08 ` Andrew Lunn
2013-11-09 21:26 ` Arnaud Ebalard [this message]
2013-11-09 21:26 ` Arnaud Ebalard
2013-11-09 23:57 ` Sebastian Hesselbarth
2013-11-09 23:57 ` Sebastian Hesselbarth
2013-11-10 0:42 ` Arnaud Ebalard
2013-11-10 0:42 ` Arnaud Ebalard
2013-11-10 0:57 ` Sebastian Hesselbarth
2013-11-10 0:57 ` Sebastian Hesselbarth
2013-11-11 0:32 ` Arnaud Ebalard
2013-11-11 0:32 ` Arnaud Ebalard
2013-11-11 1:09 ` Sebastian Hesselbarth
2013-11-11 1:09 ` Sebastian Hesselbarth
2013-11-10 8:15 ` Thomas Petazzoni
2013-11-10 8:15 ` Thomas Petazzoni
2013-11-11 14:47 ` Jason Cooper
2013-11-11 14:47 ` Jason Cooper
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=871u2pmfm3.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.