Re: [PATCH] staging: new asus fan driver

From: Greg Kroah-Hartman
Date: Tue Nov 05 2013 - 21:36:18 EST


On Tue, Nov 05, 2013 at 01:54:51PM -0600, Felipe Contreras wrote:
> On Tue, Nov 5, 2013 at 9:16 AM, Greg Kroah-Hartman
> <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > On Tue, Nov 05, 2013 at 08:29:40AM -0600, Felipe Contreras wrote:
> >> On Tue, Nov 5, 2013 at 6:52 AM, Greg Kroah-Hartman
> >> <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> >> > On Tue, Nov 05, 2013 at 02:59:47AM -0600, Felipe Contreras wrote:
> >> >> Simple driver to enable control of the fan in ASUS laptops. So far this
> >> >> has only been tested in ASUS Zenbook Prime UX31A, but according to some
> >> >> online reference [1], it should work in other models as well.
> >> >>
> >> >> The implementation is very straight-forward, the only caveat is that the
> >> >> fan speed needs to be saved after it has been manually changed because
> >> >> it won't be reported properly until it goes back to 'auto' mode.
> >> >>
> >> >> [1] http://forum.notebookreview.com/asus/705656-fan-control-asus-prime-ux31-ux31a-ux32a-ux32vd.html
> >> >>
> >> >> Signed-off-by: Felipe Contreras <felipe.contreras@xxxxxxxxx>
> >> >> ---
> >> >>
> >> >> Two incarnations of this code exists [1][2], one that used ACPI methods
> >> >> directly, and one through WMI. Unfortunately the WMI version needs us to pass
> >> >> physicall addresses which is not exactly clean, and that's the only reason the
> >> >> code is proposed for staging.
> >> >>
> >> >> Most likely this cannot graduate until acpica gets support to receive virtual
> >> >> addresses.
> >> >
> >> > When is that going to happen?
> >>
> >> I don't know. Presumably when I get it done, which might be never, or
> >> a few days from now. Most likely it will take some time.
> >
> > Then why not just merge this to the proper place, instead of relying on
> > staging? I don't want to have to have a staging driver that is waiting
> > on external things to get it merged, instead of issues with the driver
> > itself, as you are now putting the burden of maintenance on me, for no
> > good reason other than you being too "lazy" to get other stuff done :)
>
> *I* am too lazy? I am doing all the work. Nobody else is doing
> anything about it.
>
> Linux is throttling the CPU speed of this machine without trying to to
> increase the fan speed first, that is not ideal.
>
> So I do the first step of writing the fan driver and I get told it
> shouldn't be a separate driver, it should be part of wmi, only to be
> told later by the same person that it doesn't belong in wmi when I put
> it there. Then I get told virt_to_physical() is not good, and it
> should move to staging, and then I'm told you don't want it.

Who told you to put it in staging? That is the person that needs to be
corrected here, it's not your issue, sorry, I didn't realize the
backstory here.

And I would push back on those people, if it solves your problem, and
there's no other good solution, then it should go into the "real" part
of the kernel. Who has rejected this driver and for what reason?

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/