Skip to content

Rj/timestamp tweaks#255

Merged
jordens merged 8 commits into
masterfrom
rj/timestamp-tweaks
Feb 2, 2021
Merged

Rj/timestamp tweaks#255
jordens merged 8 commits into
masterfrom
rj/timestamp-tweaks

Conversation

@jordens

@jordens jordens commented Feb 2, 2021

Copy link
Copy Markdown
Member

No description provided.

The sampling timer and the timestamping timer have the same period.
The sampling interval and the batch size are powers of two.
If the timestamping timer wraps at a power of two larger than the
batch period, it will wrap in sync with the batch period.

Even if it didn't the RPLL would handle that. But it requires that the
timer wraps at the u32/i32 boundary (or be shifted left to wrap there).
@jordens
jordens requested a review from ryan-summers February 2, 2021 12:39
Comment thread src/hardware/configuration.rs
Comment thread src/hardware/configuration.rs Outdated
Comment thread src/bin/lockin-external.rs
Comment thread src/bin/lockin-external.rs Outdated
@jordens
jordens requested a review from ryan-summers February 2, 2021 13:42
@jordens
jordens merged commit 14abaad into master Feb 2, 2021
@bors
bors Bot deleted the rj/timestamp-tweaks branch February 2, 2021 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants