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@xxxxxxxxxxxxxxxxxxxxx/ > Fixes: bb01ad30570b0 ("net: dsa: microchip: ptp: manipulating > absolute time using ptp hw clock") > Signed-off-by: Martin Whitaker <foss@xxxxxxxxxxxxxxxxxxxxx> Acked-by: Arun Ramadoss <arun.ramadoss@xxxxxxxxxxxxx>
