Re: [Outreachy kernel] reordering remove actions in iio drivers

[Alison Schofield <amsfield22@gmail.com>]

On Sun, Feb 28, 2016 at 06:56:56PM +0100, Julia Lawall wrote:
> On Sun, 28 Feb 2016, Alison Schofield wrote:
> 
> > Hi Amitoj,
> > 
> > I just worked on a similar patch, and yours caught my attention.
> > It's not clear that it is OK to reorder the .remove actions.
> > Although it may seem counterintuitive to do anything *after*
> > you've unregistered your device, it is indeed done purposely
> > and safely in many iio drivers.  It usually needs to stay that way.
> > 
> > Check out more discussions of this on linux-iio.
> > 
> > And, consider the source...I'm just trying to figure this stuff
> > out too!
> 
> Which patch of Amitoj's are you referring to (or which file), and if 
> possible do you have a more specific link for the discussion on linux-iio?
> 
> thanks,
> julia

I'm seeing this threaded with original patch, but I guess you're not.
It is in reference to:
	[PATCH] staging: iio: accel: Replace iio_device_register with
	devm_iio_device_register

There is a lot of linux-iio discussion when this devm was originally
introduced with 30+ patches to migrate. Search devm_iio_device_register

and another later on when an intern tried to apply it:
http://article.gmane.org/gmane.linux.kernel.iio/13354/match=devm_iio_register

The accel case might be ok...just a heads up on the considerations.

alisons