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. Thierry
Attachment:
pgpciknf3iqhQ.pgp
Description: PGP signature