wr-servo: be silent on expected incorrect stamps

When adjusting the time, we expect stamps to be invalid for a while. Don't print errors unless the fact repeats unexpectedly. Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

wr-servo: be silent on expected incorrect stamps
When adjusting the time, we expect stamps to be invalid for a while. Don't print errors unless the fact repeats unexpectedly. Signed-off-by: Alessandro Rubini <rubini@gnudd.com>
a54e9679 · Alessandro Rubini · c1e3299a · a54e9679
Commit a54e9679 authored Oct 24, 2013 by Alessandro Rubini
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 3 deletions

wr-servo.c proto-ext-whiterabbit/wr-servo.c +7 -3

No files found.
--- a/proto-ext-whiterabbit/wr-servo.c
+++ b/proto-ext-whiterabbit/wr-servo.c
@@ -256,6 +256,7 @@ int wr_servo_update(struct pp_instance *ppi)
 	uint64_t tics;
 	uint64_t big_delta_fix;
 	uint64_t delay_ms_fix;
+	static int errcount;
 	TimeInternal ts_offset, ts_offset_hw /*, ts_phase_adjust */;
@@ -264,11 +265,14 @@ int wr_servo_update(struct pp_instance *ppi)
 	if(!s->t1.correct || !s->t2.correct ||
 	   !s->t3.correct || !s->t4.correct) {
-		pp_error("%s: TimestampsIncorrect: %d %d %d %d\n", __func__,
+		errcount++;
-			    s->t1.correct, s->t2.correct,
+		if (errcount > 5) /* a 2-3 in a row are expected */
-			    s->t3.correct, s->t4.correct);
+			pp_error("%s: TimestampsIncorrect: %d %d %d %d\n",
+				 __func__, s->t1.correct, s->t2.correct,
+				 s->t3.correct, s->t4.correct);
 		return 0;
 	}
+	errcount = 0;
 	cur_servo_state.update_count++;