[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