Skip to content

Update utility cores to inline hdl variants (Vivado 2024.2+) #1874

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
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update utility cores to inline hdl variants (Vivado 2024.2+) #1874

wants to merge 1 commit into from

Conversation

bluncan
Copy link
Contributor

@bluncan bluncan commented Aug 8, 2025

PR Description

With Vivado 2024.2, Xilinx introduced new utility IPs variants called "inline_hdl" that are now recommended and should lead to faster build times.

The old variants are supposedly being removed in the 2026 Vivado release.

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)
  • Documentation

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

FilipG24
FilipG24 requested changes Aug 8, 2025
View reviewed changes
@bluncan
Update utility cores to inline hdl variants
612ef4b 
With Vivado 2024.2, Xilinx introduced new utility IPs
variants called "inline_hdl" that are now recommended
and should lead to faster build times.

The old variants are supposedly being removed in the
2026 Vivado release.

Signed-off-by: Bogdan Luncan <[email protected]>
@bluncan bluncan force-pushed the dev_replace_with_inline_hdl branch from a4695d8 to 612ef4b Compare August 8, 2025 10:19
FilipG24
FilipG24 approved these changes Aug 8, 2025
View reviewed changes
Copy link
Contributor

@FilipG24 FilipG24 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! I tested the ad_ip_instance function and it works well! I recommend a CI build for all affected projects!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FilipG24 FilipG24 FilipG24 approved these changes

@IuliaCMoldovan IuliaCMoldovan Awaiting requested review from IuliaCMoldovan IuliaCMoldovan is a code owner

@AndrDragomir AndrDragomir Awaiting requested review from AndrDragomir AndrDragomir is a code owner

@AndreiGrozav AndreiGrozav Awaiting requested review from AndreiGrozav AndreiGrozav is a code owner

@podgori podgori Awaiting requested review from podgori podgori is a code owner

@StancaPop StancaPop Awaiting requested review from StancaPop StancaPop is a code owner

@PopPaul2021 PopPaul2021 Awaiting requested review from PopPaul2021 PopPaul2021 is a code owner

@caosjr caosjr Awaiting requested review from caosjr caosjr is a code owner

@PIoandan PIoandan Awaiting requested review from PIoandan PIoandan is a code owner

@LBFFilho LBFFilho Awaiting requested review from LBFFilho LBFFilho is a code owner

@alin724 alin724 Awaiting requested review from alin724 alin724 is a code owner

@sarpadi sarpadi Awaiting requested review from sarpadi sarpadi is a code owner

@ladace ladace Awaiting requested review from ladace ladace is a code owner

@acostina acostina Awaiting requested review from acostina acostina is a code owner

@gastmaier gastmaier Awaiting requested review from gastmaier gastmaier is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bluncan @FilipG24