Re: [RFC 54/72] au1000_eth/mipsnet/tc35815: Move the MIPS drivers

Re: [RFC 54/72] au1000_eth/mipsnet/tc35815: Move the MIPS drivers

[Ralf Baechle]

On Sat, Jun 25, 2011 at 06:59:05AM -0700, Jeff Kirsher wrote:

> Move the MIPS drivers into drivers/net/ethernet/mips/ and
> make the necessary Kconfig and Makefile changes.

NAck.

sni_82596.c is a driver for the same chip but used in particular Siemens-
Nixdorf systems.  Lasi_82596 is an Intel NIC being used in PARISC systems.
Not a good reason th throw PA-RISC drivers into the same directory after
all PA-RISC has no relation to MIPS at all.  So maybe they should go into
drivers/net/ethernet/i82596 or drivers/net/ethernet/intel.  And wherever
they go, both drivers #include lib82596.c which your patch did not move
around.

It seems that lp486e should probably go into the same directory and I
suspect ni52 and sun3_82586 are close enough that they should also in
the faint hope that somebody will factor out the common code.

MIPSnet is a driver for the emulated network card of MIPSsim, a software
simulator by MIPS Technologies and the only net thingy in your patch that
truely is "MIPS".

The remaining happen to be used by particular systems that happen to be
based on a MIPS processor but there is no technial communality.  MIPS is
a processor architecture - not a system architecture and grouping things
by processor architecture hasn't worked so well imho for other things in
the past, see for example sound.

If we're already moving things around maybe au1000_net.[ch] can be renamed
to alchemy.[ch].

Your patch moves the Makefile and Kconfig bits of lantiq_etop but doesn't
move the actual lantiq_etop.c driver.

> +++ b/drivers/net/ethernet/mips/Kconfig
> @@ -0,0 +1,61 @@
> +#
> +# MIPS device configuration
> +#
> +
> +config NET_MIPS
> +	bool "MIPS/PS-RISC devices"

It's speeled PA-RISC.

  Ralf

Re: [RFC 54/72] au1000_eth/mipsnet/tc35815: Move the MIPS drivers

[Jeff Kirsher]

[-- Attachment #1: Type: text/plain, Size: 869 bytes --]

On Sat, 2011-06-25 at 09:13 -0700, Ralf Baechle wrote:
> On Sat, Jun 25, 2011 at 06:59:05AM -0700, Jeff Kirsher wrote:
> 
> > Move the MIPS drivers into drivers/net/ethernet/mips/ and
> > make the necessary Kconfig and Makefile changes.
> 
> NAck.
> 
> sni_82596.c is a driver for the same chip but used in particular
> Siemens-
> Nixdorf systems.  Lasi_82596 is an Intel NIC being used in PARISC
> systems.
> Not a good reason th throw PA-RISC drivers into the same directory
> after
> all PA-RISC has no relation to MIPS at all.  So maybe they should go
> into
> drivers/net/ethernet/i82596 or drivers/net/ethernet/intel.  And
> wherever
> they go, both drivers #include lib82596.c which your patch did not
> move
> around. 

My bad, I had grouped these drivers because they both used lib82596.c
and had thought I had moved that file as well.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 490 bytes --]