XGMAC also supports Safety Features. This patch implements the
configuration and handling of this feature in XGMAC core.

Signed-off-by: Jose Abreu <joab...@synopsys.com>
---
Cc: Giuseppe Cavallaro <peppe.cavall...@st.com>
Cc: Alexandre Torgue <alexandre.tor...@st.com>
Cc: Jose Abreu <joab...@synopsys.com>
Cc: "David S. Miller" <da...@davemloft.net>
Cc: Maxime Coquelin <mcoquelin.st...@gmail.com>
Cc: net...@vger.kernel.org
Cc: linux-st...@st-md-mailman.stormreply.com
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
---
 drivers/net/ethernet/stmicro/stmmac/dwxgmac2.h     |  26 ++
 .../net/ethernet/stmicro/stmmac/dwxgmac2_core.c    | 281 +++++++++++++++++++++
 drivers/net/ethernet/stmicro/stmmac/dwxgmac2_dma.c |   4 +
 3 files changed, 311 insertions(+)

diff --git a/drivers/net/ethernet/stmicro/stmmac/dwxgmac2.h 
b/drivers/net/ethernet/stmicro/stmmac/dwxgmac2.h
index 29bbe8218600..3d8782f99f64 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwxgmac2.h
+++ b/drivers/net/ethernet/stmicro/stmmac/dwxgmac2.h
@@ -110,6 +110,12 @@
 #define XGMAC_HWFEAT_RXCHCNT           GENMASK(15, 12)
 #define XGMAC_HWFEAT_TXQCNT            GENMASK(9, 6)
 #define XGMAC_HWFEAT_RXQCNT            GENMASK(3, 0)
+#define XGMAC_HW_FEATURE3              0x00000128
+#define XGMAC_HWFEAT_ASP               GENMASK(15, 14)
+#define XGMAC_MAC_DPP_FSM_INT_STATUS   0x00000150
+#define XGMAC_MAC_FSM_CONTROL          0x00000158
+#define XGMAC_PRTYEN                   BIT(1)
+#define XGMAC_TMOUTEN                  BIT(0)
 #define XGMAC_MDIO_ADDR                        0x00000200
 #define XGMAC_MDIO_DATA                        0x00000204
 #define XGMAC_MDIO_C22P                        0x00000220
@@ -154,6 +160,16 @@
 #define XGMAC_TC_PRTY_MAP1             0x00001044
 #define XGMAC_PSTC(x)                  GENMASK((x) * 8 + 7, (x) * 8)
 #define XGMAC_PSTC_SHIFT(x)            ((x) * 8)
+#define XGMAC_MTL_ECC_CONTROL          0x000010c0
+#define XGMAC_MTL_SAFETY_INT_STATUS    0x000010c4
+#define XGMAC_MEUIS                    BIT(1)
+#define XGMAC_MECIS                    BIT(0)
+#define XGMAC_MTL_ECC_INT_ENABLE       0x000010c8
+#define XGMAC_RPCEIE                   BIT(12)
+#define XGMAC_ECEIE                    BIT(8)
+#define XGMAC_RXCEIE                   BIT(4)
+#define XGMAC_TXCEIE                   BIT(0)
+#define XGMAC_MTL_ECC_INT_STATUS       0x000010cc
 #define XGMAC_MTL_TXQ_OPMODE(x)                (0x00001100 + (0x80 * (x)))
 #define XGMAC_TQS                      GENMASK(25, 16)
 #define XGMAC_TQS_SHIFT                        16
@@ -218,6 +234,16 @@
 #define XGMAC_TDPS                     GENMASK(29, 0)
 #define XGMAC_RX_EDMA_CTRL             0x00003044
 #define XGMAC_RDPS                     GENMASK(29, 0)
+#define XGMAC_DMA_SAFETY_INT_STATUS    0x00003064
+#define XGMAC_MCSIS                    BIT(31)
+#define XGMAC_MSUIS                    BIT(29)
+#define XGMAC_MSCIS                    BIT(28)
+#define XGMAC_DEUIS                    BIT(1)
+#define XGMAC_DECIS                    BIT(0)
+#define XGMAC_DMA_ECC_INT_ENABLE       0x00003068
+#define XGMAC_DCEIE                    BIT(1)
+#define XGMAC_TCEIE                    BIT(0)
+#define XGMAC_DMA_ECC_INT_STATUS       0x0000306c
 #define XGMAC_DMA_CH_CONTROL(x)                (0x00003100 + (0x80 * (x)))
 #define XGMAC_PBLx8                    BIT(16)
 #define XGMAC_DMA_CH_TX_CONTROL(x)     (0x00003104 + (0x80 * (x)))
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_core.c 
b/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_core.c
index e2dbebeb59e9..19dfb72cab11 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_core.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_core.c
@@ -530,6 +530,284 @@ static void dwxgmac2_update_vlan_hash(struct 
mac_device_info *hw, u32 hash,
        }
 }
 
