Re: [PATCHv2 0/3] Add G762/G763 PWM fan controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 28, 2013 at 01:19:23PM +0200, Thierry Reding wrote:
> On Tue, May 28, 2013 at 12:15:04PM +0200, Arnaud Ebalard wrote:
> > Hi Arnd,
> > 
> > Arnd Bergmann <arnd@xxxxxxxx> 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).
> 
> What Arnd proposes doesn't sounds like a very good idea. PWM-controlled
> fans should be able to work just fine using a generic driver that uses
> the PWM framework, similar to the PWM backlight and LED drivers.
> 
Isn't that exactly what Arnd proposed ?

Guenter
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux