Re: [PATCH] Driver Core patches for 2.6.7

[Dmitry Torokhov <dtor_core@ameritech.net>]

On Tuesday 22 June 2004 06:31 pm, Greg KH wrote:
> On Tue, Jun 22, 2004 at 06:21:01PM -0500, Dmitry Torokhov wrote:
> > On Tuesday 22 June 2004 12:41 pm, Greg KH wrote:
> > 
> > >  
> > >  void class_unregister(struct class * cls)
> > >  {
> > >  	pr_debug("device class '%s': unregistering\n",cls->name);
> > > +	remove_class_attrs(cls);
> > >  	subsystem_unregister(&cls->subsys);
> > >  }
> > >  
> > 
> > Question: is it necessary to call remove_class_attrs? I thought that sysfs
> > automatically destroys all children when parent is destroyed? Am I imagining
> > things?
> 
> No, you aren't imagining things.  But it's considered "good form" to
> remove them if you can, as this will probably change in 2.7, and it will
> make my life easier trying to audit the whole tree at that time...
> 

Is there any specific reason for such a change? I kinda like the idea of
registering attributes and then having driver code do the bean counting
for me.

Btw, is there a chance for platform_device_register_simple that I sent to
you earlier be included?

-- 
Dmitry