On Sun, Nov 20, 2011 at 10:09:24PM +0400, Ilya Yanok wrote: > 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? I2C core takes care of clearing it after remove() or failed probe(). Thanks. -- 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