From patchwork Wed Jul 6 21:54:07 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [Bugme-new,Bug,38862] New: No support for DGE-530T Rev C1 Date: Wed, 06 Jul 2011 11:54:07 -0000 From: Andrew Morton X-Patchwork-Id: 103581 Message-Id: <20110706145407.7e06eece.akpm@linux-foundation.org> To: James Henderson Cc: "bugme-daemon@bugzilla.kernel.org" , "netdev@vger.kernel.org" , Stephen Hemminger On Wed, 6 Jul 2011 17:48:44 -0400 James Henderson wrote: > > Did you test simply adding that device to the driver? > > > > --- a/drivers/net/skge.c~a > > +++ a/drivers/net/skge.c > > @@ -89,6 +89,7 @@ static DEFINE_PCI_DEVICE_TABLE(skge_id_t > > { PCI_DEVICE(PCI_VENDOR_ID_SYSKONNECT, PCI_DEVICE_ID_SYSKONNECT_YU) }, > > { PCI_DEVICE(PCI_VENDOR_ID_DLINK, PCI_DEVICE_ID_DLINK_DGE510T) }, > > { PCI_DEVICE(PCI_VENDOR_ID_DLINK, 0x4b01) }, /* DGE-530T */ > > + { PCI_DEVICE(PCI_VENDOR_ID_DLINK, 0x4302) }, /* DGE-530T Rev C1 */ > > { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x4320) }, > > { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x5005) }, /* Belkin */ > > { PCI_DEVICE(PCI_VENDOR_ID_CNET, PCI_DEVICE_ID_CNET_GIGACARD) }, > > _ > > > > > No I haven't tested that change - I don't have a kernel development > environment setup and unfortunately I don't have any more work time to > budget to the issue beyond reporting it. > > Also, I meant to say that Rev B2 has PCI id 1186:4B01 (although you seem > to have figured that out). OK, I suppose we can add that info thusly: --- a/drivers/net/skge.c~drivers-net-skgec-support-dlink-dge-530t-rev-c1 +++ a/drivers/net/skge.c @@ -88,7 +88,8 @@ static DEFINE_PCI_DEVICE_TABLE(skge_id_t { PCI_DEVICE(PCI_VENDOR_ID_SYSKONNECT, PCI_DEVICE_ID_SYSKONNECT_GE) }, { PCI_DEVICE(PCI_VENDOR_ID_SYSKONNECT, PCI_DEVICE_ID_SYSKONNECT_YU) }, { PCI_DEVICE(PCI_VENDOR_ID_DLINK, PCI_DEVICE_ID_DLINK_DGE510T) }, - { PCI_DEVICE(PCI_VENDOR_ID_DLINK, 0x4b01) }, /* DGE-530T */ + { PCI_DEVICE(PCI_VENDOR_ID_DLINK, 0x4b01) }, /* DGE-530T Rev B2 */ + { PCI_DEVICE(PCI_VENDOR_ID_DLINK, 0x4302) }, /* DGE-530T Rev C1 */ { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x4320) }, { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x5005) }, /* Belkin */ { PCI_DEVICE(PCI_VENDOR_ID_CNET, PCI_DEVICE_ID_CNET_GIGACARD) },