]> www.infradead.org Git - users/hch/misc.git/commitdiff
net: ti: am65-cpsw: Update hw timestamping filter for PTPv1 RX packets
authorVishnu Singh <v-singh1@ti.com>
Wed, 17 Sep 2025 04:14:55 +0000 (09:44 +0530)
committerJakub Kicinski <kuba@kernel.org>
Thu, 18 Sep 2025 22:42:28 +0000 (15:42 -0700)
CPTS module of CPSW supports hardware timestamping of PTPv1 packets.Update
the "hwtstamp_rx_filters" of CPSW driver to enable timestamping of received
PTPv1 packets. Also update the advertised capability to include PTPv1.

Signed-off-by: Vishnu Singh <v-singh1@ti.com>
Reviewed-by: Vadim Fedorenko <vadim.fedorenko@linux.dev>
Reviewed-by: MD Danish Anwar <danishanwar@ti.com>
Link: https://patch.msgid.link/20250917041455.1815579-1-v-singh1@ti.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/ti/am65-cpsw-ethtool.c
drivers/net/ethernet/ti/am65-cpsw-nuss.c

index 9032444435e90cdc285e9b4d647912505c34a888..c57497074ae60d88b4f5944d3386bfc3a987ce95 100644 (file)
@@ -694,17 +694,20 @@ static int am65_cpsw_get_ethtool_ts_info(struct net_device *ndev,
                                         struct kernel_ethtool_ts_info *info)
 {
        struct am65_cpsw_common *common = am65_ndev_to_common(ndev);
-       unsigned int ptp_v2_filter;
-
-       ptp_v2_filter = BIT(HWTSTAMP_FILTER_PTP_V2_L4_EVENT)     |
-                       BIT(HWTSTAMP_FILTER_PTP_V2_L4_SYNC)      |
-                       BIT(HWTSTAMP_FILTER_PTP_V2_L4_DELAY_REQ) |
-                       BIT(HWTSTAMP_FILTER_PTP_V2_L2_EVENT)     |
-                       BIT(HWTSTAMP_FILTER_PTP_V2_L2_SYNC)      |
-                       BIT(HWTSTAMP_FILTER_PTP_V2_L2_DELAY_REQ) |
-                       BIT(HWTSTAMP_FILTER_PTP_V2_EVENT)        |
-                       BIT(HWTSTAMP_FILTER_PTP_V2_SYNC)         |
-                       BIT(HWTSTAMP_FILTER_PTP_V2_DELAY_REQ);
+       unsigned int ptp_filter;
+
+       ptp_filter = BIT(HWTSTAMP_FILTER_PTP_V2_L4_EVENT)       |
+                    BIT(HWTSTAMP_FILTER_PTP_V2_L4_SYNC)        |
+                    BIT(HWTSTAMP_FILTER_PTP_V2_L4_DELAY_REQ)   |
+                    BIT(HWTSTAMP_FILTER_PTP_V2_L2_EVENT)       |
+                    BIT(HWTSTAMP_FILTER_PTP_V2_L2_SYNC)        |
+                    BIT(HWTSTAMP_FILTER_PTP_V2_L2_DELAY_REQ)   |
+                    BIT(HWTSTAMP_FILTER_PTP_V2_EVENT)          |
+                    BIT(HWTSTAMP_FILTER_PTP_V2_SYNC)           |
+                    BIT(HWTSTAMP_FILTER_PTP_V2_DELAY_REQ)      |
+                    BIT(HWTSTAMP_FILTER_PTP_V1_L4_EVENT)       |
+                    BIT(HWTSTAMP_FILTER_PTP_V1_L4_SYNC)        |
+                    BIT(HWTSTAMP_FILTER_PTP_V1_L4_DELAY_REQ);
 
        if (!IS_ENABLED(CONFIG_TI_K3_AM65_CPTS))
                return ethtool_op_get_ts_info(ndev, info);
@@ -716,7 +719,7 @@ static int am65_cpsw_get_ethtool_ts_info(struct net_device *ndev,
                SOF_TIMESTAMPING_RAW_HARDWARE;
        info->phc_index = am65_cpts_phc_index(common->cpts);
        info->tx_types = BIT(HWTSTAMP_TX_OFF) | BIT(HWTSTAMP_TX_ON);
-       info->rx_filters = BIT(HWTSTAMP_FILTER_NONE) | ptp_v2_filter;
+       info->rx_filters = BIT(HWTSTAMP_FILTER_NONE) | ptp_filter;
        return 0;
 }
 
index 8b2364f5f73124539d7d769c616efb36d4baeffa..110eb2da8dbc1d1d18e315f94f118f78349c1a11 100644 (file)
@@ -1813,6 +1813,9 @@ static int am65_cpsw_nuss_hwtstamp_set(struct net_device *ndev,
        case HWTSTAMP_FILTER_NONE:
                port->rx_ts_enabled = false;
                break;
+       case HWTSTAMP_FILTER_PTP_V1_L4_EVENT:
+       case HWTSTAMP_FILTER_PTP_V1_L4_SYNC:
+       case HWTSTAMP_FILTER_PTP_V1_L4_DELAY_REQ:
        case HWTSTAMP_FILTER_PTP_V2_L4_EVENT:
        case HWTSTAMP_FILTER_PTP_V2_L4_SYNC:
        case HWTSTAMP_FILTER_PTP_V2_L4_DELAY_REQ:
@@ -1823,7 +1826,7 @@ static int am65_cpsw_nuss_hwtstamp_set(struct net_device *ndev,
        case HWTSTAMP_FILTER_PTP_V2_SYNC:
        case HWTSTAMP_FILTER_PTP_V2_DELAY_REQ:
                port->rx_ts_enabled = true;
-               cfg.rx_filter = HWTSTAMP_FILTER_PTP_V2_EVENT;
+               cfg.rx_filter = HWTSTAMP_FILTER_PTP_V2_EVENT | HWTSTAMP_FILTER_PTP_V1_L4_EVENT;
                break;
        case HWTSTAMP_FILTER_ALL:
        case HWTSTAMP_FILTER_SOME:
@@ -1884,8 +1887,8 @@ static int am65_cpsw_nuss_hwtstamp_get(struct net_device *ndev,
        cfg.flags = 0;
        cfg.tx_type = port->tx_ts_enabled ?
                      HWTSTAMP_TX_ON : HWTSTAMP_TX_OFF;
-       cfg.rx_filter = port->rx_ts_enabled ?
-                       HWTSTAMP_FILTER_PTP_V2_EVENT : HWTSTAMP_FILTER_NONE;
+       cfg.rx_filter = port->rx_ts_enabled ? HWTSTAMP_FILTER_PTP_V2_EVENT |
+                       HWTSTAMP_FILTER_PTP_V1_L4_EVENT : HWTSTAMP_FILTER_NONE;
 
        return copy_to_user(ifr->ifr_data, &cfg, sizeof(cfg)) ? -EFAULT : 0;
 }