Skip to content

library: jesd204: Fixup Vivado exiting with error #1243

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

Merged
merged 1 commit into from
Jan 16, 2024
Merged

Conversation

gastmaier
Copy link
Contributor

@gastmaier gastmaier commented Jan 3, 2024

PR Description

The lack of the create_xgui_files causes Vivado to exit with an error when running
multiple Vivado instances (parallel make case).
jesd204_common was missing this tcl command.

To reproduce

(cd library ; make clean all -j8)

Fails on main, but not with the fix.
Important: the ip.log between branches are identical, the only difference is that
Vivado exists with an error code at main.

Tested with Vivado 2022.2 and 2023.1.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the code style guidelines
  • I have performed a self-review of changes
  • I have compiled all hdl projects and libraries affected by this PR
  • I have tested in hardware affected projects, at least on relevant boards
  • I have commented my code, at least hard-to-understand parts
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe files, Copyright etc)
  • I have not introduced new Warnings/Critical Warnings on compilation
  • I have added new hdl testbenches or updated existing ones

The lack of the create_xgui_files causes Vivado to exit with an error
when running multiple Vivado instances (parallel make case)

Signed-off-by: Jorge Marques <jorge.marques@analog.com>
@gastmaier gastmaier merged commit 107b043 into main Jan 16, 2024
@gastmaier gastmaier deleted the jesd-make-fixup branch January 16, 2024 12:43
bia1708 pushed a commit to bia1708/hdl that referenced this pull request Apr 5, 2024
)

The lack of the create_xgui_files causes Vivado to exit with an error
when running multiple Vivado instances (parallel make case)

Signed-off-by: Jorge Marques <jorge.marques@analog.com>
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