igc: Fix the typo in the PTM Control macro
authorSasha Neftin <sasha.neftin@intel.com>
Mon, 21 Aug 2023 17:17:21 +0000 (10:17 -0700)
committerJakub Kicinski <kuba@kernel.org>
Wed, 23 Aug 2023 00:32:15 +0000 (17:32 -0700)
The IGC_PTM_CTRL_SHRT_CYC defines the time between two consecutive PTM
requests. The bit resolution of this field is six bits. That bit five was
missing in the mask. This patch comes to correct the typo in the
IGC_PTM_CTRL_SHRT_CYC macro.

Fixes: a90ec8483732 ("igc: Add support for PTP getcrosststamp()")
Signed-off-by: Sasha Neftin <sasha.neftin@intel.com>
Tested-by: Naama Meir <naamax.meir@linux.intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Link: https://lore.kernel.org/r/20230821171721.2203572-1-anthony.l.nguyen@intel.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/intel/igc/igc_defines.h

index 44a5070..2f780cc 100644 (file)
 #define IGC_PTM_CTRL_START_NOW BIT(29) /* Start PTM Now */
 #define IGC_PTM_CTRL_EN                BIT(30) /* Enable PTM */
 #define IGC_PTM_CTRL_TRIG      BIT(31) /* PTM Cycle trigger */
-#define IGC_PTM_CTRL_SHRT_CYC(usec)    (((usec) & 0x2f) << 2)
+#define IGC_PTM_CTRL_SHRT_CYC(usec)    (((usec) & 0x3f) << 2)
 #define IGC_PTM_CTRL_PTM_TO(usec)      (((usec) & 0xff) << 8)
 
 #define IGC_PTM_SHORT_CYC_DEFAULT      10  /* Default Short/interrupted cycle interval */