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

Rename Artix 7 directory to something "Xilinx Series 7" #310

Closed
mithro opened this issue Jan 3, 2019 · 3 comments
Closed

Rename Artix 7 directory to something "Xilinx Series 7" #310

mithro opened this issue Jan 3, 2019 · 3 comments

Comments

@mithro
Copy link
Contributor

mithro commented Jan 3, 2019

As we add Zynq and Kintex support, 95% of the stuff will be shared with what is currently in the Artix 7 directory. We should rename the Artix 7 to something more generic.
@mithro
Copy link
Contributor Author

mithro commented Jan 3, 2019

I suggest we go with something like xc7?

The Artix parts all start with XC7A. The Kintex with XC7K, the Virtex with XC7V and the Spartan 7 with XC7S.

@mithro
Copy link
Contributor Author

mithro commented Jan 3, 2019

Another option would be series7...

@kgugala
Copy link
Contributor

kgugala commented Jan 3, 2019

I like xc7 - it explicitly says this is Xilinx' 7 series (not a random 7 series). With that name we shouldn't have conflicts with any future 7 series from other vendor.

@litghost litghost closed this as completed Feb 4, 2019
mkurc-ant referenced this issue in antmicro/f4pga-arch-defs Jun 1, 2021
@sharmaln
Merge pull request #310 from antmicro/fix-make-env
3005298 
Fix make env
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

No branches or pull requests
3 participants
@mithro @litghost @kgugala