Hi Grant, On Tue, 2012-12-04 at 14:56 +0000, Grant Likely wrote: > Instead of clearing the pointer only in the spi code, perhaps the > checks should be in really_probe() and __device_release_driver() so it > covers all bus types. Also, don't clear the pointer. Just use > dev_err() to report on the driver bug so that it gets fixed properly.
I agree with you. I just figured out that on 2012-05-23, Hans de Goede added a patch (0998d063) to enforce drvdata to be NULL after device_release or on probe failure. Good to know. Thanks, Vivien ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ spi-devel-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/spi-devel-general
