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

Bug Fixes and Enhancements from LDMX Project Development #1166

Draft
wants to merge 113 commits into
base: pre-release
Choose a base branch
from

Conversation

bengineerd
Copy link
Contributor

@bengineerd bengineerd commented Jun 12, 2024

Description

There are many small changes, mostly to fix issues found in simulation.

Details

  • AxiStreamResize
    • Better error reporting if generic assert fails
  • Ad9249ReadoutGroup2
    • Initialize signal to avoid unknowns in simulation
  • Ad9249Group (simulation)
    • Fix incomplete sensitivity list
  • Pgp2FcAxi
    • Apply TX FC bus from input to PGP block
  • PGP2FC GTY IP Core
    • Remove "project_parameters" from XCI file
    • Allows XCI to load with different US+ FPGAs
  • PGP2FC GTY IP Core Wrapper
    • Output rxPmaResetDone
      • Needed for external MMCM reset
    • Update resets based on simulation discoveries
    • Set freerun frequency to 185.71/2 MHz
  • Add GTH IP Core and wrapper for PGP2FC
  • ClockManagerUltraScale (Plus)
    • Resolve unknowns on DrpDo output from GT to zeros to avoid propagating them in simulation
  • GtRxAlignCheck
    • Use registered values for read-only AXI-Lite registers
    • Apply bug fix so resetErr only reset the FSM when resetDone is high
  • AxiLiteRingBuffer
    • Fix bugs in Rogue Device
  • Rogue Devices
    • Add several registers to the "NoConfig" group

bengineerd and others added 30 commits November 22, 2022 15:47
cbakalis-slac and others added 30 commits September 25, 2024 14:36
Remove project specific outputdir value
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants