Thread (4 messages) 4 messages, 2 authors, 2005-03-23

Re: Resend: [NETDRV] Merge register_netdev calls

From: Jeff Garzik <hidden>
Date: 2004-09-20 19:03:51

Herbert Xu wrote:
On Fri, Jun 11, 2004 at 12:08:55PM +1000, herbert wrote:
quoted
In fact it's really making these ISA/MCA probe() functions more
like the ones we have for PCI.

To illustrate this, let's take the first driver touched by 4/x.
In 3c503, the function init_module() essentially does

for each ioaddr
	if (do_el2_probe(ioaddr) == 0)
		return 0

And do_el2_probe() just calls el2_probe1() which is similar
to your average PCI probe function except that the first thing
it does is to make sure that the device exists at ioaddr.

This is not that different from PCI where it would look
like

for each PCI device matching the vendor/product numbers
	if (do_el2_probe(device) == 0)
		return 0

Now before my patch, register_netdev was being called just
after do_el2_probe() returns.  My patch simply moves it to
the end of el2_probe1() which is exactly what would happen
if this were a PCI driver.

I've rediffed it against your net-drivers-2.6 tree.
so,

there was a lot of churn and I held off on this patch.

could you be convinced to rediff and resend (again)?

	Jeff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help