Skip to content

clint: systemRDL based register generation - #252

Merged
creinwar merged 18 commits into
mainfrom
fischeti/rdl-clint
Aug 7, 2026
Merged

creinwar merged 18 commits into
mainfrom
fischeti/rdl-clint

Conversation

@fischeti

@fischeti fischeti commented Nov 30, 2025 •

Copy link
Copy Markdown
Contributor

Integrates new clint version with systemRDL registers and an APB interface.

TODO:

@fischeti
fischeti marked this pull request as ready for review November 30, 2025 17:30
@fischeti
fischeti requested a review from paulsc96 December 1, 2025 09:13
@fischeti fischeti mentioned this pull request Dec 1, 2025
6 tasks done
@paulsc96

paulsc96 commented Dec 2, 2025

Copy link
Copy Markdown
Collaborator

I updated peakrdl-regblock and the build works in CI now.

I have no objections to this PR in principle. The question is if we want to coordinate this with other migrations to SystemRDL (and, with it, a migration from regbus to APB peripheral interconnect?) or merge it as is. The additional adapter is fine for a first approach, but I'd like to see a more principled integration in the next release.

Or, that's what I would say if the next release was my problem. But it is very likely not. So if you manage to identify a responsible maintainer for this repo, you can decide the next steps with them.

@fischeti

fischeti commented Dec 4, 2025

Copy link
Copy Markdown
Contributor Author

Replacing the Regbus with an APB bus is definitely the end goal here.

I also don't like the converters too much, and migrating everything at once to SystemRDL would be nicer. However, I think this endeavour has a better chance of success when if it is approached step by step. First migrating all dependencies and then Cheshire itself, which would get rid of all the reg2apb converters again.

For me both ways would be fine. I will check with possible future maintainers first, as you said.

@fischeti
fischeti requested a review from creinwar December 5, 2025 08:55
@fischeti
fischeti force-pushed the fischeti/rdl-clint branch 2 times, most recently from f5dc500 to 739b0ba Compare January 16, 2026 08:23
@fischeti
fischeti force-pushed the fischeti/rdl-clint branch from e8027aa to 91a5749 Compare March 4, 2026 13:46
@fischeti
fischeti marked this pull request as draft March 4, 2026 13:47
@fischeti
fischeti force-pushed the fischeti/rdl-clint branch 5 times, most recently from 62c6bfc to eb9f817 Compare March 6, 2026 07:29
@fischeti
fischeti force-pushed the fischeti/rdl-clint branch from eb9f817 to 9e2f3b0 Compare March 11, 2026 20:34
@fischeti
fischeti changed the base branch from main to fischeti/rdl March 11, 2026 22:15
@fischeti
fischeti force-pushed the fischeti/rdl-clint branch from 63f3818 to 7ca3eb8 Compare March 12, 2026 07:41
@fischeti
fischeti force-pushed the fischeti/rdl-clint branch from 7ca3eb8 to 3f829ae Compare March 12, 2026 07:59
@fischeti
fischeti force-pushed the fischeti/rdl branch 2 times, most recently from ccb6a90 to 4402e32 Compare March 12, 2026 09:23
@fischeti
fischeti force-pushed the fischeti/rdl-clint branch 2 times, most recently from cc5c764 to 343fdc2 Compare March 12, 2026 10:32
@fischeti
fischeti force-pushed the fischeti/rdl-clint branch from 343fdc2 to 5a17309 Compare March 12, 2026 15:40
@fischeti
fischeti marked this pull request as ready for review March 13, 2026 15:04
@fischeti

Copy link
Copy Markdown
Contributor Author

@creinwar @ezelioli can we merge this PR alongside pulp-platform/clint#5?

@fischeti
fischeti force-pushed the fischeti/rdl-clint branch from 37ed3fa to 81ece5c Compare August 7, 2026 12:11
ezelioli
ezelioli previously approved these changes Aug 7, 2026
creinwar
creinwar previously approved these changes Aug 7, 2026

@creinwar creinwar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks!
Once the clint PR is merged and the bender is pointing to a release we can merge this 👍

@fischeti
fischeti dismissed stale reviews from creinwar and ezelioli via b9b5663 August 7, 2026 15:16
@fischeti

fischeti commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

I merged pulp-platform/clint#5 now. I will check the Cheshire CI once more, and then create the new CLINT release and bump it here

@fischeti

fischeti commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

I updated it now, so it would be ready to merge as soon as the CI passes :)

@creinwar
creinwar merged commit 6234e9e into main Aug 7, 2026
18 of 20 checks passed
@creinwar
creinwar deleted the fischeti/rdl-clint branch August 7, 2026 19:19
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.

4 participants