Fix some issues with br_fifo_test_harness and use for push credit test - Githubissues

xlsynth / bedrock-rtl

High quality and composable base RTL libraries in SystemVerilog

Apache License 2.0

11 stars 2 forks source link

Fix some issues with br_fifo_test_harness and use for push credit test #211

Closed zhemao-openai closed 3 days ago

zhemao-openai commented 4 days ago

The handling of negedge/posedge in the testbench could cause some spurious simulation failures. This change makes things more consistent.
Use the br_fifo_test_harness to check br_fifo_flops_push_credit. The Depth needs to be adjusted to allow full throughput as the test harness expects.
Make sure the timeout counter is actually decrementing.

Stack:

212
211 ⬅

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.