netdev - Re: [PATCH net-next v2 0/8] net: phy: move PHY package code to its own source file

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: <6cb88f4b-15c9-4b6c-8226-63442b79492c@gmail.com>
Date: Sat, 1 Mar 2025 12:16:23 +0100
From: Heiner Kallweit <hkallweit1@...il.com>
To: Andrew Lunn <andrew@...n.ch>,
 Russell King - ARM Linux <linux@...linux.org.uk>,
 Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
 Eric Dumazet <edumazet@...gle.com>, David Miller <davem@...emloft.net>,
 Daniel Golle <daniel@...rotopia.org>, Qingfang Deng <dqfext@...il.com>,
 SkyLake Huang <SkyLake.Huang@...iatek.com>,
 Matthias Brugger <matthias.bgg@...il.com>,
 AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
 Richard Cochran <richardcochran@...il.com>
Cc: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
 "linux-arm-kernel@...ts.infradead.org"
 <linux-arm-kernel@...ts.infradead.org>, linux-mediatek@...ts.infradead.org,
 linux-arm-msm@...r.kernel.org, Robert Marko <robimarko@...il.com>,
 Köry Maincent <kory.maincent@...tlin.com>
Subject: Re: [PATCH net-next v2 0/8] net: phy: move PHY package code to its
 own source file

On 28.02.2025 22:44, Heiner Kallweit wrote:
> This series contributes to cleaning up phylib by moving PHY package
> related code to its own source file.
> 
> v2:
> - rename the getters
> - add a new header file phylib.h, which is used by PHY drivers only
> 
> Heiner Kallweit (8):
>   net: phy: move PHY package code from phy_device.c to own source file
>   net: phy: add getters for public members in struct phy_package_shared
>   net: phy: qca807x: use new phy_package_shared getters
>   net: phy: micrel: use new phy_package_shared getters
>   net: phy: mediatek: use new phy_package_shared getters
>   net: phy: mscc: use new phy_package_shared getters
>   net: phy: move PHY package related code from phy.h to phy_package.c
>   net: phy: remove remaining PHY package related definitions from phy.h
> 
>  drivers/net/phy/Makefile              |   3 +-
>  drivers/net/phy/mediatek/mtk-ge-soc.c |   7 +-
>  drivers/net/phy/micrel.c              |   9 +-
>  drivers/net/phy/mscc/mscc_main.c      |   2 +
>  drivers/net/phy/mscc/mscc_ptp.c       |  14 +-
>  drivers/net/phy/phy-core.c            |   1 +
>  drivers/net/phy/phy_device.c          | 237 -----------------
>  drivers/net/phy/phy_package.c         | 350 ++++++++++++++++++++++++++
>  drivers/net/phy/phylib-internal.h     |   2 +
>  drivers/net/phy/phylib.h              |  28 +++
>  drivers/net/phy/qcom/qca807x.c        |  16 +-
>  include/linux/phy.h                   | 124 ---------
>  12 files changed, 409 insertions(+), 384 deletions(-)
>  create mode 100644 drivers/net/phy/phy_package.c
>  create mode 100644 drivers/net/phy/phylib.h
> 

CI is complaining about one issue in patch 7 which I have to correct.
Will wait for other feedback before resubmitting.
--
pw-bot: cr