lkml.org 
[lkml]   [2010]   [Nov]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRe: [PATCHv1] staging: Synaptics RMI4 touchpad driver support
    On Tue, Nov 02, 2010 at 02:06:11PM +0000, Alan Cox wrote:
    > On Tue, 2 Nov 2010 07:03:37 -0700
    > Greg KH <greg@kroah.com> wrote:
    >
    > > On Tue, Nov 02, 2010 at 05:38:45PM +0530, Naveen Kumar G wrote:
    > > > --- /dev/null
    > > > +++ b/drivers/staging/ste_rmi4/TODO
    > > > @@ -0,0 +1,7 @@
    > > > +TODO
    > > > +----
    > > > +
    > > > +Wait for the official upstream synaptics rmi4 clearpad drivers as promised over the past few months
    > > > +Merge any device support needed from this driver into it
    > > > +Delete this driver
    > > Huh?
    > >
    > > Why not just add this driver to the kernel tree instead? When the
    > > "promised" driver then eventually shows up (who is promising it?) then
    > > delete the thing.
    >
    > I've suggested that in the past.

    Then we should do that.

    > > But until then, why is this needed to go into the staging tree? Is
    > > there any other main reason it should be here and not in the "real" part
    > > of the kernel?
    >
    > It's a driver nobody wants to commit to keeping around when there is
    > eventually a general purpose driver. In other words things like module
    > names will change.

    module names change is not a big deal, right?

    So, this driver should go into the proper subsystem then, and not
    staging. Staging is for code that is either working on getting merged
    to the main kernel, or it is on the way out. Not to just sit there and
    wait for some other thing to happen (well, for the most part, there are
    some exceptions of course...)

    thanks,

    greg k-h



    \
     
     \ /
      Last update: 2010-11-02 15:23    [W:2.433 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site