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.

Sign up for GitHub

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

Jump to bottom

Module.prepare/rtlil.convert should not propagate inputs to toplevel unless explicitly requested #30

Closed
whitequark opened this issue Jan 27, 2019 · 4 comments
Closed

Module.prepare/rtlil.convert should not propagate inputs to toplevel unless explicitly requested #30

whitequark opened this issue Jan 27, 2019 · 4 comments
Labels
bug
Milestone
0.1

Comments

@whitequark
Copy link
Contributor

No description provided.

@whitequark whitequark added the bug label Jan 27, 2019
@sbourdeauducq
Copy link
Member

I'm also hitting this (and what is described in #35). @whitequark can you prioritize it?

@whitequark
Copy link
Contributor Author

@sbourdeauducq Sure, I was going to fix it ASAP anyway, but haven't been able to get much done in general.

@sbourdeauducq
Copy link
Member

This solution produces warnings such as Warning: Wire top.\con.arbiter.ibus__sel [3] is used but has no driver..

Shouldn't undriven signals take their reset value like they do in Migen?

@whitequark
Copy link
Contributor Author

You are right. I thought they would be reset but I didn't realize that code would not run for other reasons. That's #35 then and I will fix it.

@whitequark whitequark added this to the 0.1 milestone Jun 28, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
0.1
Development

No branches or pull requests
2 participants
@whitequark @sbourdeauducq