Skip to content

Introduce counterSource to more easily distribute the counter#1186

Draft
lmbollen wants to merge 1 commit intomainfrom
lucas/improve-bus-routing
Draft

Introduce counterSource to more easily distribute the counter#1186
lmbollen wants to merge 1 commit intomainfrom
lucas/improve-bus-routing

Conversation

@lmbollen
Copy link
Contributor

What (what did you do)
Make counter distribution consistent across demos.

Why (context, issues, etc.)
To improve both consistency and routing in of the different demos.

Dear reviewer (anything you'd like the reviewer to pay close attention to?)
Does it matter that we now get undefined contents out of the counter for the first n cycles? I don't think so. and it is also easier to place + route + optimize for vivado.

AI disclaimer (heads-up for more than inline autocomplete)
No AI here

@lmbollen lmbollen force-pushed the lucas/improve-bus-routing branch 3 times, most recently from 13e0a09 to b92f23d Compare February 10, 2026 15:48
@lmbollen lmbollen force-pushed the lucas/improve-bus-routing branch from 13e0a09 to 63ef605 Compare February 24, 2026 10:39
@lmbollen lmbollen force-pushed the lucas/improve-bus-routing branch from 63ef605 to b499ee8 Compare February 24, 2026 10:40
@lmbollen lmbollen changed the title Make counter distribution consistent across demos Introduce counterSource to more easily distribute the counter Feb 24, 2026
@martijnbastiaan
Copy link
Contributor

Does it matter that we now get undefined contents out of the counter for the first n cycles? I don't think so. and it is also easier to place + route + optimize for vivado.

The reset should be aligned with what the counter calls "zero", but it's okay if it looks like this:

in reset:   1   1   0   0   0   0   0
counter:    X   X   X   X   2   3   4

I think that's what you're currently doing, but you could make it more obvious by adjusting the initial contents of the counter to be 0 - delay instead of repeatedly calling - 1.

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