[net-next,v3,00/10] net: freescale: ucc_geth: Phylink conversion

Message

Maxime Chevallier Dec. 3, 2024, 12:43 p.m. UTC

Hello everyone,

This is V3 of the phylink conversion for ucc_geth.

The main changes in this V3 are related to error handling in the patches
1 and 10 to report an error when the deprecated "interface" property is
found in DT. Doing so, I found and addressed some issues with the jump
labels in the error paths, impacting patches 1 and 10.

The rest of the changes are just a rebase on net-next.

Some of the V2 changes haven't been reviewed, so I stress out that I'm
still uncertain about the way WoL is handled is patches 4 and 10.

Thanks,

Maxime

Link to V1: https://lore.kernel.org/netdev/[e-mail]/
Link to V2: https://lore.kernel.org/netdev/[e-mail]/

Maxime Chevallier (10):
  net: freescale: ucc_geth: Drop support for the "interface" DT property
  net: freescale: ucc_geth: split adjust_link for phylink conversion
  net: freescale: ucc_geth: Use netdev->phydev to access the PHY
  net: freescale: ucc_geth: Fix WOL configuration
  net: freescale: ucc_geth: Use the correct type to store WoL opts
  net: freescale: ucc_geth: Simplify frame length check
  net: freescale: ucc_geth: Hardcode the preamble length to 7 bytes
  net: freescale: ucc_geth: Move the serdes configuration around
  net: freescale: ucc_geth: Introduce a helper to check Reduced modes
  net: freescale: ucc_geth: phylink conversion

 drivers/net/ethernet/freescale/Kconfig        |   3 +-
 drivers/net/ethernet/freescale/ucc_geth.c     | 602 +++++++-----------
 drivers/net/ethernet/freescale/ucc_geth.h     |  22 +-
 .../net/ethernet/freescale/ucc_geth_ethtool.c |  74 +--
 4 files changed, 266 insertions(+), 435 deletions(-)

Comments

[e-mail] Dec. 6, 2024, 1:50 p.m. UTC | #1

Hello:

This series was applied to netdev/net-next.git (main)
by David S. Miller <[e-mail]>:

On Tue,  3 Dec 2024 13:43:11 +0100 you wrote:
> Hello everyone,
> 
> This is V3 of the phylink conversion for ucc_geth.
> 
> The main changes in this V3 are related to error handling in the patches
> 1 and 10 to report an error when the deprecated "interface" property is
> found in DT. Doing so, I found and addressed some issues with the jump
> labels in the error paths, impacting patches 1 and 10.
> 
> [...]

Here is the summary with links:
  - [net-next,v3,01/10] net: freescale: ucc_geth: Drop support for the "interface" DT property
    https://git.kernel.org/netdev/net-next/c/3e42bb998c6d
  - [net-next,v3,02/10] net: freescale: ucc_geth: split adjust_link for phylink conversion
    https://git.kernel.org/netdev/net-next/c/1e59fd163100
  - [net-next,v3,03/10] net: freescale: ucc_geth: Use netdev->phydev to access the PHY
    https://git.kernel.org/netdev/net-next/c/43068024cc2a
  - [net-next,v3,04/10] net: freescale: ucc_geth: Fix WOL configuration
    https://git.kernel.org/netdev/net-next/c/d2adc441a19a
  - [net-next,v3,05/10] net: freescale: ucc_geth: Use the correct type to store WoL opts
    https://git.kernel.org/netdev/net-next/c/420d56e4de52
  - [net-next,v3,06/10] net: freescale: ucc_geth: Simplify frame length check
    https://git.kernel.org/netdev/net-next/c/270ec339126a
  - [net-next,v3,07/10] net: freescale: ucc_geth: Hardcode the preamble length to 7 bytes
    https://git.kernel.org/netdev/net-next/c/dba25f75383f
  - [net-next,v3,08/10] net: freescale: ucc_geth: Move the serdes configuration around
    https://git.kernel.org/netdev/net-next/c/efc52055b756
  - [net-next,v3,09/10] net: freescale: ucc_geth: Introduce a helper to check Reduced modes
    https://git.kernel.org/netdev/net-next/c/02d4a6498b30
  - [net-next,v3,10/10] net: freescale: ucc_geth: phylink conversion
    https://git.kernel.org/netdev/net-next/c/53036aa8d031

You are awesome, thank you!