Skip to content

PR to merge chips-IT changes to iDMA#109

Open
RiccardoGandolfi wants to merge 3 commits into
pulp-platform:develfrom
FondazioneChipsIT:rg/pulp_cluster_pr
Open

PR to merge chips-IT changes to iDMA#109
RiccardoGandolfi wants to merge 3 commits into
pulp-platform:develfrom
FondazioneChipsIT:rg/pulp_cluster_pr

Conversation

@RiccardoGandolfi

Copy link
Copy Markdown

Just added a parameter to the iDMA_legalizer_page_splitter module.

Related to this PR: pulp-platform/pulp_cluster#115

@yvantor

yvantor commented Jun 1, 2026

Copy link
Copy Markdown

@RiccardoGandolfi make sure you align the parameter definition to the IP name convention (i.s. BurstLen instead of Burst_len). I would also move it before type parameters since it is a pure number (so I would group it with the other int unsigned parameters).

@RiccardoGandolfi

Copy link
Copy Markdown
Author

@RiccardoGandolfi make sure you align the parameter definition to the IP name convention (i.s. BurstLen instead of Burst_len). I would also move it before type parameters since it is a pure number (so I would group it with the other int unsigned parameters).

Should be done now

@RiccardoGandolfi RiccardoGandolfi requested a review from yvantor June 9, 2026 07:33
@RiccardoGandolfi RiccardoGandolfi force-pushed the rg/pulp_cluster_pr branch 2 times, most recently from 6ae11a5 to 81b2ee6 Compare June 9, 2026 12:49
@RiccardoGandolfi RiccardoGandolfi changed the base branch from master to devel June 10, 2026 09:20
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.

2 participants