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

WIP add XC7K420T #1901

Closed
wants to merge 5 commits into from
Closed

WIP add XC7K420T #1901

wants to merge 5 commits into from

Conversation

hansfbaier
Copy link
Collaborator

@hansfbaier hansfbaier commented Apr 13, 2022

Add support for XC7K420T.
This seemed to have been a very popular part for mining and is available for relatively cheap on AliExpress.
This is actually an easy part to support because it has no high speed banks (it has GTX Transceivers in the place of those).

@jeremyherbert
Copy link

@hansfbaier just wanted to chime in and say I’m a big fan of your work! Keep it up! I’m very much looking forward to using the k7 with an open toolchain.

@hansfbaier
Copy link
Collaborator Author

@jeremyherbert Thank you!

@hansfbaier
Copy link
Collaborator Author

@kgugala It looks like the CI scripts are missing the db prepare for the bigger kintex parts. What is to be done here? Can I do it?
image

@hansfbaier
Copy link
Collaborator Author

@mithro When will the CI be able to run non-webpack parts?

Signed-off-by: Hans Baier <hansfbaier@gmail.com>
Signed-off-by: Hans Baier <hansfbaier@gmail.com>
Signed-off-by: Hans Baier <hansfbaier@gmail.com>
@mithro
Copy link
Contributor

mithro commented Apr 16, 2022

Support for non-webpack parts was added in #1805

@acomodi - Can you confirm this ended up also happening in the GitHub Actions version?

@hansfbaier
Copy link
Collaborator Author

Closing this pull request in favor of #1908

@hansfbaier hansfbaier closed this Apr 18, 2022
@hansfbaier
Copy link
Collaborator Author

hansfbaier commented Apr 18, 2022

@jeremyherbert First success: #1908 .
See here if you want to play with the results: kazkojima/db-workspace-for-kintex7#4

@hansfbaier
Copy link
Collaborator Author

Support for non-webpack parts was added in #1805

@acomodi - Can you confirm this ended up also happening in the GitHub Actions version?

I think I found why the license is unused (because XRAY_SETTINGS is only kintex7 if the k70t part is used), and I already have a fix: 0ba5127

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.

None yet

3 participants