2023-04-19 15:30:29

by Shmuel Hazan

[permalink] [raw]
Subject: [PATCH v3 0/3] net: mvpp2: tai: add extts support

This patch series adds support for PTP event capture on the Aramda
80x0/70x0. This feature is mainly used by tools linux ts2phc(3) in order
to synchronize a timestamping unit (like the mvpp2's TAI) and a system
DPLL on the same PCB.

The patch series includes 3 patches: the second one implements the
actual extts function.

Changes in v2:
* Fixed a deadlock in the poll worker.
* Removed tabs from comments.

Changes in v3:
* Added more explanation about the change in behavior in mvpp22_tai_start.
* Explain the reason for choosing 95ms as a polling rate.

Shmuel Hazan (3):
net: mvpp2: tai: add refcount for ptp worker
net: mvpp2: tai: add extts support
dt-bindings: net: marvell,pp2: add extts docs

.../devicetree/bindings/net/marvell,pp2.yaml | 18 +
.../net/ethernet/marvell/mvpp2/mvpp2_tai.c | 334 ++++++++++++++++--
2 files changed, 317 insertions(+), 35 deletions(-)


base-commit: 3e7bb4f2461710b70887704af7f175383251088e
--
2.40.0


2023-04-19 15:30:53

by Shmuel Hazan

[permalink] [raw]
Subject: [PATCH v3 3/3] dt-bindings: net: marvell,pp2: add extts docs

Add some documentation and example for enabling extts on the marvell
mvpp2 TAI.

Signed-off-by: Shmuel Hazan <[email protected]>
---
.../devicetree/bindings/net/marvell,pp2.yaml | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/marvell,pp2.yaml b/Documentation/devicetree/bindings/net/marvell,pp2.yaml
index 4eadafc43d4f..5e4fc9c5dc92 100644
--- a/Documentation/devicetree/bindings/net/marvell,pp2.yaml
+++ b/Documentation/devicetree/bindings/net/marvell,pp2.yaml
@@ -31,6 +31,21 @@ properties:
"#size-cells":
const: 0

+ pinctrl-0: true
+ pinctrl-1: true
+
+ pinctrl-names:
+ description:
+ When present, must have one state named "default",
+ and may contain a second name named "extts". The former
+ state sets up pins for ordinary operation without extts
+ support whereas the latter state will enable receiving
+ external timestamp events.
+ minItems: 1
+ items:
+ - const: default
+ - const: extts
+
clocks:
minItems: 2
items:
@@ -241,6 +256,9 @@ examples:
<&cp0_clk 1 5>, <&cp0_clk 1 6>, <&cp0_clk 1 18>;
clock-names = "pp_clk", "gop_clk", "mg_clk", "mg_core_clk", "axi_clk";
marvell,system-controller = <&cp0_syscon0>;
+ pinctrl-names = "default", "extts";
+ pinctrl-0 = <&cp1_mpp6_gpio>;
+ pinctrl-1 = <&cp1_mpp6_ptp>;

ethernet-port@0 {
interrupts = <ICU_GRP_NSR 39 IRQ_TYPE_LEVEL_HIGH>,
--
2.40.0

2023-04-21 07:51:01

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v3 3/3] dt-bindings: net: marvell,pp2: add extts docs

On 19/04/2023 17:14, Shmuel Hazan wrote:
> Add some documentation and example for enabling extts on the marvell
> mvpp2 TAI.
>
> Signed-off-by: Shmuel Hazan <[email protected]>
> ---


Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof