Re: [PATCH 11/13] mac68k: macmace fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





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@xxxxxxxxxxxxx

-
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Video for Linux]     [Yosemite News]     [Linux S/390]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux