Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752912AbbHUUYV (ORCPT ); Fri, 21 Aug 2015 16:24:21 -0400 Received: from smtp-1b.atlantis.sk ([80.94.52.26]:35241 "EHLO smtp-1b.atlantis.sk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752158AbbHUUYT (ORCPT ); Fri, 21 Aug 2015 16:24:19 -0400 From: Ondrej Zary To: netdev@vger.kernel.org Subject: ipg and dl2k mess Date: Fri, 21 Aug 2015 22:23:58 +0200 User-Agent: KMail/1.9.10 (enterprise35 0.20100827.1168748) Cc: Kernel development list , Francois Romieu , Sorbica Shieh MIME-Version: 1.0 Content-Disposition: inline X-Length: 784 Content-Type: Text/Plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Message-Id: <201508212224.00090.linux@rainbow-software.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2561 Lines: 63 Hello, I've got an Asus NX1101 card with ICPlus IP1000A chip: 02:01.0 Ethernet controller [0200]: Sundance Technology Inc / IC Plus Corp IP1000 Family Gigabit Ethernet [13f0:1023] (rev 41) Subsystem: ASUSTeK Computer Inc. NX1101 [1043:8180] Flags: bus master, 66MHz, medium devsel, latency 32, IRQ 19 I/O ports at a000 [size=256] Memory at f5000000 (32-bit, non-prefetchable) [size=256] [virtual] Expansion ROM at 30000000 [disabled] [size=64K] Capabilities: [50] Power Management version 2 It does not work properly because the ipg driver is broken - it loses packets (easily reproduced by ping -f) and stops working under load with no messages (copying a 200MB file using scp at 100mbit is enough to reproduce it). The dl2k (for TC902x chips, DL2000 is probably a rebranded TC902x) driver is very similar to ipg (for IP1000A). According to datasheets, IP1000A chip looks like a TC9021 with integrated PHY. The patch below is enough to make my IP1000A card work with dl2k driver - no more lost packets and hangs. Haven't tested gigabit speed yet - the PHY will probably need some tweaking but that should be easy. So maybe we should add IP1000A support to dl2k and remove the broken ipg driver. Does anyone have HW to test? diff --git a/drivers/net/ethernet/dlink/dl2k.c b/drivers/net/ethernet/dlink/dl2k.c index cf0a5fc..d5a60fe 100644 --- a/drivers/net/ethernet/dlink/dl2k.c +++ b/drivers/net/ethernet/dlink/dl2k.c @@ -433,9 +455,9 @@ rio_open (struct net_device *dev) alloc_list (dev); - /* Get station address */ - for (i = 0; i < 6; i++) - dw8(StationAddr0 + i, dev->dev_addr[i]); + /* Set station address */ + for (i = 0; i < 3; i++) + dw16(StationAddr0 + 2 * i, cpu_to_le16(((u16 *)dev->dev_addr)[i])); set_multicast (dev); if (np->coalesce) { diff --git a/drivers/net/ethernet/dlink/dl2k.h b/drivers/net/ethernet/dlink/dl2k.h index 23c07b0..da35e66 100644 --- a/drivers/net/ethernet/dlink/dl2k.h +++ b/drivers/net/ethernet/dlink/dl2k.h @@ -411,6 +411,7 @@ struct netdev_private { static const struct pci_device_id rio_pci_tbl[] = { {0x1186, 0x4000, PCI_ANY_ID, PCI_ANY_ID, }, {0x13f0, 0x1021, PCI_ANY_ID, PCI_ANY_ID, }, + {0x13f0, 0x1023, PCI_ANY_ID, PCI_ANY_ID, }, { } }; MODULE_DEVICE_TABLE (pci, rio_pci_tbl); -- Ondrej Zary -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/