Skip to content

Commit 1ce2654

Browse files
Furong XuPaolo Abeni
Furong Xu
authored and
Paolo Abeni
committed
net: stmmac: xgmac: fix a typo of register name in DPP safety handling
DDPP is copied from Synopsys Data book: DDPP: Disable Data path Parity Protection. When it is 0x0, Data path Parity Protection is enabled. When it is 0x1, Data path Parity Protection is disabled. The macro name should be XGMAC_DPP_DISABLE. Fixes: 46eba19 ("net: stmmac: xgmac: fix handling of DPP safety error for DMA channels") Signed-off-by: Furong Xu <[email protected]> Reviewed-by: Serge Semin <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Paolo Abeni <[email protected]>
1 parent b083d24 commit 1ce2654

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

drivers/net/ethernet/stmicro/stmmac/dwxgmac2.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,7 @@
304304
#define XGMAC_TXCEIE BIT(0)
305305
#define XGMAC_MTL_ECC_INT_STATUS 0x000010cc
306306
#define XGMAC_MTL_DPP_CONTROL 0x000010e0
307-
#define XGMAC_DDPP_DISABLE BIT(0)
307+
#define XGMAC_DPP_DISABLE BIT(0)
308308
#define XGMAC_MTL_TXQ_OPMODE(x) (0x00001100 + (0x80 * (x)))
309309
#define XGMAC_TQS GENMASK(25, 16)
310310
#define XGMAC_TQS_SHIFT 16

drivers/net/ethernet/stmicro/stmmac/dwxgmac2_core.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -928,7 +928,7 @@ dwxgmac3_safety_feat_config(void __iomem *ioaddr, unsigned int asp,
928928
/* 5. Enable Data Path Parity Protection */
929929
value = readl(ioaddr + XGMAC_MTL_DPP_CONTROL);
930930
/* already enabled by default, explicit enable it again */
931-
value &= ~XGMAC_DDPP_DISABLE;
931+
value &= ~XGMAC_DPP_DISABLE;
932932
writel(value, ioaddr + XGMAC_MTL_DPP_CONTROL);
933933

934934
return 0;

0 commit comments

Comments
 (0)