Hello,
loosing data at 50Hz is really strange. I am currently running an old MPU-6500 at 200Hz on I2C @400KHz without any issue (using HiKey board).
Can you tell me what is the chip you are using, the board (CPU+freq) and the I2C bus speed?
Anyway, your solution seems to be good, better then putting 0 for timestamps.
Best regards,
JB
From: Martin Kelly <mkelly@xxxxxxxx>
Sent: Wednesday, March 28, 2018 2:34:02 AM
To: Jean-Baptiste Maneyrol; Jonathan Cameron
Cc: linux-iio@xxxxxxxxxxxxxxx
Subject: Re: How to handle missing timestamps? (was Re: [PATCH] iio: imu: inv_mpu6050: improve missing timestamp handling)
On 03/27/2018 01:47 AM, Jean-Baptiste Maneyrol wrote:
> Hello,
>
> this raises a good question of which interrupts we are missing (the oldest ones or the newest ones).
>
> My bet would be that we are loosing interrupts from the newest data (the irq thread can take too much time and we loose interrupt). In this case manual timestamping would rather be:
>
> sample 0 (oldest timestamp): interrupt timestamp
> sample 1: interrupt timestamp + 0.1 seconds
> sample 2: interrupt timestamp + 0.2 seconds
> sample 3 (newest timestamp): interrupt timestamp + 0.3 seconds
>
> Can you check with your setup that this is really what is happening?
>
Yes, good point. I did a lot of testing and determined that the sequence
of events is something like this:
IRQ --> timestamp
new datum
new datum
new datum
IRQ --> timestamp
Specifically:
- At 50 Hz, interrupts are being generated at about 30 Hz.
- The timestamps in the FIFO correspond to the *newest* data, not the
oldest.
I tried interpolating using both assumptions: timestamps corresponding
to oldest and then to newest data. Using timestamps corresponding to
oldest data, we get timestamps that are not monotonically increasing and
thus time moving backwards as the data flows. Using timestamps
corresponding to oldest data, we get monotonically increasing time as it
should be, and the data looks pretty consistent.
What I settled on is to use the newer timestamps we see to backdate the
older data, the ones we remove first from the FIFO. It appears to be
working nicely.
I sent a v2 of the patch that does this interpolation.
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
