mbox series

[net-next,v3,0/8] net: ethernet: fs_enet: Cleanup and phylink conversion

Message ID 20240904171822.64652-1-maxime.chevallier@bootlin.com (mailing list archive)
Headers show
Series net: ethernet: fs_enet: Cleanup and phylink conversion | expand

Message

Maxime Chevallier Sept. 4, 2024, 5:18 p.m. UTC
Hi everyone,

This is V3 of a series that cleans-up fs_enet, with the ultimate goal of
converting it to phylink (patch 8).

The main changes compared to V2 are :
 - Reviewed-by tags from Andrew were gathered
 - Patch 5 now includes the removal of now unused includes, thanks
   Andrew for spotting this
 - Patch 4 is new, it reworks the adjust_link to move the spinlock
   acquisition to a more suitable location. Although this dissapears in
   the actual phylink port, it makes the phylink conversion clearer on
   that point
 - Patch 8 includes fixes in the tx_timeout cancellation, to prevent
   taking rtnl twice when canceling a pending tx_timeout. Thanks Jakub
   for spotting this.

Link to V2: https://lore.kernel.org/netdev/20240829161531.610874-1-maxime.chevallier@bootlin.com/
Link to V1: https://lore.kernel.org/netdev/20240828095103.132625-1-maxime.chevallier@bootlin.com/

Maxime Chevallier (8):
  net: ethernet: fs_enet: convert to SPDX
  net: ethernet: fs_enet: cosmetic cleanups
  net: ethernet: fs_enet: drop the .adjust_link custom fs_ops
  net: ethernet: fs_enet: only protect the .restart() call in
    .adjust_link
  net: ethernet: fs_enet: drop unused phy_info and mii_if_info
  net: ethernet: fs_enet: use macros for speed and duplex values
  net: ethernet: fs_enet: simplify clock handling with devm accessors
  net: ethernet: fs_enet: phylink conversion

 .../net/ethernet/freescale/fs_enet/Kconfig    |   2 +-
 .../ethernet/freescale/fs_enet/fs_enet-main.c | 444 ++++++++----------
 .../net/ethernet/freescale/fs_enet/fs_enet.h  |  27 +-
 .../net/ethernet/freescale/fs_enet/mac-fcc.c  |  17 +-
 .../net/ethernet/freescale/fs_enet/mac-fec.c  |  15 +-
 .../net/ethernet/freescale/fs_enet/mac-scc.c  |  11 +-
 .../ethernet/freescale/fs_enet/mii-bitbang.c  |   5 +-
 .../net/ethernet/freescale/fs_enet/mii-fec.c  |   5 +-
 8 files changed, 219 insertions(+), 307 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Sept. 9, 2024, 9:30 a.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (main)
by David S. Miller <davem@davemloft.net>:

On Wed,  4 Sep 2024 19:18:13 +0200 you wrote:
> Hi everyone,
> 
> This is V3 of a series that cleans-up fs_enet, with the ultimate goal of
> converting it to phylink (patch 8).
> 
> The main changes compared to V2 are :
>  - Reviewed-by tags from Andrew were gathered
>  - Patch 5 now includes the removal of now unused includes, thanks
>    Andrew for spotting this
>  - Patch 4 is new, it reworks the adjust_link to move the spinlock
>    acquisition to a more suitable location. Although this dissapears in
>    the actual phylink port, it makes the phylink conversion clearer on
>    that point
>  - Patch 8 includes fixes in the tx_timeout cancellation, to prevent
>    taking rtnl twice when canceling a pending tx_timeout. Thanks Jakub
>    for spotting this.
> 
> [...]

Here is the summary with links:
  - [net-next,v3,1/8] net: ethernet: fs_enet: convert to SPDX
    https://git.kernel.org/netdev/net-next/c/30ba6d2f3463
  - [net-next,v3,2/8] net: ethernet: fs_enet: cosmetic cleanups
    https://git.kernel.org/netdev/net-next/c/2b29ac68e786
  - [net-next,v3,3/8] net: ethernet: fs_enet: drop the .adjust_link custom fs_ops
    https://git.kernel.org/netdev/net-next/c/96bf0c4e9f48
  - [net-next,v3,4/8] net: ethernet: fs_enet: only protect the .restart() call in .adjust_link
    https://git.kernel.org/netdev/net-next/c/aa3672be731d
  - [net-next,v3,5/8] net: ethernet: fs_enet: drop unused phy_info and mii_if_info
    https://git.kernel.org/netdev/net-next/c/6b576b2d4430
  - [net-next,v3,6/8] net: ethernet: fs_enet: use macros for speed and duplex values
    https://git.kernel.org/netdev/net-next/c/21c6321459aa
  - [net-next,v3,7/8] net: ethernet: fs_enet: simplify clock handling with devm accessors
    https://git.kernel.org/netdev/net-next/c/c614acf6e8e1
  - [net-next,v3,8/8] net: ethernet: fs_enet: phylink conversion
    https://git.kernel.org/netdev/net-next/c/41f5fbffd177

You are awesome, thank you!