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

Triple-repeated statements #46

Closed
jeffdi opened this issue Sep 26, 2022 · 3 comments · Fixed by #48
Closed

Triple-repeated statements #46

jeffdi opened this issue Sep 26, 2022 · 3 comments · Fixed by #48
Assignees
@jeffdi
Labels
RTL Changes to verilog source

Comments

@jeffdi
Copy link
Collaborator

jeffdi commented Sep 26, 2022

No description provided.

@jeffdi jeffdi moved this to In Progress in Caravel Redesign Sep 26, 2022
@jeffdi jeffdi moved this from In Progress to Todo in Caravel Redesign Sep 26, 2022
@jeffdi
Copy link
Collaborator Author

jeffdi commented Sep 26, 2022

mgmt_core.v: 3342, 3374

@jeffdi jeffdi self-assigned this Sep 26, 2022
@jeffdi jeffdi moved this from Todo to In Progress in Caravel Redesign Sep 26, 2022
@jeffdi jeffdi mentioned this issue Sep 28, 2022
Merged
@jeffdi jeffdi linked a pull request Sep 28, 2022 that will close this issue
Wb address fix #48
Merged
@jeffdi jeffdi moved this from In Progress to Implementation Complete in Caravel Redesign Sep 28, 2022
@jeffdi jeffdi added the RTL Changes to verilog source label Sep 28, 2022
@RTimothyEdwards
Copy link
Contributor

@jeffdi : Is this the same as issue #34 (pull request #35)?

@jeffdi
Copy link
Collaborator Author

jeffdi commented Sep 30, 2022 •
edited
Loading

@RTimothyEdwards : I think these are different. I believe the issue reported is due to a new version of litex than was used by us. I can't find the issue being reported. I will test further.

@azwefabless azwefabless moved this from Implementation Complete to Team Design Reviewed in Caravel Redesign Oct 5, 2022
@jeffdi jeffdi moved this from Team Design Reviewed to Verification Passed in Caravel Redesign Oct 7, 2022
@azwefabless azwefabless moved this from Verification Passed to Done in Caravel Redesign Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeffdi jeffdi

Labels
RTL Changes to verilog source
Projects
Caravel Redesign
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Wb address fix efabless/caravel_mgmt_soc_litex
2 participants
@RTimothyEdwards @jeffdi