Re: [PATCH] gpiolib: Allow user-selection

[Greg KH <greg@kroah.com>]

On Wed, Jul 02, 2008 at 10:41:27PM -0700, David Brownell wrote:
> The new "-next" process is still working out.  Backwards-incompatible
> changes will *always* make problems.  In this case there seem to
> have been three such changes:
> 
> 	- device_create() vanishing, even for users which did the
> 	  locking correctly ... this was at least something of a
> 	  graceful migration, although it would have been better
> 	  if it were deprecated first.  Change can work against
> 	  the current (2.6.26) code base.

device_create() did disappear in -next, but when it goes to Linus it
will not, it will come back as a #define and everything will be
converted back.  That was to catch the build errors when I audited the
whole tree.  The -rc1 merge cycle will handle this properly.

> 	- class.devices vanishing ... what I really needed was
> 	  more of a "is this class initialized yet" call, so
> 	  testing for class->devices.next non-null can be replaced
> 	  by a test for class->p non-null.  (Or maybe this should
> 	  be viewed as needing a "real" driver model call, so that
> 	  code which must run before driver model init can more
> 	  easily cooperate with driver model stuff that has to
> 	  run much later, after the guts are initialized.)

If you need such a function, please let me know and I can provide it, I
was not aware of anyone using class.devices, it is an
internal-to-the-driver-core field, and has moved private.

> 	- Extra parameter to class_find_device().  This could
> 	  have been done in a backwards-compatible manner, like
> 	  device_create() was, but ... it wasn't.

This was simpler as there were less users of this function.  And the
whole tree was fixed up.  If there are trees on top of -next that I
can't see, that's pretty hard for me to fix :)

thanks,

greg k-h