From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1034448AbcIYPtz (ORCPT ); Sun, 25 Sep 2016 11:49:55 -0400 Received: from mail-oi0-f66.google.com ([209.85.218.66]:35555 "EHLO mail-oi0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1034226AbcIYPty (ORCPT ); Sun, 25 Sep 2016 11:49:54 -0400 Subject: Re: [PATCH 0/3] ethernet: net: bcmgenet: only use new api ethtool_{get|set}_link_ksettings To: Philippe Reynes , jaedon.shin@gmail.com, davem@davemloft.net References: <1474817794-31791-1-git-send-email-tremyfr@gmail.com> Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org From: Florian Fainelli Message-ID: Date: Sun, 25 Sep 2016 08:49:50 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <1474817794-31791-1-git-send-email-tremyfr@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le 25/09/2016 à 08:36, Philippe Reynes a écrit : > Some times ago, a serie of patches were committed : > - commit 62469c76007e ("net: ethernet: bcmgenet: use phydev from struct net_device") > - commit 6b352ebccbcf ("net: ethernet: broadcom: bcmgenet: use new api ethtool_{get|set}_link_ksettings") > The first patch add a regression on this driver, so it should be reverted. > As the second patch depend on the former, it should be reverted too. > > The first patch is buggy because there is a "trick" in this driver. > The structure phydev is kept in the private data when the interface > go down, and used when the interface go up to enable the phy before > the function phy_connect is called. > > I don't have this hardware, neither the datasheet. So I won't > update the driver to avoid this trick. > > But the real goal of the first serie was to move to the new api > ethtool_{get|set}_link_ksettings. So I provide a new version of > the patch without the "cleaning" of driver to use the phydev > store in the net_device structure. > > Jaedon Shin (1): > Revert "net: ethernet: bcmgenet: use phydev from struct net_device" Please replace Jaedon's patch with mine which contains more background as to what the problem was and how it gets fixed. > > Philippe Reynes (2): > Revert "net: ethernet: bcmgenet: use new api > ethtool_{get|set}_link_ksettings" > net: ethernet: broadcom: bcmgenet: use new api > ethtool_{get|set}_link_ksettings Can you be consistent in the subject and just use "net: bcmgenet: " as a prefix here? > > drivers/net/ethernet/broadcom/genet/bcmgenet.c | 33 ++++++++++++----------- > drivers/net/ethernet/broadcom/genet/bcmgenet.h | 1 + > drivers/net/ethernet/broadcom/genet/bcmmii.c | 24 +++++++++-------- > 3 files changed, 31 insertions(+), 27 deletions(-) > -- Florian