This is only a cosmetic change. Using a do..while loop we avoid direct manipulation of counter loop variable when error happens reading the BT clock. --- health/mcap_sync.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/health/mcap_sync.c b/health/mcap_sync.c index bc1ffcd..78cb163 100644 --- a/health/mcap_sync.c +++ b/health/mcap_sync.c @@ -406,18 +406,18 @@ static void initialize_caps(struct mcap_mcl *mcl) /* Do clock read a number of times and measure latency */ avg = 0; - for (i = 0; i < 20; ++i) { + i = 0; + do { clock_gettime(CLK, &t1); - if (!read_btclock(mcl, &btclock, &btaccuracy)) { - --i; + if (!read_btclock(mcl, &btclock, &btaccuracy)) continue; - } clock_gettime(CLK, &t2); latency = time_us(&t2) - time_us(&t1); latencies[i] = latency; avg += latency; - } + i++; + } while (i < 20); avg /= 20; /* Calculate deviation */ -- 1.7.2.3 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html