[net-next,v6,0/3] Add support for QSGMII mode for J721e CPSW9G to am65-cpsw driver

Message

s-vadapalli Jan. 4, 2023, 10:34 a.m. UTC

Add compatible to am65-cpsw driver for J721e CPSW9G, which contains 8
external ports and 1 internal host port.

Add support to power on and power off the SERDES PHY which is used by the
CPSW MAC.

=========
Changelog
=========
v5 -> v6:
1. Add member "serdes_phy" in struct "am65_cpsw_slave_data" to store the
   SERDES PHY for each port, if present. This is done to cache the SERDES
   PHY beforehand, without depending on devm_of_phy_get().
2. Rebase series on net-next tree.

v4 -> v5:
1. Update subject of all patches in the series to "PATCH net-next".
2. Rebase series on net-next tree.

v3 -> v4:
1. Fix subject of patch-1/3, updating it to:
   "dt-bindings: net: ti: k3-am654-cpsw-nuss: Add J721e CPSW9G support"
   and collect Reviewed-by tag.
2. Rebase series on linux-next tree tagged: next-20221107.

v2 -> v3:
1. Run 'make DT_CHECKER_FLAGS=-m dt_binding_check' and fix errors and
   warnings corresponding to the patch for:
   Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml
   with the latest dt-schema and yamllint.

v1 -> v2:
1. Drop all patches corresponding to SGMII mode. This is done since I do
   not have a method to test SGMII in the standard mode which uses an
   SGMII PHY. The previous series used SGMII in a fixed-link mode,
   bypassing the SGMII PHY. I will post the SGMII patches in a future
   series after testing them.
2. Drop all patches corresponding to fixed-link in the am65-cpsw driver.
   This is done since PHYLINK takes care of fixed-link automatically and
   there is no need to deal with fixed-link in a custom fashion.
3. Fix indentation errors in k3-am65-cpsw-nuss.yaml.
4. Remove the stale code which tries to power on and power off the CPSW
   MAC's phy, since the CPSW MAC's phy driver does not support it.
5. Rename the function "am65_cpsw_init_phy()" to
   "am65_cpsw_init_serdes_phy()", to indicate that the phy corresponds to
   the SERDES.
6. Invoke "am65_cpsw_disable_serdes_phy()" as a part of the cleanup that
   is associated with the "am65_cpsw_nuss_remove()" function.

v5:
https://lore.kernel.org/r/[e-mail]/
v4:
https://lore.kernel.org/r/[e-mail]/
v3:
https://lore.kernel.org/r/[e-mail]/
v2:
https://lore.kernel.org/r/[e-mail]/
v1:
https://lore.kernel.org/r/[e-mail]/

Siddharth Vadapalli (3):
  dt-bindings: net: ti: k3-am654-cpsw-nuss: Add J721e CPSW9G support
  net: ethernet: ti: am65-cpsw: Enable QSGMII mode for J721e CPSW9G
  net: ethernet: ti: am65-cpsw: Add support for SERDES configuration

 .../bindings/net/ti,k3-am654-cpsw-nuss.yaml   | 33 +++++++-
 drivers/net/ethernet/ti/am65-cpsw-nuss.c      | 76 +++++++++++++++++++
 drivers/net/ethernet/ti/am65-cpsw-nuss.h      |  1 +
 3 files changed, 106 insertions(+), 4 deletions(-)

Comments

[e-mail] Jan. 5, 2023, 11:30 a.m. UTC | #1

Hello:

This series was applied to netdev/net-next.git (master)
by Paolo Abeni <[e-mail]>:

On Wed, 4 Jan 2023 16:04:29 +0530 you wrote:
> Add compatible to am65-cpsw driver for J721e CPSW9G, which contains 8
> external ports and 1 internal host port.
> 
> Add support to power on and power off the SERDES PHY which is used by the
> CPSW MAC.
> 
> =========
> Changelog
> =========
> v5 -> v6:
> 1. Add member "serdes_phy" in struct "am65_cpsw_slave_data" to store the
>    SERDES PHY for each port, if present. This is done to cache the SERDES
>    PHY beforehand, without depending on devm_of_phy_get().
> 2. Rebase series on net-next tree.
> 
> [...]

Here is the summary with links:
  - [net-next,v6,1/3] dt-bindings: net: ti: k3-am654-cpsw-nuss: Add J721e CPSW9G support
    https://git.kernel.org/netdev/net-next/c/c85b53e32c8e
  - [net-next,v6,2/3] net: ethernet: ti: am65-cpsw: Enable QSGMII mode for J721e CPSW9G
    https://git.kernel.org/netdev/net-next/c/944131fa65d7
  - [net-next,v6,3/3] net: ethernet: ti: am65-cpsw: Add support for SERDES configuration
    https://git.kernel.org/netdev/net-next/c/dab2b265dd23

You are awesome, thank you!