2023-04-27 06:36:43

by Dominic Rath

Subject: [PATCH v2 0/3] Cadence PCIe PHY latency for PTM

Hello Everyone,

this series adds PHY latency properties to the Cadence PCIe
driver to improve PTM accuracy, and configures the necessary
values for TI's AM64x processors.

These latencies are implementation specific and need to be
configured in the PCIe IP core's registers to allow the
PCIe controller to exactly determine the RX/TX timestamps for
PCIe PTM messages.

TI doesn't document these values in the datasheet or reference
manual as of now, but provided the necessary data via TI's E2E
forums (see PATCH 3/3).

Changes from v1 to v2:
- move latency property to PHY instead of PCIe controller
- drop vendor prefix from property name
- rephrase commit message regarding optional properties
- emit an info message instead of a warning in case
an optional property is missing

Best Regards,

Dominic

Alexander Bahle (3):
dt-bindings: phy: cadence-torrent: Add latency properties
PCI: cadence: Use DT bindings to set PHY latencies
arm64: dts: ti: k3-am64: Add PCIe PHY latency DT binding

.../bindings/phy/phy-cadence-torrent.yaml | 20 ++++
arch/arm64/boot/dts/ti/k3-am642-evm.dts | 2 +
.../pci/controller/cadence/pcie-cadence-ep.c | 2 +
.../controller/cadence/pcie-cadence-host.c | 1 +
drivers/pci/controller/cadence/pcie-cadence.c | 92 +++++++++++++++++++
drivers/pci/controller/cadence/pcie-cadence.h | 23 +++++
6 files changed, 140 insertions(+)


base-commit: 0cfd8703e7da687924371e9bc77a025bdeba9637
--
2.36.0

2023-05-30 08:46:23

by Christian Gmeiner

Subject: Re: [PATCH v2 0/3] Cadence PCIe PHY latency for PTM

Hi Dominic

Am Do., 27. Apr. 2023 um 07:50 Uhr schrieb Dominic Rath <[email protected]>:
>
> Hello Everyone,
>
> this series adds PHY latency properties to the Cadence PCIe
> driver to improve PTM accuracy, and configures the necessary
> values for TI's AM64x processors.
>
> These latencies are implementation specific and need to be
> configured in the PCIe IP core's registers to allow the
> PCIe controller to exactly determine the RX/TX timestamps for
> PCIe PTM messages.
>
> TI doesn't document these values in the datasheet or reference
> manual as of now, but provided the necessary data via TI's E2E
> forums (see PATCH 3/3).
>
> Changes from v1 to v2:
> - move latency property to PHY instead of PCIe controller
> - drop vendor prefix from property name
> - rephrase commit message regarding optional properties
> - emit an info message instead of a warning in case
> an optional property is missing
>
> Best Regards,
>

Hope you send out a v3 soon.

--
greets
--
Christian Gmeiner, MSc

https://christian-gmeiner.info/privacypolicy