Re: [RFC][PATCH 0/75] pci_module_init to pci_register_driver conversion

2006-08-17 Thread Greg KH
On Thu, Aug 17, 2006 at 08:35:12AM -0400, Jeff Garzik wrote:
> Greg KH wrote:
> >On Thu, Aug 17, 2006 at 04:26:35AM +, Michal Piotrowski wrote:
> >>Hi,
> >>
> >>pci_module_init is obsolete.
> >>
> >>This patch series converts pci_module_init to pci_register_driver.
> >>
> >>
> >>Can I remove this?
> >>
> >>include/linux/pci.h:385
> >>/*
> >> * pci_module_init is obsolete, this stays here till we fix up all usages 
> >> of it
> >> * in the tree.
> >> */
> >>#define pci_module_init pci_register_driver
> >
> >As repeated numerous times, it's up to the network developers if they
> >will take this or not.
> >
> >I'll hold off on taking this series, please push it through the driver
> >subsystem maintainers.
> 
> It's already in subsystem trees, in fact.

Great, it can wait until 2.6.19.

> But it is most definitely not 2.6.18-rc material :)

Agreed.

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC][PATCH 0/75] pci_module_init to pci_register_driver conversion

2006-08-17 Thread Jeff Garzik
Actually, I stand corrected:  netdev-2.6.git#upstream does not contain a 
pci_register_driver() patch.


Send the drivers/net piece along to me and I'll apply it.

Jeff



-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[RFC][PATCH 0/75] pci_module_init to pci_register_driver conversion

2006-08-17 Thread Michal Piotrowski

Hi,

pci_module_init is obsolete.

This patch series converts pci_module_init to pci_register_driver.

 ata/ata_generic.c |2 +-
 ata/pata_pdc2027x.c   |2 +-
 char/ipmi/ipmi_si_intf.c  |2 +-
 crypto/geode-aes.c|2 +-
 edac/k8_edac.c|2 +-
 net/3c59x.c   |2 +-
 net/8139cp.c  |2 +-
 net/8139too.c |2 +-
 net/acenic.c  |2 +-
 net/amd8111e.c|2 +-
 net/arcnet/com20020-pci.c |2 +-
 net/b44.c |2 +-
 net/bnx2.c|2 +-
 net/cassini.c |2 +-
 net/chelsio/cxgb2.c   |2 +-
 net/defxx.c   |2 +-
 net/dl2k.c|2 +-
 net/e100.c|2 +-
 net/e1000/e1000_main.c|2 +-
 net/eepro100.c|2 +-
 net/epic100.c |2 +-
 net/fealnx.c  |2 +-
 net/forcedeth.c   |2 +-
 net/hp100.c   |2 +-
 net/ixgb/ixgb_main.c  |2 +-
 net/natsemi.c |2 +-
 net/ne2k-pci.c|2 +-
 net/ns83820.c |2 +-
 net/pci-skeleton.c|2 +-
 net/pcnet32.c |2 +-
 net/r8169.c   |2 +-
 net/rrunner.c |2 +-
 net/s2io.c|2 +-
 net/saa9730.c |2 +-
 net/sis190.c  |2 +-
 net/sis900.c  |2 +-
 net/sk98lin/skge.c|2 +-
 net/skfp/skfddi.c |2 +-
 net/skge.c|2 +-
 net/starfire.c|2 +-
 net/sundance.c|2 +-
 net/sungem.c  |2 +-
 net/tc35815.c |2 +-
 net/tg3.c |2 +-
 net/tokenring/3c359.c |2 +-
 net/tokenring/lanstreamer.c   |2 +-
 net/tokenring/olympic.c   |2 +-
 net/tulip/de2104x.c   |2 +-
 net/tulip/de4x5.c |2 +-
 net/tulip/dmfe.c  |2 +-
 net/tulip/tulip_core.c|2 +-
 net/tulip/winbond-840.c   |2 +-
 net/tulip/xircom_tulip_cb.c   |2 +-
 net/typhoon.c |2 +-
 net/via-rhine.c   |2 +-
 net/via-velocity.c|2 +-
 net/wan/dscc4.c   |2 +-
 net/wan/farsync.c |2 +-
 net/wan/lmc/lmc_main.c|2 +-
 net/wan/pc300_drv.c   |2 +-
 net/wan/pci200syn.c   |2 +-
 net/wan/wanxl.c   |2 +-
 net/wireless/atmel_pci.c  |2 +-
 net/wireless/ipw2100.c|2 +-
 net/wireless/ipw2200.c|2 +-
 net/wireless/orinoco_nortel.c |2 +-
 net/wireless/orinoco_pci.c|2 +-
 net/wireless/orinoco_plx.c|2 +-
 net/wireless/orinoco_tmd.c|2 +-
 net/wireless/prism54/islpci_hotplug.c |2 +-
 net/yellowfin.c   |2 +-
 scsi/ips.c|2 +-
 scsi/megaraid.c   |2 +-
 scsi/megaraid/megaraid_sas.c  |2 +-
 scsi/tmscsim.c|2 +-
 75 files changed, 75 insertions(+), 75 deletions(-)

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group
(http://www.stardust.webpages.pl/ltg/wiki/)
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC][PATCH 0/75] pci_module_init to pci_register_driver conversion

2006-08-17 Thread Michal Piotrowski

On 17/08/06, Jeff Garzik <[EMAIL PROTECTED]> wrote:

Greg KH wrote:
> On Thu, Aug 17, 2006 at 04:26:35AM +, Michal Piotrowski wrote:
>> Hi,
>>
>> pci_module_init is obsolete.
>>
>> This patch series converts pci_module_init to pci_register_driver.
>>
>>
>> Can I remove this?
>>
>> include/linux/pci.h:385
>> /*
>>  * pci_module_init is obsolete, this stays here till we fix up all usages of 
it
>>  * in the tree.
>>  */
>> #define pci_module_init pci_register_driver
>
> As repeated numerous times, it's up to the network developers if they
> will take this or not.
>
> I'll hold off on taking this series, please push it through the driver
> subsystem maintainers.

It's already in subsystem trees, in fact.


Did you mean 
http://www.kernel.org/git/?p=linux/kernel/git/davem/net-2.6.19.git;a=shortlog
?

The patches are against 2.6.18-rc4-mm1. Unfortunately patches weren't
delivered to netdev (my fault). I'll add To: and
resend.



But it is most definitely not 2.6.18-rc material :)


I agree.



Jeff



Regards,
Michal

--
Michal K. K. Piotrowski
LTG - Linux Testers Group
(http://www.stardust.webpages.pl/ltg/wiki/)
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC][PATCH 0/75] pci_module_init to pci_register_driver conversion

2006-08-17 Thread Jeff Garzik

Arjan van de Ven wrote:

But it is most definitely not 2.6.18-rc material :)


that part of it isn't.. marking it deprecated so that people doing new
code get a warning probably IS 2.6.18, the sooner that happens the
better


I disagree.  With all the cleanups going into 2.6.19, that would just 
introduce needless build noise into 2.6.18.


Jeff



-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC][PATCH 0/75] pci_module_init to pci_register_driver conversion

2006-08-17 Thread Jeff Garzik

Greg KH wrote:

On Thu, Aug 17, 2006 at 04:26:35AM +, Michal Piotrowski wrote:

Hi,

pci_module_init is obsolete.

This patch series converts pci_module_init to pci_register_driver.


Can I remove this?

include/linux/pci.h:385
/*
 * pci_module_init is obsolete, this stays here till we fix up all usages of it
 * in the tree.
 */
#define pci_module_init pci_register_driver


As repeated numerous times, it's up to the network developers if they
will take this or not.

I'll hold off on taking this series, please push it through the driver
subsystem maintainers.


It's already in subsystem trees, in fact.

But it is most definitely not 2.6.18-rc material :)

Jeff



-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC][PATCH 0/75] pci_module_init to pci_register_driver conversion

2006-08-17 Thread Arjan van de Ven

> But it is most definitely not 2.6.18-rc material :)

that part of it isn't.. marking it deprecated so that people doing new
code get a warning probably IS 2.6.18, the sooner that happens the
better



-- 
if you want to mail me at work (you don't), use arjan (at) linux.intel.com

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC][PATCH 0/75] pci_module_init to pci_register_driver conversion

2006-08-17 Thread Greg KH
On Thu, Aug 17, 2006 at 04:26:35AM +, Michal Piotrowski wrote:
> 
> Hi,
> 
> pci_module_init is obsolete.
> 
> This patch series converts pci_module_init to pci_register_driver.
> 
> 
> Can I remove this?
> 
> include/linux/pci.h:385
> /*
>  * pci_module_init is obsolete, this stays here till we fix up all usages of 
> it
>  * in the tree.
>  */
> #define pci_module_init pci_register_driver

As repeated numerous times, it's up to the network developers if they
will take this or not.

I'll hold off on taking this series, please push it through the driver
subsystem maintainers.

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html