On Sat, 2024-08-17 at 10:41 +0100, Martin Whitaker wrote:

EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe

When performing the port_hwtstamp_set operation, ptp_schedule_worker() will be called if hardware timestamoing is enabled on any of the ports. When using multiple ports for PTP, port_hwtstamp_set is executed for each port. When called for the first time ptp_schedule_worker() returns 0. On subsequent calls it returns 1, indicating the worker is already scheduled. Currently the ksz driver treats 1 as an error and fails to complete the port_hwtstamp_set operation, thus leaving the timestamping configuration for those ports unchanged.

This patch fixes this by ignoring the ptp_schedule_worker() return value.

Link: https://lore.kernel.org/netdev/7aae307a-35ca-4209-a850-7b2749d40f90@martin-w... Fixes: bb01ad30570b0 ("net: dsa: microchip: ptp: manipulating absolute time using ptp hw clock") Signed-off-by: Martin Whitaker foss@martin-whitaker.me.uk

Acked-by: Arun Ramadoss arun.ramadoss@microchip.com