[PATCH v3] Add OLPC AP-SP input driver
Daniel Drake
dsd at laptop.org
Sat Jun 29 05:15:35 EST 2013
On Fri, Jun 28, 2013 at 12:20 PM, Dmitry Torokhov
<dmitry.torokhov at gmail.com> 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
More information about the devicetree-discuss
mailing list