net: stmmac: fix taprio configuration when base_time is in the past
authorYannick Vignon <yannick.vignon@nxp.com>
Wed, 13 Jan 2021 13:15:57 +0000 (14:15 +0100)
committerJakub Kicinski <kuba@kernel.org>
Thu, 14 Jan 2021 18:49:42 +0000 (10:49 -0800)
commitfe28c53ed71d463e187748b6b10e1130dd72ceeb
tree5ed1133850952b08b86d5c150fa727ba93722005
parentb76889ff51bfee318bea15891420e5aefd2833a0
net: stmmac: fix taprio configuration when base_time is in the past

The Synopsys TSN MAC supports Qbv base times in the past, but only up to a
certain limit. As a result, a taprio qdisc configuration with a small
base time (for example when treating the base time as a simple phase
offset) is not applied by the hardware and silently ignored.

This was observed on an NXP i.MX8MPlus device, but likely affects all
TSN-variants of the MAC.

Fix the issue by making sure the base time is in the future, pushing it by
an integer amount of cycle times if needed. (a similar check is already
done in several other taprio implementations, see for example
drivers/net/ethernet/intel/igc/igc_tsn.c#L116 or
drivers/net/dsa/sja1105/sja1105_ptp.h#L39).

Fixes: b60189e0392f ("net: stmmac: Integrate EST with TAPRIO scheduler API")
Signed-off-by: Yannick Vignon <yannick.vignon@nxp.com>
Link: https://lore.kernel.org/r/20210113131557.24651-2-yannick.vignon@oss.nxp.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/stmicro/stmmac/stmmac_tc.c