[SOLVED] if_vr(4) and DFE520-TX [working with patched if_rl]

[YongHyeon PYUN]

On Tue, Jan 15, 2013 at 01:04:49PM +0400, Ruslan Makhmatkhanov wrote:
> YongHyeon PYUN wrote on 15.01.2013 10:51:
> >
> >Hmm, I don't get it.
> >Diff inlined again.
> >
> >Index: sys/pci/if_rlreg.h
> >===================================================================
> >--- sys/pci/if_rlreg.h	(revision 245199)
> >+++ sys/pci/if_rlreg.h	(working copy)
> >@@ -1048,6 +1048,11 @@ struct rl_softc {
> >  #define	DLINK_DEVICEID_530TXPLUS		0x1300
> >
> >  /*
> >+ * D-Link DFE-520TX rev. C1 device ID
> >+ */
> >+#define	DLINK_DEVICEID_520TX_REVC1		0x4200
> >+
> >+/*
> >   * D-Link DFE-5280T device ID
> >   */
> >  #define	DLINK_DEVICEID_528T			0x4300
> >Index: sys/pci/if_rl.c
> >===================================================================
> >--- sys/pci/if_rl.c	(revision 245199)
> >+++ sys/pci/if_rl.c	(working copy)
> >@@ -148,6 +148,8 @@ static const struct rl_type rl_devs[] = {
> >  		"Delta Electronics 8139 10/100BaseTX" },
> >  	{ ADDTRON_VENDORID, ADDTRON_DEVICEID_8139, RL_8139,
> >  		"Addtron Technology 8139 10/100BaseTX" },
> >+	{ DLINK_VENDORID, DLINK_DEVICEID_520TX_REVC1, RL_8139,
> >+		"D-Link DFE-520TX (rev. C1) 10/100BaseTX" },
> >  	{ DLINK_VENDORID, DLINK_DEVICEID_530TXPLUS, RL_8139,
> >  		"D-Link DFE-530TX+ 10/100BaseTX" },
> >  	{ DLINK_VENDORID, DLINK_DEVICEID_690TXD, RL_8139,
> >>
> 
> Hooray! It is working with if_rl with your patch (loader tunable isn't 
> used). Thanks a lot for this!
> 
> Can this be committed and merged to 8/9?
> 

Yes, committed in r245485.
I will MFC to stable 9/8 after a week.

> rl1 at pci0:4:1:0:	class=0x020000 card=0x11031186 chip=0x42001186 
> rev=0x10 hdr=0x00
>     vendor     = 'D-Link System Inc'
>     class      = network
>     subclass   = ethernet
> 
> 
> rl1: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> metric 0 mtu 1500
> 	options=2008<VLAN_MTU,WOL_MAGIC>
> 	ether 90:94:e4:82:d5:e6
> 	inet 192.168.0.208 netmask 0xffffff00 broadcast 192.168.0.255
> 	nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
> 	media: Ethernet autoselect (100baseTX <full-duplex>)
> 	status: active
> 
> Ping and all other working fine.
> 

Thanks for testing!

> -- 
> Regards,
> Ruslan
> 
> Tinderboxing kills... the drives.