+struct dwxgmac3_error_desc {
+       bool valid;
+       const char *desc;
+       const char *detailed_desc;
+};
+
+#define STAT_OFF(field)                offsetof(struct stmmac_safety_stats, 
field)
+
+static void dwxgmac3_log_error(struct net_device *ndev, u32 value, bool corr,
+                              const char *module_name,
+                              const struct dwxgmac3_error_desc *desc,
+                              unsigned long field_offset,
+                              struct stmmac_safety_stats *stats)
+{
+       unsigned long loc, mask;
+       u8 *bptr = (u8 *)stats;
+       unsigned long *ptr;
+
+       ptr = (unsigned long *)(bptr + field_offset);
+
+       mask = value;
+       for_each_set_bit(loc, &mask, 32) {
+               netdev_err(ndev, "Found %s error in %s: '%s: %s'\n", corr ?
+                               "correctable" : "uncorrectable", module_name,
+                               desc[loc].desc, desc[loc].detailed_desc);
+
+               /* Update counters */
+               ptr[loc]++;
+       }
+}
+
+static const struct dwxgmac3_error_desc dwxgmac3_mac_errors[32]= {
+       { true, "ATPES", "Application Transmit Interface Parity Check Error" },
+       { true, "DPES", "Descriptor Cache Data Path Parity Check Error" },
+       { true, "TPES", "TSO Data Path Parity Check Error" },
+       { true, "TSOPES", "TSO Header Data Path Parity Check Error" },
+       { true, "MTPES", "MTL Data Path Parity Check Error" },
+       { true, "MTSPES", "MTL TX Status Data Path Parity Check Error" },
+       { true, "MTBUPES", "MAC TBU Data Path Parity Check Error" },
+       { true, "MTFCPES", "MAC TFC Data Path Parity Check Error" },
+       { true, "ARPES", "Application Receive Interface Data Path Parity Check 
Error" },
+       { true, "MRWCPES", "MTL RWC Data Path Parity Check Error" },
+       { true, "MRRCPES", "MTL RCC Data Path Parity Check Error" },
+       { true, "CWPES", "CSR Write Data Path Parity Check Error" },
+       { true, "ASRPES", "AXI Slave Read Data Path Parity Check Error" },
+       { true, "TTES", "TX FSM Timeout Error" },
+       { true, "RTES", "RX FSM Timeout Error" },
+       { true, "CTES", "CSR FSM Timeout Error" },
+       { true, "ATES", "APP FSM Timeout Error" },
+       { true, "PTES", "PTP FSM Timeout Error" },
+       { false, "UNKNOWN", "Unknown Error" }, /* 18 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 19 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 20 */
+       { true, "MSTTES", "Master Read/Write Timeout Error" },
+       { true, "SLVTES", "Slave Read/Write Timeout Error" },
+       { true, "ATITES", "Application Timeout on ATI Interface Error" },
+       { true, "ARITES", "Application Timeout on ARI Interface Error" },
+       { true, "FSMPES", "FSM State Parity Error" },
+       { false, "UNKNOWN", "Unknown Error" }, /* 26 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 27 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 28 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 29 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 30 */
+       { true, "CPI", "Control Register Parity Check Error" },
+};
+
+static void dwxgmac3_handle_mac_err(struct net_device *ndev,
+                                   void __iomem *ioaddr, bool correctable,
+                                   struct stmmac_safety_stats *stats)
+{
+       u32 value;
+
+       value = readl(ioaddr + XGMAC_MAC_DPP_FSM_INT_STATUS);
+       writel(value, ioaddr + XGMAC_MAC_DPP_FSM_INT_STATUS);
+
+       dwxgmac3_log_error(ndev, value, correctable, "MAC",
+                          dwxgmac3_mac_errors, STAT_OFF(mac_errors), stats);
+}
+
+static const struct dwxgmac3_error_desc dwxgmac3_mtl_errors[32]= {
+       { true, "TXCES", "MTL TX Memory Error" },
+       { true, "TXAMS", "MTL TX Memory Address Mismatch Error" },
+       { true, "TXUES", "MTL TX Memory Error" },
+       { false, "UNKNOWN", "Unknown Error" }, /* 3 */
+       { true, "RXCES", "MTL RX Memory Error" },
+       { true, "RXAMS", "MTL RX Memory Address Mismatch Error" },
+       { true, "RXUES", "MTL RX Memory Error" },
+       { false, "UNKNOWN", "Unknown Error" }, /* 7 */
+       { true, "ECES", "MTL EST Memory Error" },
+       { true, "EAMS", "MTL EST Memory Address Mismatch Error" },
+       { true, "EUES", "MTL EST Memory Error" },
+       { false, "UNKNOWN", "Unknown Error" }, /* 11 */
+       { true, "RPCES", "MTL RX Parser Memory Error" },
+       { true, "RPAMS", "MTL RX Parser Memory Address Mismatch Error" },
+       { true, "RPUES", "MTL RX Parser Memory Error" },
+       { false, "UNKNOWN", "Unknown Error" }, /* 15 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 16 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 17 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 18 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 19 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 20 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 21 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 22 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 23 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 24 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 25 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 26 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 27 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 28 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 29 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 30 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 31 */
+};
+
+static void dwxgmac3_handle_mtl_err(struct net_device *ndev,
+                                   void __iomem *ioaddr, bool correctable,
+                                   struct stmmac_safety_stats *stats)
+{
+       u32 value;
+
+       value = readl(ioaddr + XGMAC_MTL_ECC_INT_STATUS);
+       writel(value, ioaddr + XGMAC_MTL_ECC_INT_STATUS);
+
+       dwxgmac3_log_error(ndev, value, correctable, "MTL",
+                          dwxgmac3_mtl_errors, STAT_OFF(mtl_errors), stats);
+}
+
+static const struct dwxgmac3_error_desc dwxgmac3_dma_errors[32]= {
+       { true, "TCES", "DMA TSO Memory Error" },
+       { true, "TAMS", "DMA TSO Memory Address Mismatch Error" },
+       { true, "TUES", "DMA TSO Memory Error" },
+       { false, "UNKNOWN", "Unknown Error" }, /* 3 */
+       { true, "DCES", "DMA DCACHE Memory Error" },
+       { true, "DAMS", "DMA DCACHE Address Mismatch Error" },
+       { true, "DUES", "DMA DCACHE Memory Error" },
+       { false, "UNKNOWN", "Unknown Error" }, /* 7 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 8 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 9 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 10 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 11 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 12 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 13 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 14 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 15 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 16 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 17 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 18 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 19 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 20 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 21 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 22 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 23 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 24 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 25 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 26 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 27 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 28 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 29 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 30 */
+       { false, "UNKNOWN", "Unknown Error" }, /* 31 */
+};
+
+static void dwxgmac3_handle_dma_err(struct net_device *ndev,
+                                   void __iomem *ioaddr, bool correctable,
+                                   struct stmmac_safety_stats *stats)
+{
+       u32 value;
+
+       value = readl(ioaddr + XGMAC_DMA_ECC_INT_STATUS);
+       writel(value, ioaddr + XGMAC_DMA_ECC_INT_STATUS);
+
+       dwxgmac3_log_error(ndev, value, correctable, "DMA",
+                          dwxgmac3_dma_errors, STAT_OFF(dma_errors), stats);
+}
+
+static int dwxgmac3_safety_feat_config(void __iomem *ioaddr, unsigned int asp)
+{
+       u32 value;
+
+       if (!asp)
+               return -EINVAL;
+
+       /* 1. Enable Safety Features */
+       writel(0x0, ioaddr + XGMAC_MTL_ECC_CONTROL);
+
+       /* 2. Enable MTL Safety Interrupts */
+       value = readl(ioaddr + XGMAC_MTL_ECC_INT_ENABLE);
+       value |= XGMAC_RPCEIE; /* RX Parser Memory Correctable Error */
+       value |= XGMAC_ECEIE; /* EST Memory Correctable Error */
+       value |= XGMAC_RXCEIE; /* RX Memory Correctable Error */
+       value |= XGMAC_TXCEIE; /* TX Memory Correctable Error */
+       writel(value, ioaddr + XGMAC_MTL_ECC_INT_ENABLE);
+
+       /* 3. Enable DMA Safety Interrupts */
+       value = readl(ioaddr + XGMAC_DMA_ECC_INT_ENABLE);
+       value |= XGMAC_DCEIE; /* Descriptor Cache Memory Correctable Error */
+       value |= XGMAC_TCEIE; /* TSO Memory Correctable Error */
+       writel(value, ioaddr + XGMAC_DMA_ECC_INT_ENABLE);
+
+       /* Only ECC Protection for External Memory feature is selected */
+       if (asp <= 0x1)
+               return 0;
+
+       /* 4. Enable Parity and Timeout for FSM */
+       value = readl(ioaddr + XGMAC_MAC_FSM_CONTROL);
+       value |= XGMAC_PRTYEN; /* FSM Parity Feature */
+       value |= XGMAC_TMOUTEN; /* FSM Timeout Feature */
+       writel(value, ioaddr + XGMAC_MAC_FSM_CONTROL);
+
+       return 0;
+}
+
+static int dwxgmac3_safety_feat_irq_status(struct net_device *ndev,
+                                          void __iomem *ioaddr,
+                                          unsigned int asp,
+                                          struct stmmac_safety_stats *stats)
+{
+       bool err, corr;
+       u32 mtl, dma;
+       int ret = 0;
+
+       if (!asp)
+               return -EINVAL;
+
+       mtl = readl(ioaddr + XGMAC_MTL_SAFETY_INT_STATUS);
+       dma = readl(ioaddr + XGMAC_DMA_SAFETY_INT_STATUS);
+
+       err = (mtl & XGMAC_MCSIS) || (dma & XGMAC_MCSIS);
+       corr = false;
+       if (err) {
+               dwxgmac3_handle_mac_err(ndev, ioaddr, corr, stats);
+               ret |= !corr;
+       }
+
+       err = (mtl & (XGMAC_MEUIS | XGMAC_MECIS)) ||
+             (dma & (XGMAC_MSUIS | XGMAC_MSCIS));
+       corr = (mtl & XGMAC_MECIS) || (dma & XGMAC_MSCIS);
+       if (err) {
+               dwxgmac3_handle_mtl_err(ndev, ioaddr, corr, stats);
+               ret |= !corr;
+       }
+
+       err = dma & (XGMAC_DEUIS | XGMAC_DECIS);
+       corr = dma & XGMAC_DECIS;
+       if (err) {
+               dwxgmac3_handle_dma_err(ndev, ioaddr, corr, stats);
+               ret |= !corr;
+       }
+
+       return ret;
+}
+
+static const struct dwxgmac3_error {
+       const struct dwxgmac3_error_desc *desc;
+} dwxgmac3_all_errors[] = {
+       { dwxgmac3_mac_errors },
+       { dwxgmac3_mtl_errors },
+       { dwxgmac3_dma_errors },
+};
+
+static int dwxgmac3_safety_feat_dump(struct stmmac_safety_stats *stats,
+                                    int index, unsigned long *count,
+                                    const char **desc)
+{
+       int module = index / 32, offset = index % 32;
+       unsigned long *ptr = (unsigned long *)stats;
+
+       if (module >= ARRAY_SIZE(dwxgmac3_all_errors))
+               return -EINVAL;
+       if (!dwxgmac3_all_errors[module].desc[offset].valid)
+               return -EINVAL;
+       if (count)
+               *count = *(ptr + index);
+       if (desc)
+               *desc = dwxgmac3_all_errors[module].desc[offset].desc;
+       return 0;
+}
+
 const struct stmmac_ops dwxgmac210_ops = {
        .core_init = dwxgmac2_core_init,
        .set_mac = dwxgmac2_set_mac,
@@ -559,6 +837,9 @@ const struct stmmac_ops dwxgmac210_ops = {
        .pcs_get_adv_lp = NULL,
        .debug = NULL,
        .set_filter = dwxgmac2_set_filter,
+       .safety_feat_config = dwxgmac3_safety_feat_config,
+       .safety_feat_irq_status = dwxgmac3_safety_feat_irq_status,
+       .safety_feat_dump = dwxgmac3_safety_feat_dump,
        .set_mac_loopback = dwxgmac2_set_mac_loopback,
        .rss_configure = dwxgmac2_rss_configure,
        .update_vlan_hash = dwxgmac2_update_vlan_hash,
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_dma.c 
b/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_dma.c
index b50e275e76c2..e4a1c877f2e1 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_dma.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_dma.c
@@ -399,6 +399,10 @@ static void dwxgmac2_get_hw_feature(void __iomem *ioaddr,
                ((hw_cap & XGMAC_HWFEAT_TXQCNT) >> 6) + 1;
        dma_cap->number_rx_queues =
                ((hw_cap & XGMAC_HWFEAT_RXQCNT) >> 0) + 1;
+
+       /* MAC HW feature 3 */
+       hw_cap = readl(ioaddr + XGMAC_HW_FEATURE3);
+       dma_cap->asp = (hw_cap & XGMAC_HWFEAT_ASP) >> 14;
 }
 
 static void dwxgmac2_rx_watchdog(void __iomem *ioaddr, u32 riwt, u32 nchan)
-- 
2.7.4

Reply via email to