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

Add IODELAY. #295

Merged
merged 3 commits into from
Dec 13, 2024
Merged

Add IODELAY. #295

merged 3 commits into from
Dec 13, 2024

Conversation

yrabbit
Copy link
Collaborator

@yrabbit yrabbit commented Nov 30, 2024

Each IO contains an IODELAY module, providing a total of 128 (0~127) delays. The delay may be set statically or dynamically.

The delay block is enabled immediately before/after the IO pad.

Delay is formally part of IOLOGIC, but can be used without ddr/oser/ides as shown in the iodelay.v example.

The examples blinky-oddr.v and oser4.v are modified to test the new primitive at the same time.

Each IO contains an IODELAY module, providing a total of 128 (0~127)
delays. The delay may be set statically or dynamically.

The delay block is enabled immediately before/after the IO pad.

Delay is formally part of IOLOGIC, but can be used without ddr/oser/ides
- as shown in the iodelay.v example.

The examples blinky-oddr.v and oser4.v are modified to test the new
primitive at the same time.

Signed-off-by: YRabbit <[email protected]>
@yrabbit yrabbit marked this pull request as ready for review November 30, 2024 08:31
@pepijndevos
Copy link
Member

oh no CI broke :(
but yay IODELAY

@yrabbit
Copy link
Collaborator Author

yrabbit commented Dec 6, 2024

oh no CI broke :( but yay IODELAY

Yeah, let's wait until Yosys learns to distinguish between chip families and their primitives and then we'll restart CI for all hung PRs to be sure:)

@pepijndevos
Copy link
Member

Oh I guess they didn't merge that yet huh...

@yrabbit yrabbit merged commit 3c154d0 into YosysHQ:master Dec 13, 2024
11 of 14 checks passed
@yrabbit yrabbit deleted the iodelay branch December 13, 2024 10:34
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