Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Received-SPF: Fail (protection.outlook.com: domain of arri.de does not
 designate 217.111.95.7 as permitted sender) receiver=protection.outlook.com;
 client-ip=217.111.95.7; helo=mta.arri.de;
From:   Christian Eggers <ceggers@arri.de>
To:     <linux-kernel@vger.kernel.org>, <netdev@vger.kernel.org>,
        Arun Ramadoss <arun.ramadoss@microchip.com>
CC:     <woojung.huh@microchip.com>, <UNGLinuxDriver@microchip.com>,
        <andrew@lunn.ch>, <vivien.didelot@gmail.com>,
        <f.fainelli@gmail.com>, <olteanv@gmail.com>, <davem@davemloft.net>,
        <edumazet@google.com>, <kuba@kernel.org>, <pabeni@redhat.com>,
        <linux@armlinux.org.uk>, <Tristram.Ha@microchip.com>,
        <richardcochran@gmail.com>
Subject: Re: [Patch net-next v4 00/13] net: dsa: microchip: add PTP support for KSZ9563/KSZ8563 and LAN937x
Date:   Mon, 12 Dec 2022 16:48:29 +0100
Message-ID: <5890893.lOV4Wx5bFT@n95hx1g2>
Organization: Arnold & Richter Cine Technik GmbH & Co. Betriebs KG
In-Reply-To: <20221212102639.24415-1-arun.ramadoss@microchip.com>
References: <20221212102639.24415-1-arun.ramadoss@microchip.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Dec 2022 15:48:30.8467
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: c76250da-e3a3-43dc-fb42-08dadc585184
X-MS-Exchange-CrossTenant-Id: e6a73a5a-614d-4c51-b3e3-53b660a9433a
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=e6a73a5a-614d-4c51-b3e3-53b660a9433a;Ip=[217.111.95.7];Helo=[mta.arri.de]
X-MS-Exchange-CrossTenant-AuthSource: DB5EUR02FT029.eop-EUR02.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR07MB8909
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Monday, 12 December 2022, 11:26:26 CET, Arun Ramadoss wrote:
> KSZ9563/KSZ8563 and  LAN937x switch are capable for supporting IEEE 1588 PTP
> protocol.  LAN937x has the same PTP register set similar to KSZ9563, hence the
> implementation has been made common for the KSZ switches.  KSZ9563 does not
> support two step timestamping but LAN937x supports both.  Tested the 1step &
> 2step p2p timestamping in LAN937x and p2p1step timestamping in KSZ9563.
> 
> This patch series is based on the Christian Eggers PTP support for KSZ9563.
> Applied the Christian patch and updated as per the latest refactoring of KSZ
> series code. The features added on top are PTP packet Interrupt
> implementation based on nested handler, LAN937x two step timestamping and
> programmable per_out pins.
> 
> Link: https://www.spinics.net/lists/netdev/msg705531.html
> 
> Patch v3 -> v4
> - removed IRQF_TRIGGER_FALLING from the request_threaded_irq of ptp msg
> - addressed review comments on patch 10 periodic output
> - added sign off in patch 6 & 9
> - reverted to set PTP_1STEP bit for lan937x which is missed during v3 regression
> 
> Patch v2-> v3
> - used port_rxtstamp for reconstructing the absolute timestamp instead of
> tagger function pointer.
> - Reverted to setting of 802.1As bit.
> 
> Patch v1 -> v2
> - GPIO perout enable bit is different for LAN937x and KSZ9x. Added new patch
> for configuring LAN937x programmable pins.
> - PTP enabled in hardware based on both tx and rx timestamping of all the user
> ports.
> - Replaced setting of 802.1AS bit with P2P bit in PTP_MSG_CONF1 register.
> 
> RFC v2 -> Patch v1
> - Changed the patch author based on past patch submission
> - Changed the commit message prefix as net: dsa: microchip: ptp
> Individual patch changes are listed in correspondig commits.
> 
> RFC v1 -> v2
> - Added the p2p1step timestamping and conditional execution of 2 step for
>   LAN937x only.
> - Added the periodic output support
> 
> Arun Ramadoss (5):
>   net: dsa: microchip: ptp: add 4 bytes in tail tag when ptp enabled
>   net: dsa: microchip: ptp: enable interrupt for timestamping
>   net: dsa: microchip: ptp: add support for perout programmable pins
>   net: dsa: microchip: ptp: lan937x: add 2 step timestamping
>   net: dsa: microchip: ptp: lan937x: Enable periodic output in LED pins
> 
> Christian Eggers (8):
>   net: dsa: microchip: ptp: add the posix clock support
>   net: dsa: microchip: ptp: Initial hardware time stamping support
>   net: dsa: microchip: ptp: manipulating absolute time using ptp hw
>     clock
>   net: ptp: add helper for one-step P2P clocks
>   net: dsa: microchip: ptp: add packet reception timestamping
>   net: dsa: microchip: ptp: add packet transmission timestamping
>   net: dsa: microchip: ptp: move pdelay_rsp correction field to tail tag
>   net: dsa: microchip: ptp: add periodic output signal
> 
>  MAINTAINERS                             |    1 +
>  drivers/net/dsa/microchip/Kconfig       |   11 +
>  drivers/net/dsa/microchip/Makefile      |    5 +
>  drivers/net/dsa/microchip/ksz_common.c  |   44 +-
>  drivers/net/dsa/microchip/ksz_common.h  |   48 +
>  drivers/net/dsa/microchip/ksz_ptp.c     | 1187 +++++++++++++++++++++++
>  drivers/net/dsa/microchip/ksz_ptp.h     |   86 ++
>  drivers/net/dsa/microchip/ksz_ptp_reg.h |  142 +++
>  include/linux/dsa/ksz_common.h          |   53 +
>  include/linux/ptp_classify.h            |   71 ++
>  net/dsa/tag_ksz.c                       |  213 +++-
>  11 files changed, 1843 insertions(+), 18 deletions(-)
>  create mode 100644 drivers/net/dsa/microchip/ksz_ptp.c
>  create mode 100644 drivers/net/dsa/microchip/ksz_ptp.h
>  create mode 100644 drivers/net/dsa/microchip/ksz_ptp_reg.h
>  create mode 100644 include/linux/dsa/ksz_common.h
> 
> 
> base-commit: 6d534ee057b62ca9332b988619323ee99c7847c1
> 

For the whole series:
Tested-by: Christian Eggers <ceggers@arri.de>  # on KSZ9563 with P2P/BC/L2/1-step