netdev - Re: [PATCH net-next 0/3] net: phy: add new version of phy_do

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]

Date:   Mon, 20 Jan 2020 14:05:38 -0800
From:   Florian Fainelli <f.fainelli@...il.com>
To:     Heiner Kallweit <hkallweit1@...il.com>,
        Andrew Lunn <andrew@...n.ch>,
        David Miller <davem@...emloft.net>,
        Mark Einon <mark.einon@...il.com>,
        Jay Cliburn <jcliburn@...il.com>,
        Chris Snook <chris.snook@...il.com>
Cc:     "netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: Re: [PATCH net-next 0/3] net: phy: add new version of phy_do_ioctl
 and convert suitable drivers

On 1/20/20 1:15 PM, Heiner Kallweit wrote:
> We just added phy_do_ioctl, but it turned out that we need another
> version of this function that doesn't check whether net_device is
> running. So rename phy_do_ioctl to phy_do_ioctl_running and add a
> new version of phy_do_ioctl. Eventually convert suitable drivers
> to use phy_do_ioctl.
> 
> Heiner Kallweit (3):
>   net: phy: rename phy_do_ioctl to phy_do_ioctl_running
>   net: phy: add new version of phy_do_ioctl
>   net: convert suitable network drivers to use phy_do_ioctl
> 
>  drivers/net/ethernet/agere/et131x.c          | 11 +----------
>  drivers/net/ethernet/atheros/ag71xx.c        | 10 +---------
>  drivers/net/ethernet/faraday/ftgmac100.c     | 11 +----------
>  drivers/net/ethernet/freescale/fec_mpc52xx.c | 12 +-----------
>  drivers/net/ethernet/rdc/r6040.c             | 10 +---------
>  drivers/net/ethernet/realtek/r8169_main.c    |  2 +-
>  drivers/net/phy/phy.c                        | 12 +++++++++++-
>  include/linux/phy.h                          |  1 +
>  8 files changed, 18 insertions(+), 51 deletions(-)

Now the diffstat is convincing :) Thanks!
-- 
Florian