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

Codebreaker Lab Failure #316

Closed
Zach227 opened this issue Jun 6, 2022 · 6 comments
Closed

Codebreaker Lab Failure #316

Zach227 opened this issue Jun 6, 2022 · 6 comments

Comments

@Zach227
Copy link

Zach227 commented Jun 6, 2022

When running the Codebreaker Lab through the f4pga flow with the default Yosys parser it works correctly. When the Surelog parser is used, it generates a bit stream but does not function properly on the board. The seven segment display works but it does not decrypt the message.

Full Project: codebreaker.zip (not working with Surelog)

Replace decrypt_rc4.sv from the project with this edited version and the project will work with Surelog. There are comments on lines 38 and 121 pointing out the 2 changes to the file. When working properly, the programmed FPGA will decrypt the message and stop the timer at 11.88 seconds. It will print the message to a UART receiver (19200 baud, 8 data bits, 1 stop bit, odd parity).

@Zach227 Zach227 changed the title Codebreaker Lab Failure SYMBIFLOW-CLASSROOM Codebreaker Lab Failure Jun 6, 2022
@rkapuscik
Copy link
Contributor

Thanks for the report.

One of the changes (explicit width of enum variable) is I think a duplicate of #311 (it seems to be the same test case?). I'm not sure about the second change (merging the always_ff), we'll look into that.

@Zach227 Zach227 changed the title SYMBIFLOW-CLASSROOM Codebreaker Lab Failure Codebreaker Lab Failure Jun 7, 2022
@rkapuscik
Copy link
Contributor

I've retested this and after the enum variable fix the Yosys tree is functionally the same with or without merging the always_ff block, so no changes should be needed anymore.

Please retest with the latest plugin.

@westonMS
Copy link

We have re-tested this with the latest plugin and the design works now. However, we want fixes to be available to all users when they use the main install. I re-installed the tool-chain with the latest install instructions and this is not fixed without manually updating the plugin.

@mithro
Copy link
Contributor

mithro commented Jul 16, 2022

FYI - @kgugala

@tmichalak
Copy link
Collaborator

@westonMS we are working on a conda_lock_update mechanism - see more detailed comment

@Zach227
Copy link
Author

Zach227 commented Jul 26, 2022

I have retested with the latest install instructions and this issue is resolved.

@Zach227 Zach227 closed this as completed Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants