Repository navigation
clint: systemRDL based register generation - #252
Conversation
|
I updated 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. |
|
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. |
f5dc500 to
739b0ba
Compare
e8027aa to
91a5749
Compare
62c6bfc to
eb9f817
Compare
eb9f817 to
9e2f3b0
Compare
63f3818 to
7ca3eb8
Compare
7ca3eb8 to
3f829ae
Compare
ccb6a90 to
4402e32
Compare
cc5c764 to
343fdc2
Compare
343fdc2 to
5a17309
Compare
|
@creinwar @ezelioli can we merge this PR alongside pulp-platform/clint#5? |
deps: Bump clint deps: Bump clint deps: Bump clint
37ed3fa to
81ece5c
Compare
creinwar
left a comment
There was a problem hiding this comment.
LGTM, thanks!
Once the clint PR is merged and the bender is pointing to a release we can merge this 👍
|
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 |
|
I updated it now, so it would be ready to merge as soon as the CI passes :) |
Integrates new clint version with systemRDL registers and an APB interface.
TODO:
peakrdl-regblockpackage is too old, which omits parameter definition in the generated packages.ERROR: Cannot start server on port 3042: already in use.errors during FPGA boot jobsclintto new release