Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wb_acq_core] On trigger acquisitions we can have a sample not belonging to the correct transaction #52

Closed
lerwys opened this issue Jan 13, 2016 · 1 comment
Labels

Comments

@lerwys
Copy link
Contributor

lerwys commented Jan 13, 2016

Problem: On trigger acquisitions, for some unknown reason, we could have the signal "fifo_fc_mux_cnt" on acq_fc_fifo module ending in something different than 0 at the end of a transaction.

This causes the the first sample of the next transaction to have atoms belonging to a previous transaction.

This could be detected in simulation, but our testbench was incomplete and not checking this issue properly. The testbench is comparing the output of acq_fc_fifo with the data read from DDR. So, problems arising before the acq_fc_fifo or in itself would not be detected!

Roadmap:

  1. Fix testbench to correctly validates our module (comparing input data with the data read from DDR)
  2. Investigate issue to why fifo_fc_mux_cnt signal ends in something different than 0 at the end of some transactions.
@lerwys lerwys added the bug label Jan 13, 2016
@lerwys
Copy link
Contributor Author

lerwys commented Jan 13, 2016

Commit 8ffd615 (fixing trigger alignement) did not solve the error completely, although it improved the testbench completion

lerwys added a commit that referenced this issue Jan 13, 2016
…cnt signal error

This is related to github issue #52, in that
the fifo_fc_mux_cnt signal is different than 0, after
the transaction ends. This points to an alignment
problem in the acquisition (typically trigger acquisitions)
and subsequent transactions could be wrong (first sample
belonging to the previous transaction).
@lerwys lerwys closed this as completed in dc57ccb Jan 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant