From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Daney Subject: Re: [PATCH v2 0/3] net/phy: Improvements to Cavium Thunder MDIO code. Date: Fri, 11 Mar 2016 16:08:44 -0800 Message-ID: <56E35E0C.9040500@caviumnetworks.com> References: <1457718791-6505-1-git-send-email-ddaney.cavm@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Cc: "David S. Miller" , , , Florian Fainelli , Robert Richter , Sunil Goutham , Kumar Gala , Ian Campbell , Mark Rutland , Pawel Moll , Rob Herring , , Radha Mohan Chintakuntla , David Daney To: David Daney Return-path: Received: from mail-by2on0063.outbound.protection.outlook.com ([207.46.100.63]:2120 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751720AbcCLAIv (ORCPT ); Fri, 11 Mar 2016 19:08:51 -0500 In-Reply-To: <1457718791-6505-1-git-send-email-ddaney.cavm@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: I am going to send a new version of this set. David Daney On 03/11/2016 09:53 AM, David Daney wrote: > From: David Daney > > Changes from v1: > > - In 1/3 Add back check for non-OF objects in bgx_init_of_phy(). It > is probably not necessary, but better safe than sorry... > > The firmware on many Cavium Thunder systems configures the MDIO bus > hardware to be probed as a PCI device. In order to use the MDIO bus > drivers in this configuration, we must add PCI probing to the driver. > > There are two parts to this set of three patches: > > 1) Cleanup the PHY probing code in thunder_bgx.c to handle the case > where there is no PHY attached to a port, as well as being more > robust in the face of driver loading order by use of > -EPROBE_DEFER. > > 2) Split mdio-octeon.c into two drivers, one with platform probing, > and the other with PCI probing. Common code is shared between the > two. > > Tested on several different Thunder and OCTEON systems, also compile > tested on x86_64. > > David Daney (3): > net: thunderx: Cleanup PHY probing code. > phy: mdio-octeon: Refactor into two files/modules > phy: mdio-thunder: Add driver for Cavium Thunder SoC MDIO buses. > > .../devicetree/bindings/net/cavium-mdio.txt | 61 ++++- > drivers/net/ethernet/cavium/thunder/thunder_bgx.c | 29 ++- > drivers/net/phy/Kconfig | 22 +- > drivers/net/phy/Makefile | 2 + > drivers/net/phy/mdio-cavium.c | 149 +++++++++++ > drivers/net/phy/mdio-cavium.h | 119 +++++++++ > drivers/net/phy/mdio-octeon.c | 280 ++------------------- > drivers/net/phy/mdio-thunder.c | 154 ++++++++++++ > 8 files changed, 535 insertions(+), 281 deletions(-) > create mode 100644 drivers/net/phy/mdio-cavium.c > create mode 100644 drivers/net/phy/mdio-cavium.h > create mode 100644 drivers/net/phy/mdio-thunder.c >