-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add in a software clock control DUT #654
base: rs/ilaPlot
Are you sure you want to change the base?
Conversation
I should also probably report that somewhere in the combined total of the work from the three PRs (between this one and the two it depends on) the clock update frequency variability mentioned in issue #587 seems to have disappeared. Looking at the |
As per the discussion in the meeting earlier today, the The issue is, as far as I can tell, that the The current |
Adding to work to be done: |
89ab9a9
to
24817b5
Compare
10ad426
to
d2a0587
Compare
d2a0587
to
7df6bac
Compare
d420ced
to
79591c5
Compare
…es that weren't ready.
79591c5
to
550add3
Compare
This PR adds in a new SwCc DUT based on the work done in my two other PRs. As of the current PR it mostly passes CI except for linting, but that's okay since there's still work to be done.
As for what work is still yet to be done:
rs/measure-sw-cc
branch. The only other DUT this appears in isFullMeshSwCc
.syncIn
toIlaPlotSetup
(see line 908 and lines 937-947 as of time of writing) in order to avoid twosyncRst
asserts, which then causescaptureCond
to becomeCalibrate :: CaptureCondition
for a second time, which is problematic.