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

xilinx.cpp: After programming, go to bypass, PR for #436 #440

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

UweBonnes
Copy link
Contributor

Needed for xc7s50 on VMM3 boards to detect FLASH.

Needed for xc7s50 on VMM3 boards to detect FLASH
@trabucayre
Copy link
Owner

LGTM, just a small question:
An oneshot BYPASS read is enough? Not required to poll until DONE is high?

Thanks!

@UweBonnes
Copy link
Contributor Author

I first has a loop waiting for DONE and a counter for the loop. The loop was only executed once, so I removed the loop and still flash was recognized. Removing the BYPASS instruction resulted again in a fail to recognize the flash.

@trabucayre trabucayre merged commit 8ed388c into trabucayre:master Mar 6, 2024
11 of 13 checks passed
@trabucayre
Copy link
Owner

Applied Thanks @UweBonnes !

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