[RFC/PATCH] dns323: Support for HW rev C1

Benjamin Herrenschmidt benh at kernel.crashing.org
Wed Apr 21 17:11:34 EDT 2010


On Wed, 2010-04-21 at 10:32 -0400, Nicolas Pitre wrote:
> > Oh a couple more things...
> > 
> > The fans are controlled via 2 GPIOs on this (see the explanation in
> the
> > patch). Do you guys think I should create some kind of pseudo hwmon
> > device so it can be controlled with one file where you write "off",
> > "low", "high", or should I just let userspace cope.
> 
> If you fancy doing such thing then that would be more user friendly.  
> And that could provide a common abstraction for all models. 

Well, other models have an i2c fan controller that is supported by
hwmon, so I'm not sure it's necessarily -that- a good idea to change
that (but then I've never been a fan of hwmon either :-)

I'm half tempted to actually stick a thermal control kernel thread in
there and leave userspace out of the game too, like I do on most macs. 

I'll think about it. No time right now anyways.

Cheers,
Ben.





More information about the linux-arm-kernel mailing list