lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Sun, 18 Mar 2007 13:26:37 +1100 (EST)
From:	Finn Thain <fthain@...egraphics.com.au>
To:	Brad Boyer <flar@...andria.com>
cc:	Geert Uytterhoeven <geert@...ux-m68k.org>,
	Jeff Garzik <jgarzik@...ox.com>, netdev@...r.kernel.org,
	linux-m68k@...r.kernel.org
Subject: Re: [PATCH 11/13] mac68k: macmace fixes



On Sat, 17 Mar 2007, Brad Boyer wrote:

> On Sat, Mar 17, 2007 at 08:08:33PM +1100, Finn Thain wrote:
> > Index: linux-2.6.20/drivers/net/Kconfig
> > ===================================================================
> > --- linux-2.6.20.orig/drivers/net/Kconfig	2007-03-06 19:22:38.000000000 +1100
> > +++ linux-2.6.20/drivers/net/Kconfig	2007-03-06 20:15:52.000000000 +1100
> > @@ -347,8 +347,8 @@ config MACSONIC
> >  	  be called macsonic.
> >  
> >  config MACMACE
> > -	bool "Macintosh (AV) onboard MACE ethernet (EXPERIMENTAL)"
> > -	depends on NET_ETHERNET && MAC && EXPERIMENTAL
> > +	bool "Macintosh (AV) onboard MACE ethernet"
> > +	depends on NET_ETHERNET && MAC
> >  	select CRC32
> >  	help
> >  	  Support for the onboard AMD 79C940 MACE Ethernet controller used in
> 
> Have you tried making this a tristate option? It looked like the code 
> had all the proper hooks to be built as a module, but it currently isn't 
> allowed by the Kconfig.

Yes, I tried it. The problem was the psc global, which wouldn't allow it 
to link as a module. So this patch just focuses on getting the driver 
working, which is what the debian devs needed. I wasn't particularly 
concerned with enhancements, so I just left MACMACE as a boolean.

Even though no other code uses the PSC's DMA engine, there's probably an 
argument for making the Peripheral Controller Chip a device in it's own 
right. (Well, it serves as the interrupt controller too. And there are 
other patches in this series that fix the PSC interrupt problems.)

Modernization needs to happen, but I'm just trying to fix borken stuff 
because I think that needs to happen first.

-f

> 
> 	Brad Boyer
> 	flar@...andria.com
> 
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Powered by blists - more mailing lists