Skip to content

Add batcher cocotb regression coverage#1426

Merged
ruck314 merged 3 commits into
pre-releasefrom
batcher-tests
Jun 15, 2026
Merged

Add batcher cocotb regression coverage#1426
ruck314 merged 3 commits into
pre-releasefrom
batcher-tests

Conversation

@bengineerd

@bengineerd bengineerd commented May 28, 2026

Copy link
Copy Markdown
Contributor

Description

Adds focused cocotb regression coverage for the AXI Stream batcher family, including the batcher data path, AXI-Lite control/status interface, event-builder path, and common/async clock configurations.

Details

This batcher-focused slice adds thin VHDL wrappers under protocols/batcher/wrappers/ so cocotb can drive and observe the existing RTL through flattened AXI Stream and AXI-Lite interfaces. The wrappers cover:

  • AxiStreamBatcher
  • AxiStreamBatcherAxil
  • AxiStreamBatcherEventBuilder

The new Python regression support under tests/protocols/batcher/ adds shared batcher helpers and directed tests for:

  • batch formatting, frame boundaries, sequence/count behavior, and payload preservation;
  • ready/valid behavior and output frame checking for the core batcher path;
  • AXI-Lite register access, control/status behavior, and software-visible batcher configuration;
  • event-builder routing/formatting behavior through the wrapper topology;
  • common-clock and asynchronous-clock configurations used by the batcher/event-builder paths.

The branch also adds batcher regression planning and handoff notes under docs/plans/batcher-regression/ and updates the shared RTL regression notes with the batcher progress.

This PR is stacked after #1425 and should be merged after packetizer-tests and before rssi-tests.

Related

Depends on #1425. Follow-up stack: #1427.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated no new comments.

@ruck314 ruck314 force-pushed the packetizer-tests branch from eb616cb to 5a937e0 Compare June 14, 2026 22:48
Add thin AXI Stream / AXI-Lite wrappers so cocotb can drive and observe the
existing batcher RTL through flattened interfaces:
- AxiStreamBatcherWrapper
- AxiStreamBatcherAxilWrapper (common + independent AXI-Lite clock modes)
- AxiStreamBatcherEventBuilderWrapper (adds TDEST route mode)
Directed cocotb tests for the AXI Stream batcher family:
- batch formatting, frame boundaries, sequence/count, payload preservation
- ready/valid and output frame checking on the core batcher path
- AXI-Lite register access and control/status behavior
- event-builder routing/formatting through the wrapper topology
- common-clock and asynchronous-clock configurations
@ruck314 ruck314 force-pushed the packetizer-tests branch from 5a937e0 to 965fc67 Compare June 14, 2026 22:53
@ruck314 ruck314 changed the base branch from packetizer-tests to pre-release June 14, 2026 22:54
@ruck314 ruck314 marked this pull request as ready for review June 15, 2026 14:50
@ruck314 ruck314 requested a review from slacrherbst as a code owner June 15, 2026 14:50
@ruck314 ruck314 merged commit 452f253 into pre-release Jun 15, 2026
5 checks passed
@ruck314 ruck314 deleted the batcher-tests branch June 15, 2026 15:01
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