On Fri, Jun 28, 2013 at 12:20 PM, Dmitry Torokhov <[email protected]> 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? Thanks Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
