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

[ci] Add Yosys synthesis flow #1501

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

rkapuscik
Copy link

Opening a draft PR to get some early input and suggestions.

As proposed in #1481, this PR adds a Github action workflow that synthesizes Ibex via Yosys, using Surelog-UHDM frontend.

It obtains a release from Antmicro's plugin repository and edalize fork (until UHDM support is merged in lowRISC/edalize#3).

Each flow generates an EDIF artifact that can be downloaded and processed further locally.

Rafal Kapuscik added 7 commits January 3, 2022 09:22
Signed-off-by: Rafal Kapuscik <rkapuscik@antmicro.com>
Signed-off-by: Rafal Kapuscik <rkapuscik@antmicro.com>
Signed-off-by: Rafal Kapuscik <rkapuscik@antmicro.com>
Signed-off-by: Rafal Kapuscik <rkapuscik@antmicro.com>
Signed-off-by: Rafal Kapuscik <rkapuscik@antmicro.com>
Signed-off-by: Rafal Kapuscik <rkapuscik@antmicro.com>
Signed-off-by: Rafal Kapuscik <rkapuscik@antmicro.com>
@rkapuscik rkapuscik marked this pull request as draft January 4, 2022 16:56
Copy link
Contributor

@rswarbrick rswarbrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, it's cool that this works!

One question: The first commit adds a special core file for the UHDM flow. Eventually, I assume we'd want to combine this with the existing core files (since we're just specifying a different tool). Is there a reason that's difficult at the moment?

@rkapuscik
Copy link
Author

I based this on a patch we use to add other targets, but I don't think there will be any problem with rolling that into the existing core. I agree that it would be cleaner.

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