igc: Fix igc_ptp_rx_pktstamp()
authorAndre Guedes <andre.guedes@intel.com>
Wed, 10 Mar 2021 06:42:56 +0000 (22:42 -0800)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Thu, 11 Mar 2021 17:37:48 +0000 (09:37 -0800)
commitfc9e5020971d57d7d0b3fef9e2ab2108fcb5588b
tree3f432b693ff4b95ad97adec7be217059fa9752ed
parent9a4a1cdc5ab52118c1f2b216f4240830b6528d32
igc: Fix igc_ptp_rx_pktstamp()

The comment describing the timestamps layout in the packet buffer is
wrong and the code is actually retrieving the timestamp in Timer 1
reference instead of Timer 0. This hasn't been a big issue so far
because hardware is configured to report both timestamps using Timer 0
(see IGC_SRRCTL register configuration in igc_ptp_enable_rx_timestamp()
helper). This patch fixes the comment and the code so we retrieve the
timestamp in Timer 0 reference as expected.

This patch also takes the opportunity to get rid of the hw.mac.type check
since it is not required.

Fixes: 81b055205e8ba ("igc: Add support for RX timestamping")
Signed-off-by: Andre Guedes <andre.guedes@intel.com>
Signed-off-by: Vedang Patel <vedang.patel@intel.com>
Signed-off-by: Jithu Joseph <jithu.joseph@intel.com>
Reviewed-by: Maciej Fijalkowski <maciej.fijalkowski@intel.com>
Tested-by: Dvora Fuxbrumer <dvorax.fuxbrumer@linux.intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/igc/igc.h
drivers/net/ethernet/intel/igc/igc_ptp.c