Searched hist:fc9e5020971d57d7d0b3fef9e2ab2108fcb5588b (Results 1 – 2 of 2) sorted by relevance
/openbmc/linux/drivers/net/ethernet/intel/igc/ |
H A D | igc_ptp.c | diff fc9e5020971d57d7d0b3fef9e2ab2108fcb5588b Wed Mar 10 00:42:56 CST 2021 Andre Guedes <andre.guedes@intel.com> 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>
|
H A D | igc.h | diff fc9e5020971d57d7d0b3fef9e2ab2108fcb5588b Wed Mar 10 00:42:56 CST 2021 Andre Guedes <andre.guedes@intel.com> 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>
|