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

Inconsistent use of 'V' value prefix #9

Closed
daveshah1 opened this issue Jul 18, 2020 · 1 comment · Fixed by #17
Closed

Inconsistent use of 'V' value prefix #9

daveshah1 opened this issue Jul 18, 2020 · 1 comment · Fixed by #17

Comments

@daveshah1
Copy link

Most of the 1/0 choices in https://github.com/SymbiFlow/prjuray-db/blob/master/zynqusp/segbits_clem.db use .V1/.V0 to distinguish value possibilities.

e.g.

CLEM.ABCDFF.CEUSED.V0 !14_07
CLEM.ABCDFF.CEUSED.V1 14_07
CLEM.ABCDFF.CLKINV.V0 !08_19
CLEM.ABCDFF.CLKINV.V1 08_19
CLEM.AFF.INIT.V0 12_02
CLEM.AFF.INIT.V1 !12_02

but WA[789]USED seems to be an exception:

CLEM.WA7USED.0 !08_23
CLEM.WA7USED.1 08_23
CLEM.WA8USED.0 !11_22
CLEM.WA8USED.1 11_22
CLEM.WA9USED.0 !09_22
CLEM.WA9USED.1 09_22

It would simplify things generating/parsing FASM if one convention could be established here (I personally don't mind either way.)
@litghost
Copy link
Contributor

This is a bug, I"ll submit a fix today.

@litghost litghost mentioned this issue Jul 20, 2020
Merged
tmichalak pushed a commit to tmichalak/prjuray that referenced this issue Aug 25, 2020
@litghost
Merge pull request f4pga#9 from litghost/add_travis
eb0341b 
Add initial Travis CI configuration.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix a feature without a A-Z feature prefix. f4pga/prjuray
2 participants
@litghost @daveshah1