Re: [PATCH] deprecate pci_module_init

[Greg KH <greg@kroah.com>]

On Wed, Nov 03, 2004 at 02:13:39PM -0500, Jeff Garzik wrote:
> Chris Wedgwood wrote:
> >On Wed, Nov 03, 2004 at 12:46:40PM -0500, Jeff Garzik wrote:
> >
> >
> >>Wrong.  There are way too many __correct__ drivers to do this at
> >>present.
> >
> >
> >i could claim the same is true of MODULE_PARM yet we spew
> >warning-galore there...
> 
> There is a 2.4 version of module_param().
> 
> The semantics of pci_module_init() versus pci_register_driver() are 
> different across 2.4/2.6.  If you deprecate pci_module_init(), you are 
> breaking drivers which right now can be ported to 2.4 with a simple cp(1).

Yes, but any driver assuming that the pci_module_init() functionality
has not changed from 2.4 to 2.6 will be broken.  I've fixed up all of
the drivers that did this in the 2.6 tree.

> It's just downright silly to deprecate the API that is used most heavily 
> in drivers.

Due to the change in the way the function works, I'm slowly changing
drivers over to the new function.  It's just too dangerous over time to
leave it alone.

Chris, I agree with Christoph, convert more drivers over before marking
this function "obsolete".  In fact, just convert everyone and then
delete the #define all together, that's what I have been working toward.

thanks,

greg k-h