(copied to tech-net, and reply-to there)

On Tue, Jul 31, 2001 at 07:23:50PM +0300, Jukka Andberg wrote:
> I happen to own the card mentioned on the subject line. To make NetBSD
> recognise it I cloned the pcmciadevs entries for DE-660 and made the
> obvious changes. Is this like the right thing to do?

In general, you would be lucky, but as

> Anyway it seems to work fine,

you were. I guess you might miss some new bells and wistles, but as long as
we don't have information to judge, it's ok. I'm forwarding this to tech-net,
to get more insight.

> save that after cold-boot one needs to
> remove and replug it, otherwise it is not found (the hardware bug i
> expect).

Non-Amigans: this is an Amiga 1200 pcmcia interface PAL bug. Nothing you need
to be worried about.

> dmesgs before and after patches, and patches follow..:
> 
> ---
> 
> pccard0 at mainbus0
> pcmcia0 at pccard0
> D-Link, DE-660+, A0004743118001 (manufacturer 0xffffffff, product 0xffffffff) at pcmcia0,  function 0 not configured
> 
> pccard0 at mainbus0
> pcmcia0 at pccard0
> ne0 at pcmcia0 function 0
> ne0: D-Link DE-660+ Ethernet
> ne0: Ethernet address 00:50:ba:34:63:17
> 
> Index: if_ne_pcmcia.c
> ===================================================================
> RCS file: /cvsroot/syssrc/sys/dev/pcmcia/if_ne_pcmcia.c,v
> retrieving revision 1.78
> diff -u -r1.78 if_ne_pcmcia.c
> --- if_ne_pcmcia.c	2001/07/05 08:03:45	1.78
> +++ if_ne_pcmcia.c	2001/07/30 18:55:56
> @@ -131,6 +131,11 @@
>        PCMCIA_CIS_DLINK_DE660,
>        0, -1, { 0x00, 0x80, 0xc8 } },
> 
> +    { PCMCIA_STR_DLINK_DE660PLUS,
> +      PCMCIA_VENDOR_INVALID, PCMCIA_PRODUCT_INVALID,
> +      PCMCIA_CIS_DLINK_DE660PLUS,
> +      0, -1, { 0x00, 0x80, 0xc8 } },
> +
>      { PCMCIA_STR_RPTI_EP400,
>        PCMCIA_VENDOR_INVALID, PCMCIA_PRODUCT_INVALID,
>        PCMCIA_CIS_RPTI_EP400,
> Index: pcmciadevs
> ===================================================================
> RCS file: /cvsroot/syssrc/sys/dev/pcmcia/pcmciadevs,v
> retrieving revision 1.132
> diff -u -r1.132 pcmciadevs
> --- pcmciadevs	2001/07/09 06:57:59	1.132
> +++ pcmciadevs	2001/07/30 18:56:00
> @@ -325,6 +325,7 @@
>  product PLANEX FNW3700T		-1 Planex FNW-3700-T
>  product DLINK DE650		{ "D-Link", "DE-650", NULL, NULL } D-Link DE-650
>  product DLINK DE660		{ "D-Link", "DE-660", NULL, NULL } D-Link DE-660
> +product DLINK DE660PLUS		{ "D-Link", "DE-660+", NULL, NULL } D-Link DE-660+
>  product RPTI EP400		{ "RPTI&spLTD.", "EP400", "CISV100", NULL } RPTI EP400
>  product RPTI EP401		{ "RPTI", "EP401&spEthernet&spNE2000&spCompatible", NULL, NULL } RPTI EP401
>  product ACCTON EN2212		{ "ACCTON", "EN2212", NULL, NULL } Accton EN2212
> 
>