Re: [PATCH 0/19] New set of input patches

From: Vojtech Pavlik
Date: Mon Jun 28 2004 - 02:53:07 EST


On Mon, Jun 28, 2004 at 02:13:58AM -0500, Dmitry Torokhov wrote:

> > IMO drivers have no bussiness messing with the serio locks. We could use
> > 'plug' and 'unplug' functions like the network driver use, or handle it
> > inside the driver, but taking the lock is the wrong thing to do.
>
> OK, I just don't want to introduce another lock just for that...

I think a bit in flags "PSMOUSE_ENABLED", like we have the
"ATKBD_ENABLED" bit might be just fine - handle the interrupt, but throw
away the data during the protocol switch. We aren't interested in the
data anyway.

> > > 15-synaptics-passthrough-handling.patch
> > > - If data looks like a pass-through packet and tuchpad has
> > > pass-through capability do not pass it to the main handler
> > > if child port is disconnected.
> >
> > I'll have to look closer on this one - I think we want to pass the data
> > to the serio layer even if there is no driver listening on the
> > passthrough serio.
>
> We probably should issue serio_interrupt on child port to force rescan but
> that packet has no business in parent's motion handling routine and that's
> what this patch tries to fix. Anyway, I will look at it more later.

Indeed, we need it for the rescan. It shouldn't be that hard to fix at
once.

>
> > > (*) These patches have also been sent to Greg KH.
> >
> > Did he accept them already?
>
> No, not yet. He promised to take a look at platoform_device_register_simple by
> the end of the week but I guess kernel.bkbits.net troubles might intervene...
> And other 2 I just send out today.

Ok. I'll wait then.

--
Vojtech Pavlik
SuSE Labs, SuSE CR
-
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/