Hi Dmitry, one quick question before I repost the updated patch: On 14.11.2011 21:28, Dmitry Torokhov wrote: >> +err0: >> + dev_set_drvdata(&client->dev, NULL); > > Not needed. Do you really think leaving a pointer to a non-existing structure is a good idea? Regards, Ilya. -- 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