Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

daq2/3: Switch to data_offload #1592

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

daq2/3: Switch to data_offload #1592

wants to merge 2 commits into from

Conversation

podgori
Copy link
Contributor

@podgori podgori commented Feb 25, 2025

PR Description

This commit adds support for the Data Offload IP, replacing the dacfifo/adcfifo IPs.
For DAQ2, the project is updated to import the generic MIG DDR3 configuration script, replacing the local script.

Must be merged after this PR: #1573

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

AndrDragomir
AndrDragomir previously approved these changes Mar 25, 2025
Copy link
Contributor

@AndrDragomir AndrDragomir left a comment

Choose a reason for hiding this comment

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

Validated all affected projects in hardware except daq3+kcu105 (timing issues when building, unrelated to this PR)

podgori added 2 commits April 3, 2025 14:44
Signed-off-by: Ionut Podgoreanu <ionut.podgoreanu@analog.com>
Signed-off-by: Ionut Podgoreanu <ionut.podgoreanu@analog.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants