Re: [PATCH net-next] Amiga PCMCIA 100 MBit card support

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

 



Dave,

Am 21.12.2018 um 13:10 schrieb David Miller:
From: ALeX Kazik <alex@xxxxxxxx>
Date: Wed, 19 Dec 2018 23:24:32 +0100

+	bool "PCMCIA NE2000 100MBit support"
+	default n
+	depends on ARM_ETHERH=n && AX88796=n && HYDRA=n && MAC8390=n
+	depends on MCF8390=n && NE2000=n && NE2K_PCI=n && PCMCIA_AXNET=n
+	depends on PCMCIA_PCNET=n && STNIC=n && ULTRA=n && WD80x3=n
+	depends on XSURF100=n && ZORRO8390=n
+	---help---

Generally I'm not too happy with these changes, they are all very hackish.

I see only one option to address that: rewrite apne.c to include lib8390.c directly.

Moving the inb() and outb() redefines into the m68k version of io.h (which Geert is likely to veto because of its hackishness) would still require a Kconfig hack.

And in particular this huge complicated Kconfig construct is not
maintainable at all.

We can trim this down a bit (for reasons I've outlined before):

+if APNE
+config APNE100MBIT
+	bool "PCMCIA NE2000 100MBit support"
+	default n
+	depends on NE2000=n && PCMCIA_AXNET=n
+	depends on PCMCIA_PCNET=n && STNIC=n && ULTRA=n && WD80x3=n

In all honesty, I doubt anyone could ever use the stnic, ultra and wd drivers on m68k (Geert?) so these could be omitted as well. Not sure pcnet_cs or axnet_cs are possible to use on the Amiga PCMCIA slot, so all that remains in practice is the ne driver (which is used on Atari).

Still too ugly?

Cheers,

	Michael


Sorry.




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

  Powered by Linux