net/igc: fix disabling timesync
Checks
Commit Message
When disabling timesync, we should clear the IGC_RXPBS_CFG_TS_EN bit
of IGC_RXPBS, the patch fixes this.
Fixes: 4f6fbbf6f17d ("net/igc: support IEEE 1588 PTP")
Cc: stable@dpdk.org
Signed-off-by: Wenwu Ma <wenwux.ma@intel.com>
---
drivers/net/igc/igc_ethdev.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
Tested-by: Tingting Liao <tingtingx.liao@intel.com>
________________________________
> -----Original Message-----
> From: Ma, WenwuX <wenwux.ma@intel.com>
> Sent: Friday, March 15, 2024 09:06
> To: dev@dpdk.org <dev@dpdk.org>; Guo, Junfeng <junfeng.guo@intel.com>; Su, Simei <simei.su@intel.com>
> Cc: Liao, TingtingX <tingtingx.liao@intel.com>; Ma, WenwuX <wenwux.ma@intel.com>; stable@dpdk.org <stable@dpdk.org>
> Subject: [PATCH] net/igc: fix disabling timesync
>
> When disabling timesync, we should clear the IGC_RXPBS_CFG_TS_EN bit
> of IGC_RXPBS, the patch fixes this.
>
> Fixes: 4f6fbbf6f17d ("net/igc: support IEEE 1588 PTP")
> Cc: stable@dpdk.org
>
> Signed-off-by: Wenwu Ma <wenwux.ma@intel.com>
> ---
On Mon, Mar 18, 2024 at 04:11:14AM +0000, Liao, TingtingX wrote:
> Tested-by: Tingting Liao <tingtingx.liao@intel.com>
> __________________________________________________________________
>
> > -----Original Message-----
>
> > From: Ma, WenwuX <wenwux.ma@intel.com>
>
> > Sent: Friday, March 15, 2024 09:06
>
> > To: dev@dpdk.org <dev@dpdk.org>; Guo, Junfeng
> <junfeng.guo@intel.com>; Su, Simei <simei.su@intel.com>
>
> > Cc: Liao, TingtingX <tingtingx.liao@intel.com>; Ma, WenwuX
> <wenwux.ma@intel.com>; stable@dpdk.org <stable@dpdk.org>
>
> > Subject: [PATCH] net/igc: fix disabling timesync
>
> >
>
> > When disabling timesync, we should clear the IGC_RXPBS_CFG_TS_EN bit
>
> > of IGC_RXPBS, the patch fixes this.
>
> >
>
> > Fixes: 4f6fbbf6f17d ("net/igc: support IEEE 1588 PTP")
>
> > Cc: stable@dpdk.org
>
> >
>
> > Signed-off-by: Wenwu Ma <wenwux.ma@intel.com>
>
This seems a small enough, low-risk fix, so applying to
dpdk-next-net-intel.
Thanks,
/Bruce
@@ -2853,7 +2853,7 @@ eth_igc_timesync_disable(struct rte_eth_dev *dev)
IGC_WRITE_REG(hw, IGC_TSYNCRXCTL, 0);
val = IGC_READ_REG(hw, IGC_RXPBS);
- val &= IGC_RXPBS_CFG_TS_EN;
+ val &= ~IGC_RXPBS_CFG_TS_EN;
IGC_WRITE_REG(hw, IGC_RXPBS, val);
val = IGC_READ_REG(hw, IGC_SRRCTL(0));