Thread (32 messages) 32 messages, 5 authors, 2020-11-30

Re: [PATCH net-next v3 00/12] net: dsa: microchip: PTP support for KSZ956x

From: Christian Eggers <ceggers@arri.de>
Date: 2020-11-19 05:30:54
Also in: linux-devicetree, lkml

Hi Vladimir,

On Thursday, 19 November 2020, 00:40:18 CET, Vladimir Oltean wrote:
On Wed, Nov 18, 2020 at 09:30:01PM +0100, Christian Eggers wrote:
quoted
This series adds support for PTP to the KSZ956x and KSZ9477 devices.

There is only little documentation for PTP available on the data sheet
[1] (more or less only the register reference). Questions to the
Microchip support were seldom answered comprehensively or in reasonable
time. So this is more or less the result of reverse engineering.
[...]
One thing that should definitely not be part of this series though is
patch 11/12. Christian, given the conversation we had on your previous
patch:
https://lore.kernel.org/netdev/20201113025311.jpkplhmacjz6lkc5@skbuf/ (local)
sorry, I didn't read that carefully enough. Some of the other requested changes
were quite challenging for me. Additionally, finding the UDP checksum bug
needed some time for identifying because I didn't recognize that when it got
introduced.
as well as the documentation patch that was submitted in the meantime:
https://lore.kernel.org/netdev/20201117213826.18235-1-a.fatoum@pengutronix.de/ (local) 
I am not subscribed to the list. 
obviously you chose to completely disregard that. May we know why? How
are you even making use of the PTP_CLK_REQ_PPS feature?
Of course I will drop that patch from the next series.

regards
Christian


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help