On Fri, Feb 15, 2013 at 10:41:22AM -0800, Doug Anderson wrote: > Stephen Warren pointed out as part of a code review of another mux > driver that there should be no need to have > "platform_set_drvdata(pdev, NULL)" in the remove function. Get rid of > it in the i2c-mux-gpio driver. > > See Stephen's comment at: > > http://www.gossamer-threads.com/lists/linux/kernel/1678627?do=post_view_threaded#1678627 > > Signed-off-by: Doug Anderson <diand...@chromium.org>
I think we should do this at least I2C subsystem wide (51 occasions of setting some *_drvdata to NULL), even better kernel-wide. Patching individual drivers won't cut the confusion around this issue, I am afraid. Thanks, Wolfram -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
