aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCorinna Vinschen <vinschen@redhat.com>2017-06-23 08:26:30 -0400
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>2017-08-08 20:46:25 -0400
commit2643e6e90210e16c978919617170089b7c2164f7 (patch)
treead396274692ea43208632c4381a4b4646009c57a
parentd75372a2daf5dc48207ee9e5592917e893cddb87 (diff)
igb: Remove incorrect "unexpected SYS WRAP" log message
TSAUXC.DisableSystime is never set, so SYSTIM runs into a SYS WRAP every 1100 secs on 80580/i350/i354 (40 bit SYSTIM) and every 35000 secs on 80576 (45 bit SYSTIM). This wrap event sets the TSICR.SysWrap bit unconditionally. However, checking TSIM at interrupt time shows that this event does not actually cause the interrupt. Rather, it's just bycatch while the actual interrupt is caused by, for instance, TSICR.TXTS. The conclusion is that the SYS WRAP is actually expected, so the "unexpected SYS WRAP" message is entirely bogus and just helps to confuse users. Drop it. Signed-off-by: Corinna Vinschen <vinschen@redhat.com> Acked-by: Jacob Keller <jacob.e.keller@intel.com> Tested-by: Aaron Brown <aaron.f.brown@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
-rw-r--r--drivers/net/ethernet/intel/igb/igb_main.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/ethernet/intel/igb/igb_main.c b/drivers/net/ethernet/intel/igb/igb_main.c
index 5d0a75c1ba0c..1a99164d5d11 100644
--- a/drivers/net/ethernet/intel/igb/igb_main.c
+++ b/drivers/net/ethernet/intel/igb/igb_main.c
@@ -5746,8 +5746,6 @@ static void igb_tsync_interrupt(struct igb_adapter *adapter)
5746 event.type = PTP_CLOCK_PPS; 5746 event.type = PTP_CLOCK_PPS;
5747 if (adapter->ptp_caps.pps) 5747 if (adapter->ptp_caps.pps)
5748 ptp_clock_event(adapter->ptp_clock, &event); 5748 ptp_clock_event(adapter->ptp_clock, &event);
5749 else
5750 dev_err(&adapter->pdev->dev, "unexpected SYS WRAP");
5751 ack |= TSINTR_SYS_WRAP; 5749 ack |= TSINTR_SYS_WRAP;
5752 } 5750 }
5753 5751