Skip to content

Conversation

@rswarbrick
Copy link
Contributor

This gets rid of a "todo when you get round to it" note that I left myself and forgot about!

@rswarbrick rswarbrick requested a review from Razer6 September 2, 2025 14:26
@rswarbrick rswarbrick requested a review from a team as a code owner September 2, 2025 14:26
@rswarbrick rswarbrick requested review from eshapira and removed request for a team September 2, 2025 14:26
@rswarbrick rswarbrick added Component:DV DV issue: testbench, test case, etc. IP:racl_ctrl labels Sep 2, 2025
@rswarbrick
Copy link
Contributor Author

@Razer6: I hadn't been thinking about DV for racl_ctrl for a while, but have just remembered that this is pending. Could you take a look at this PR?

We already had num_subscribing_ips (which exposes NumSubscribingIps).
Expose the number of external subscribers too.

Signed-off-by: Rupert Swarbrick <[email protected]>
@rswarbrick rswarbrick force-pushed the racl-ctrl-subscribers branch from fd77cd0 to bfa0474 Compare February 10, 2026 12:28
@rswarbrick
Copy link
Contributor Author

Force-push rebases this to the current master, but no other change.

Copy link
Contributor

@hcallahan-lowrisc hcallahan-lowrisc left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the cleanup!

@rswarbrick rswarbrick added this pull request to the merge queue Feb 10, 2026
Merged via the queue into lowRISC:master with commit 682eb75 Feb 10, 2026
45 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component:DV DV issue: testbench, test case, etc. IP:racl_ctrl

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants