From: arno@natisbad.org (Arnaud Ebalard)
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
Guenter Roeck <linux@roeck-us.net>, Andrew Lunn <andrew@lunn.ch>,
Russell King - ARM Linux <linux@arm.linux.org.uk>,
Jason Cooper <jason@lakedaemon.net>,
linux-doc@vger.kernel.org, devicetree-discuss@lists.ozlabs.org,
Olivier Mouchet <olivier.mouchet@gmail.com>,
Rob Herring <rob.herring@calxeda.com>,
lm-sensors@lm-sensors.org,
Grant Likely <grant.likely@secretlab.ca>,
Rob Landley <rob@landley.net>, Jean Delvare <khali@linux-fr.org>,
Simon Guinot <simon.guinot@sequanux.org>
Subject: Re: [lm-sensors] [PATCHv2 0/3] Add G762/G763 PWM fan controller
Date: Tue, 28 May 2013 10:15:04 +0000 [thread overview]
Message-ID: <87li6z2yxj.fsf@natisbad.org> (raw)
In-Reply-To: 152190814.eApXVbVBPA@wuerfel
Hi Arnd,
Arnd Bergmann <arnd@arndb.de> writes:
> On Tuesday 28 May 2013 00:02:29 Arnaud Ebalard wrote:
>>
>> This series adds support for GMT G762/G763. This work is based on a
>> basic version for 2.6.31 kernel developed Olivier Mouchet for LaCie
>> NAS. Updates have been performed to run on recent kernels. Support has
>> been completed and additional features added: ability to configure
>> various characteristics from .dts file, better initialization, alarms
>> and error reporting support, gear mode, polarity, fan pulse per
>> revolution, fan startup voltage control.
>
> I wonder if this could be split into two separate drivers, one for
> the pwm subsystem, and one for a hardware-independent fan controller
> based on the pwm interfaces.
To be honest, I wouldn't even know how to start in order to do that.
Additionally, it would be worth the effort if other drivers could be
easily refactored using this idea. I'll let hwmon people provide some
feedback and try and comply with their directions (based on the cpu
cycle I can spend on this).
Cheers,
a+
_______________________________________________
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: arno@natisbad.org (Arnaud Ebalard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv2 0/3] Add G762/G763 PWM fan controller
Date: Tue, 28 May 2013 12:15:04 +0200 [thread overview]
Message-ID: <87li6z2yxj.fsf@natisbad.org> (raw)
In-Reply-To: 152190814.eApXVbVBPA@wuerfel
Hi Arnd,
Arnd Bergmann <arnd@arndb.de> writes:
> On Tuesday 28 May 2013 00:02:29 Arnaud Ebalard wrote:
>>
>> This series adds support for GMT G762/G763. This work is based on a
>> basic version for 2.6.31 kernel developed Olivier Mouchet for LaCie
>> NAS. Updates have been performed to run on recent kernels. Support has
>> been completed and additional features added: ability to configure
>> various characteristics from .dts file, better initialization, alarms
>> and error reporting support, gear mode, polarity, fan pulse per
>> revolution, fan startup voltage control.
>
> I wonder if this could be split into two separate drivers, one for
> the pwm subsystem, and one for a hardware-independent fan controller
> based on the pwm interfaces.
To be honest, I wouldn't even know how to start in order to do that.
Additionally, it would be worth the effort if other drivers could be
easily refactored using this idea. I'll let hwmon people provide some
feedback and try and comply with their directions (based on the cpu
cycle I can spend on this).
Cheers,
a+
WARNING: multiple messages have this Message-ID (diff)
From: arno@natisbad.org (Arnaud Ebalard)
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
Guenter Roeck <linux@roeck-us.net>, Andrew Lunn <andrew@lunn.ch>,
Russell King - ARM Linux <linux@arm.linux.org.uk>,
Jason Cooper <jason@lakedaemon.net>,
linux-doc@vger.kernel.org, devicetree-discuss@lists.ozlabs.org,
Olivier Mouchet <olivier.mouchet@gmail.com>,
Rob Herring <rob.herring@calxeda.com>,
lm-sensors@lm-sensors.org,
Grant Likely <grant.likely@secretlab.ca>,
Rob Landley <rob@landley.net>, Jean Delvare <khali@linux-fr.org>,
Simon Guinot <simon.guinot@sequanux.org>
Subject: Re: [PATCHv2 0/3] Add G762/G763 PWM fan controller
Date: Tue, 28 May 2013 12:15:04 +0200 [thread overview]
Message-ID: <87li6z2yxj.fsf@natisbad.org> (raw)
In-Reply-To: 152190814.eApXVbVBPA@wuerfel
Hi Arnd,
Arnd Bergmann <arnd@arndb.de> writes:
> On Tuesday 28 May 2013 00:02:29 Arnaud Ebalard wrote:
>>
>> This series adds support for GMT G762/G763. This work is based on a
>> basic version for 2.6.31 kernel developed Olivier Mouchet for LaCie
>> NAS. Updates have been performed to run on recent kernels. Support has
>> been completed and additional features added: ability to configure
>> various characteristics from .dts file, better initialization, alarms
>> and error reporting support, gear mode, polarity, fan pulse per
>> revolution, fan startup voltage control.
>
> I wonder if this could be split into two separate drivers, one for
> the pwm subsystem, and one for a hardware-independent fan controller
> based on the pwm interfaces.
To be honest, I wouldn't even know how to start in order to do that.
Additionally, it would be worth the effort if other drivers could be
easily refactored using this idea. I'll let hwmon people provide some
feedback and try and comply with their directions (based on the cpu
cycle I can spend on this).
Cheers,
a+
next prev parent reply other threads:[~2013-05-28 10:15 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-27 22:02 [lm-sensors] [PATCHv2 0/3] Add G762/G763 PWM fan controller Arnaud Ebalard
2013-05-27 22:02 ` Arnaud Ebalard
2013-05-27 22:02 ` Arnaud Ebalard
2013-05-27 22:03 ` [lm-sensors] [PATCHv2 1/3] Add support for GMT " Arnaud Ebalard
2013-05-27 22:03 ` Arnaud Ebalard
2013-05-27 22:03 ` Arnaud Ebalard
2013-05-31 22:16 ` [lm-sensors] " Simon Guinot
2013-05-31 22:16 ` Simon Guinot
2013-05-31 22:16 ` Simon Guinot
2013-06-01 17:26 ` [lm-sensors] " Arnaud Ebalard
2013-06-01 17:26 ` Arnaud Ebalard
2013-06-01 17:26 ` Arnaud Ebalard
2013-06-02 15:45 ` [lm-sensors] " Arnaud Ebalard
2013-06-02 15:45 ` Arnaud Ebalard
2013-06-02 15:45 ` Arnaud Ebalard
2013-06-02 20:35 ` [lm-sensors] " Guenter Roeck
2013-06-02 20:35 ` Guenter Roeck
2013-06-02 20:35 ` Guenter Roeck
2013-06-02 21:36 ` [lm-sensors] " Arnaud Ebalard
2013-06-02 21:36 ` Arnaud Ebalard
2013-06-02 21:36 ` Arnaud Ebalard
2013-06-02 21:59 ` [lm-sensors] " Simon Guinot
2013-06-02 21:59 ` Simon Guinot
2013-06-02 21:59 ` Simon Guinot
2013-06-04 6:52 ` [lm-sensors] " Arnaud Ebalard
2013-06-04 6:52 ` Arnaud Ebalard
2013-06-04 6:52 ` Arnaud Ebalard
2013-06-04 21:23 ` [lm-sensors] " Simon Guinot
2013-06-04 21:23 ` Simon Guinot
2013-06-04 21:23 ` Simon Guinot
2013-06-11 15:15 ` [lm-sensors] " Guenter Roeck
2013-06-11 15:15 ` Guenter Roeck
2013-06-11 15:15 ` Guenter Roeck
2013-06-15 16:13 ` [lm-sensors] " Arnaud Ebalard
2013-06-15 16:13 ` Arnaud Ebalard
2013-06-15 16:13 ` Arnaud Ebalard
2013-06-04 6:51 ` [lm-sensors] " Arnaud Ebalard
2013-06-04 6:51 ` Arnaud Ebalard
2013-06-04 6:51 ` Arnaud Ebalard
2013-06-01 14:33 ` [lm-sensors] " Guenter Roeck
2013-06-01 14:33 ` Guenter Roeck
2013-06-01 14:33 ` Guenter Roeck
2013-06-02 15:39 ` [lm-sensors] " Arnaud Ebalard
2013-06-02 15:39 ` Arnaud Ebalard
2013-06-02 15:39 ` Arnaud Ebalard
2013-05-27 22:03 ` [lm-sensors] [PATCHv2 2/3] Add documentation for g762 driver Arnaud Ebalard
2013-05-27 22:03 ` Arnaud Ebalard
2013-05-27 22:03 ` Arnaud Ebalard
2013-05-27 22:03 ` [lm-sensors] [PATCHv2 3/3] Add DT bindings " Arnaud Ebalard
2013-05-27 22:03 ` Arnaud Ebalard
2013-05-27 22:03 ` Arnaud Ebalard
2013-05-27 22:15 ` [lm-sensors] [PATCHv2 0/3] Add G762/G763 PWM fan controller Arnd Bergmann
2013-05-27 22:15 ` Arnd Bergmann
2013-05-27 22:15 ` Arnd Bergmann
2013-05-28 10:15 ` Arnaud Ebalard [this message]
2013-05-28 10:15 ` Arnaud Ebalard
2013-05-28 10:15 ` Arnaud Ebalard
2013-05-28 11:19 ` [lm-sensors] " Thierry Reding
2013-05-28 11:19 ` Thierry Reding
2013-05-28 11:19 ` Thierry Reding
2013-05-28 12:29 ` [lm-sensors] " Guenter Roeck
2013-05-28 12:29 ` Guenter Roeck
2013-05-28 12:29 ` Guenter Roeck
2013-05-28 13:47 ` [lm-sensors] " Thierry Reding
2013-05-28 13:47 ` Thierry Reding
2013-05-28 13:47 ` Thierry Reding
2013-05-28 15:42 ` [lm-sensors] " Guenter Roeck
2013-05-28 15:42 ` Guenter Roeck
2013-05-28 15:42 ` Guenter Roeck
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=87li6z2yxj.fsf@natisbad.org \
--to=arno@natisbad.org \
--cc=andrew@lunn.ch \
--cc=arnd@arndb.de \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=grant.likely@secretlab.ca \
--cc=jason@lakedaemon.net \
--cc=khali@linux-fr.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=linux@roeck-us.net \
--cc=lm-sensors@lm-sensors.org \
--cc=olivier.mouchet@gmail.com \
--cc=rob.herring@calxeda.com \
--cc=rob@landley.net \
--cc=simon.guinot@sequanux.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.