netdev - Re: [PATCH v1 0/7] ACPI support for xgmac

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <fcda49b6-7a45-cd86-e33e-f8dea07c0684@gmail.com>
Date:   Mon, 3 Feb 2020 10:02:15 -0800
From:   Florian Fainelli <f.fainelli@...il.com>
To:     Calvin Johnson <calvin.johnson@....com>, linux.cj@...il.com,
        Jon Nettleton <jon@...id-run.com>, linux@...linux.org.uk,
        Makarand Pawagi <makarand.pawagi@....com>,
        cristian.sovaiala@....com, laurentiu.tudor@....com,
        ioana.ciornei@....com, V.Sethi@....com, pankaj.bansal@....com,
        "Rajesh V . Bikkina" <rajesh.bikkina@....com>,
        Jeremy Linton <jeremy.linton@....com>
Cc:     Calvin Johnson <calvin.johnson@....nxp.com>,
        Andrew Lunn <andrew@...n.ch>,
        Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
        Antoine Tenart <antoine.tenart@...tlin.com>,
        "David S. Miller" <davem@...emloft.net>,
        Dmitry Torokhov <dmitry.torokhov@...il.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Heikki Krogerus <heikki.krogerus@...ux.intel.com>,
        Heiner Kallweit <hkallweit1@...il.com>,
        Ioana Radulescu <ruxandra.radulescu@....com>,
        Madalin Bucur <madalin.bucur@....nxp.com>,
        Matteo Croce <mcroce@...hat.com>,
        Maxime Chevallier <maxime.chevallier@...tlin.com>,
        "Rafael J. Wysocki" <rafael@...nel.org>,
        Russell King <rmk+kernel@...linux.org.uk>,
        Sakari Ailus <sakari.ailus@...ux.intel.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        linux-kernel@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH v1 0/7] ACPI support for xgmac_mdio and dpaa2-mac drivers.

On 1/31/20 7:34 AM, Calvin Johnson wrote:
> From: Calvin Johnson <calvin.johnson@....nxp.com>
> 
> This patch series provides ACPI support for xgmac_mdio and dpaa2-mac
> driver. Most of the DT APIs are replaced with fwnode APIs to handle
> both DT and ACPI nodes.
> 
> Old patch by Marcin Wojtas: (mdio_bus: Introduce fwnode MDIO helpers),
> is reused in this series to get some fwnode mdio helper APIs.

Andrew's comment on your first patch is a good summary of what this
patch series does, instead of consolidating the existing code and making
it less of_* centric and more firmware agnostic, this duplicates the
existing infrastructure almost line for line to create a fwnode specific
implementation. The preference would be for you to move away from that
and use device_* properties as much as possible while making the code
capable of handling all firmware implementations.

Can you also show a few DSDT for the devices that you are working so we
can a feeling of how you represented the various properties and
parent/child devices dependencies?

> 
> 
> Calvin Johnson (6):
>   mdio_bus: modify fwnode phy related functions
>   net/fsl: add ACPI support for mdio bus
>   device property: fwnode_get_phy_mode: Change API to solve int/unit
>     warnings
>   device property: Introduce fwnode_phy_is_fixed_link()
>   net: phylink: Introduce phylink_fwnode_phy_connect()
>   dpaa2-eth: Add ACPI support for DPAA2 MAC driver
> 
> Marcin Wojtas (1):
>   mdio_bus: Introduce fwnode MDIO helpers
> 
>  drivers/base/property.c                       |  43 ++-
>  .../net/ethernet/freescale/dpaa2/dpaa2-mac.c  |  78 ++++--
>  drivers/net/ethernet/freescale/xgmac_mdio.c   |  63 +++--
>  .../net/ethernet/marvell/mvpp2/mvpp2_main.c   |   7 +-
>  drivers/net/phy/mdio_bus.c                    | 244 ++++++++++++++++++
>  drivers/net/phy/phylink.c                     |  64 +++++
>  include/linux/mdio.h                          |   3 +
>  include/linux/phylink.h                       |   2 +
>  include/linux/property.h                      |   5 +-
>  9 files changed, 450 insertions(+), 59 deletions(-)
> 


-- 
Florian