From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757210AbaKTQTA (ORCPT ); Thu, 20 Nov 2014 11:19:00 -0500 Received: from mail-pa0-f44.google.com ([209.85.220.44]:43846 "EHLO mail-pa0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754913AbaKTQS6 (ORCPT ); Thu, 20 Nov 2014 11:18:58 -0500 Message-ID: <546E1467.9090607@gmail.com> Date: Thu, 20 Nov 2014 08:18:47 -0800 From: Florian Fainelli User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Johan Hovold CC: "David S. Miller" , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Bruno Thomsen , Sascha Hauer , Mark Rutland Subject: Re: [PATCH 00/10] net: phy: add device-type abstraction References: <1416398363-32306-1-git-send-email-johan@kernel.org> In-Reply-To: <1416398363-32306-1-git-send-email-johan@kernel.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/19/2014 03:59 AM, Johan Hovold wrote: > This series adds device and device-type abstractions to the micrel > driver, and enables support for RMII-reference clock selection for > KSZ8081 and KSZ8091 devices. > > While adding support for more features for the Micrel PHYs mentioned > above, it became apparent that the configuration space is much too large > and that adding type-specific callbacks will simply not scale. Instead I > added a driver_data field to struct phy_device, which can be used to > store static device type data that can be parsed and acted on in > generic driver callbacks. This allows a lot of duplicated code to be > removed, and should make it much easier to add new features or deal with > device-type quirks in the future. > > The series has been tested on a dual KSZ8081 setup. Further testing on > other Micrel PHYs would be much appreciated. > > The recent commit a95a18afe4c8 ("phy/micrel: KSZ8031RNL RMII clock > reconfiguration bug") currently prevents KSZ8031 PHYs from using the > generic config-init. Bruno, who is the author of that patch, has agreed > to test this series and some follow-up diagnostic patches to determine > how best to incorporate these devices as well. I intend to send a > follow-up patch that removes the custom 8031 config-init and documents > this quirk, but the current series can be applied meanwhile. > > These patches are against net-next which contains some already merged > prerequisite patches to the driver. LGTM, thanks Johan! Reviewed-by: Florian Fainelli > > Johan > > > Johan Hovold (10): > net: phy: add static data field to struct phy_driver > net: phy: micrel: add device-type abstraction > net: phy: micrel: parse of nodes at probe > net: phy: micrel: add has-broadcast-disable flag to type data > net: phy: micrel: add generic clock-mode-select support > net: phy: micrel: add support for clock-mode select to KSZ8081/KSZ8091 > dt/bindings: reformat micrel eth-phy documentation > dt/bindings: add clock-select function property to micrel phy binding > net: phy: micrel: refactor interrupt config > net: phy: micrel: add copyright entry > > Documentation/devicetree/bindings/net/micrel.txt | 37 ++-- > drivers/net/phy/micrel.c | 262 ++++++++++++++--------- > include/linux/micrel_phy.h | 1 - > include/linux/phy.h | 2 + > 4 files changed, 184 insertions(+), 118 deletions(-) >