Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932581AbcK1JrG (ORCPT ); Mon, 28 Nov 2016 04:47:06 -0500 Received: from mail-wj0-f176.google.com ([209.85.210.176]:34379 "EHLO mail-wj0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754289AbcK1Jq7 (ORCPT ); Mon, 28 Nov 2016 04:46:59 -0500 From: Jerome Brunet To: netdev@vger.kernel.org, devicetree@vger.kernel.org, Florian Fainelli Cc: Jerome Brunet , Carlo Caione , Kevin Hilman , Giuseppe Cavallaro , Alexandre TORGUE , Martin Blumenstingl , Andre Roth , Andrew Lunn , Neil Armstrong , linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v3 0/4] Fix OdroidC2 Gigabit Tx link issue Date: Mon, 28 Nov 2016 10:46:45 +0100 Message-Id: <1480326409-25419-1-git-send-email-jbrunet@baylibre.com> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2074 Lines: 47 This patchset fixes an issue with the OdroidC2 board (DWMAC + RTL8211F). The platform seems to enter LPI on the Rx path too often while performing relatively high TX transfer. This eventually break the link (both Tx and Rx), and require to bring the interface down and up again to get the Rx path working again. The root cause of this issue is not fully understood yet but disabling EEE advertisement on the PHY prevent this feature to be negotiated. With this change, the link is stable and reliable, with the expected throughput performance. The patchset adds options in the generic phy driver to disable EEE advertisement, through device tree. The way it is done is very similar to the handling of the max-speed property. Changes since V2: [2] - Rename "eee-advert-disable" to "eee-broken-modes" to make the intended purpose of this option clear (flag broken configuration, not a configuration option) - Add DT bindings constants so the DT configuration is more user friendly - Submit to net-next instead of net. Changes since V1: [1] - Disable the advertisement of EEE in the generic code instead of the realtek driver. [1] : http://lkml.kernel.org/r/1479220154-25851-1-git-send-email-jbrunet@baylibre.com [2] : http://lkml.kernel.org/r/1479742524-30222-1-git-send-email-jbrunet@baylibre.com Jerome Brunet (4): net: phy: add an option to disable EEE advertisement dt-bindings: net: add EEE capability constants dt: bindings: add ethernet phy eee-broken-modes option documentation ARM64: dts: meson: odroidc2: disable advertisement EEE for GbE. Documentation/devicetree/bindings/net/phy.txt | 2 + .../arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 16 +++++ drivers/net/phy/phy.c | 3 + drivers/net/phy/phy_device.c | 80 +++++++++++++++++++--- include/dt-bindings/net/mdio.h | 19 +++++ include/linux/phy.h | 3 + 6 files changed, 114 insertions(+), 9 deletions(-) create mode 100644 include/dt-bindings/net/mdio.h -- 2.7.4