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

Got the DUT working (: #723

Draft
wants to merge 21 commits into
base: martijn/demo-dut
Choose a base branch
from
Draft

Got the DUT working (: #723

wants to merge 21 commits into from

Conversation

rslawson
Copy link
Collaborator

@rslawson rslawson commented Feb 24, 2025

Okay so maybe this commit is a bit more than that. Here's a list of the things that changed:

  • Vivado logging directory is now configurable in the CI file. Currently, this is set to _build/hitl/. Moved to Allow setting of the Vivado logging directory #725
  • Remove set -x from the OpenOCD start script, which I introduced in the head commit of the original demo DUT branch.
  • Fixes to the TCL file for OpenOCD. Why do these fixes make it work? Great question.
  • Fixes to the demo DUT. This includes:
    • Adding a whoami Circuit that allows reporting of a 32 bit identifier over Wishbone.
    • Adding a debugging ILA to both the DUT and testing wrapper.
    • Changing the reset on the FINC/FDEC pulse shaper to be on all the probe_all_programmed signal.
    • Adjusting the reduction of the FIFO overflow/underflow signals from and to or.
    • Changing noFifoOverflows and noFifoUnderflows to fifoOverflows and fifoUnderflows respectively, and adjusting their usage in the test wrapper.
  • Changes to the demo DUT driver. This includes:
    • Deasserting probe_all_programmed in the case that more test cases are added later. This will prevent any issues from occurring should that happen, otherwise it's harmless.
    • Actually asserting probe_all_programmed once all devices are programmed.
    • Setting longer timeouts for OpenOCD and GDB to terminate.
    • Adding in device ID checks using GDB memory inspection of the new whoami Wishbone components.
    • Actually starting the clock control program on all devices.
  • Added a setTimeout function to our GDB utility file. Moved to Add wrapper function around the set timeout GDB command. #726
  • Added a catch to running the driver function in Shake, allowing execution to continue to downloading the ILA data. Moved to Add a catch to the driver function to allow ILA data to be downloaded. #727

@rslawson rslawson force-pushed the rs/demo-dut branch 2 times, most recently from ca60e74 to 3458129 Compare February 25, 2025 13:21
@rslawson rslawson force-pushed the rs/demo-dut branch 23 times, most recently from 4eb8dcb to 379df03 Compare March 3, 2025 10:41
@rslawson rslawson force-pushed the rs/demo-dut branch 4 times, most recently from 3c03cdd to 546493a Compare March 3, 2025 19:49
@martijnbastiaan martijnbastiaan marked this pull request as draft March 4, 2025 08:16
@rslawson rslawson force-pushed the rs/demo-dut branch 3 times, most recently from 3e9f1dc to b48d91c Compare March 4, 2025 08:41
@rslawson rslawson force-pushed the rs/demo-dut branch 3 times, most recently from 0bdfeb4 to ed80392 Compare March 4, 2025 13:07
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.

3 participants