[PATCH v2 0/3] net: mvpp2: tai: add extts support

Shmuel Hazan posted 3 patches 2 years, 8 months ago
There is a newer version of this series
.../devicetree/bindings/net/marvell,pp2.yaml  |  18 +
.../net/ethernet/marvell/mvpp2/mvpp2_tai.c    | 334 ++++++++++++++++--
2 files changed, 317 insertions(+), 35 deletions(-)
[PATCH v2 0/3] net: mvpp2: tai: add extts support
Posted by Shmuel Hazan 2 years, 8 months ago
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.

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
Re: [PATCH v2 0/3] net: mvpp2: tai: add extts support
Posted by Andrew Lunn 2 years, 8 months ago
On Mon, Apr 17, 2023 at 08:07:38PM +0300, Shmuel Hazan wrote:
> 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.

The other think the NETDEV FAQ says is to wait at least 24 hours
before posting a new version. That give people time to actually review
the code, and likely the current version, not an old version.

    Andrew
Re: [PATCH v2 0/3] net: mvpp2: tai: add extts support
Posted by Shmuel Hazan 2 years, 8 months ago
On Tue, 2023-04-18 at 02:36 +0200, Andrew Lunn wrote:
> Caution: This is an external email. Please take care when clicking links or opening attachments.
> 
> 
> On Mon, Apr 17, 2023 at 08:07:38PM +0300, Shmuel Hazan wrote:
> > 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.
> 
> The other think the NETDEV FAQ says is to wait at least 24 hours
> before posting a new version. That give people time to actually review
> the code, and likely the current version, not an old version.
> 

Hi Andrew,

Sorry about it. Thanks for letting me know. 

>     Andrew

Re: [PATCH v2 0/3] net: mvpp2: tai: add extts support
Posted by Jakub Kicinski 2 years, 8 months ago
On Mon, 17 Apr 2023 20:07:38 +0300 Shmuel Hazan wrote:
> 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.

Please wait at least 24h between resends.
Please read the rules:
https://www.kernel.org/doc/html/next/process/maintainer-netdev.html
Re: [PATCH v2 0/3] net: mvpp2: tai: add extts support
Posted by Shmuel Hazan 2 years, 8 months ago
On Mon, 2023-04-17 at 12:01 -0700, Jakub Kicinski wrote:
> Caution: This is an external email. Please take care when clicking links or opening attachments.
> 
> 
> On Mon, 17 Apr 2023 20:07:38 +0300 Shmuel Hazan wrote:
> > 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.
> 
> Please wait at least 24h between resends.
> Please read the rules:
> https://www.kernel.org/doc/html/next/process/maintainer-netdev.html

Hi Jakub,

Sorry about it. Thanks for letting me know and linking the rules. 

-- Shmuel.