| 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
| ||
|
Message-ID: <20241213-sparx5-lan969x-switch-driver-4-v4-0-d1a72c9c4714@microchip.com>
Date: Fri, 13 Dec 2024 14:40:59 +0100
From: Daniel Machon <daniel.machon@...rochip.com>
To: <UNGLinuxDriver@...rochip.com>, Andrew Lunn <andrew+netdev@...n.ch>,
"David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, "Lars
Povlsen" <lars.povlsen@...rochip.com>, Steen Hegelund
<Steen.Hegelund@...rochip.com>, Horatiu Vultur
<horatiu.vultur@...rochip.com>, Russell King <linux@...linux.org.uk>,
<jacob.e.keller@...el.com>, <robh@...nel.org>, <krzk+dt@...nel.org>,
<conor+dt@...nel.org>
CC: <devicetree@...r.kernel.org>, <netdev@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org>,
<robert.marko@...tura.hr>
Subject: [PATCH net-next v4 0/9] net: lan969x: add RGMII support
== Description:
This series is the fourth of a multi-part series, that prepares and adds
support for the new lan969x switch driver.
The upstreaming efforts is split into multiple series (might change a
bit as we go along):
1) Prepare the Sparx5 driver for lan969x (merged)
2) Add support for lan969x (same basic features as Sparx5
provides excl. FDMA and VCAP, merged).
3) Add lan969x VCAP functionality (merged).
--> 4) Add RGMII support.
5) Add FDMA support.
== RGMII support:
The lan969x switch device includes two RGMII port interfaces (port 28
and 29) supporting data speeds of 1 Gbps, 100 Mbps and 10 Mbps.
== Patch breakdown:
Patch #1 does some preparation work.
Patch #2 adds new function: is_port_rgmii() to the match data ops.
Patch #3 uses the is_port_rgmii() in a number of places.
Patch #4 makes sure that we do not configure an RGMII device as a
low-speed device, when doing a port config.
Patch #5 makes sure we only return the PCS if the port mode requires
it.
Patch #6 adds checks for RGMII PHY modes in sparx5_verify_speeds().
Patch #7 adds registers required to configure RGMII.
Patch #8 adds RGMII implementation.
Patch #9 documents RGMII delays in the dt-bindings.
Details are in the commit description of the individual patches
To: UNGLinuxDriver@...rochip.com
To: Andrew Lunn <andrew+netdev@...n.ch>
To: David S. Miller <davem@...emloft.net>
To: Eric Dumazet <edumazet@...gle.com>
To: Jakub Kicinski <kuba@...nel.org>
To: Paolo Abeni <pabeni@...hat.com>
To: Lars Povlsen <lars.povlsen@...rochip.com>
To: Steen Hegelund <Steen.Hegelund@...rochip.com>
To: Horatiu Vultur <horatiu.vultur@...rochip.com>
To: Russell King <linux@...linux.org.uk>
To: jacob.e.keller@...el.com
To: robh@...nel.org
To: krzk+dt@...nel.org
To: conor+dt@...nel.org
Cc: devicetree@...r.kernel.org
Cc: netdev@...r.kernel.org
Cc: linux-kernel@...r.kernel.org
Cc: linux-arm-kernel@...ts.infradead.org
Cc: robert.marko@...tura.hr
Signed-off-by: Daniel Machon <daniel.machon@...rochip.com>
---
Changes in v4:
- Split patch #4 in v3 into two patches, where the new patch #5 handles
PCS selection, by returning the PCS only for ports that require it.
- Got rid of the '|' symbol for {rx,tx}-internal-delay-ps property
description in the dt-bindings (patch #9).
- Link to v3: https://lore.kernel.org/r/20241118-sparx5-lan969x-switch-driver-4-v3-0-3cefee5e7e3a@microchip.com
Changes in v3:
v2 was kindly tested by Robert Marko. Not carrying the tag to v3 since
we have changes to the handling of the delays.
- Modified lan969x_rgmii_delay_config() to not apply any MAC delay when
the {rx,tx}-internal-delay-ps properties are missing or set to 0
(patch #7).
- Removed 'required' constraint from {rx-tx}-internal-delay-ps
properties. Also added description and default value (Patch #8).
- Link to v2: https://lore.kernel.org/r/20241113-sparx5-lan969x-switch-driver-4-v2-0-0db98ac096d1@microchip.com
Changes in v2:
Most changes are in patch #7. RGMII implementation has been moved to
it's own file lan969x_rgmii.c.
Details:
- Use ETH_P_8021Q and ETH_P_8021AD instead of the Sparx5 provided
equivalents (patch #7).
- Configure MAC delays through "{rx,tx}-internal-delay-ps"
properties (patch #7).
- Add selectors for all the phase shifts that the hardware supports
(instead of only 2.0 ns, patch #7).
- Add selectors for all the port speeds (instead of only 1000 mbps.)
- Document RGMII delays in dt-bindings.
- Link to v1: https://lore.kernel.org/r/20241106-sparx5-lan969x-switch-driver-4-v1-0-f7f7316436bd@microchip.com
---
Daniel Machon (9):
net: sparx5: do some preparation work
net: sparx5: add function for RGMII port check
net: sparx5: use is_port_rgmii() throughout
net: sparx5: skip low-speed configuration when port is RGMII
net: sparx5: only return PCS for modes that require it
net: sparx5: verify RGMII speeds
net: lan969x: add RGMII registers
net: lan969x: add RGMII implementation
dt-bindings: net: sparx5: document RGMII delays
.../bindings/net/microchip,sparx5-switch.yaml | 18 ++
drivers/net/ethernet/microchip/sparx5/Makefile | 3 +-
.../ethernet/microchip/sparx5/lan969x/lan969x.c | 5 +
.../ethernet/microchip/sparx5/lan969x/lan969x.h | 10 +
.../microchip/sparx5/lan969x/lan969x_rgmii.c | 224 +++++++++++++++++++++
.../net/ethernet/microchip/sparx5/sparx5_main.c | 29 ++-
.../net/ethernet/microchip/sparx5/sparx5_main.h | 3 +
.../ethernet/microchip/sparx5/sparx5_main_regs.h | 145 +++++++++++++
.../net/ethernet/microchip/sparx5/sparx5_phylink.c | 14 +-
.../net/ethernet/microchip/sparx5/sparx5_port.c | 57 ++++--
.../net/ethernet/microchip/sparx5/sparx5_port.h | 5 +
11 files changed, 484 insertions(+), 29 deletions(-)
---
base-commit: 2c27c7663390d28bc71e97500eb68e0ce2a7223f
change-id: 20241104-sparx5-lan969x-switch-driver-4-d59b7820485a
Best regards,
--
Daniel Machon <daniel.machon@...rochip.com>
Powered by blists - more mailing lists