On Fri, Jun 28, 2013 at 01:15:35PM -0600, Daniel Drake wrote:
> On Fri, Jun 28, 2013 at 12:20 PM, Dmitry Torokhov
> <dmitry.torokhov@xxxxxxxxx> wrote:
> >> +static void olpc_apsp_close(struct serio *port)
> >> +{
> >> + struct olpc_apsp *priv = port->port_data;
> >> + unsigned int tmp;
> >> +
> >> + if (--priv->open_count == 0) {
> >
> > Both need locking. It looks like you need the song and dance similar to
> > what i8042 has to do with it's multiplexed ports. Or ps2mult.c...
>
> I thought they would both be locked already, due to this comment in serio.c:
>
> /* called from serio_driver->connect/disconnect methods under serio_mutex */
>
> Or am I missing something there?
Hmm, you are right... I'll add a comment there to remind myself.
--
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
