Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use an existing standard for start/stop/reset MMIO control registers #193

Open
jvanstraten opened this issue Sep 24, 2019 · 0 comments
Open
Labels
enhancement New feature or request

Comments

@jvanstraten
Copy link
Collaborator

Right now, the MMIO interface expected by the software layers consists of a control register at address 0 with reset/stop/start and a status register with done/busy/idle at address 4, followed by a generic 64-bit result register. But we never really put any more thought into this other than just putting in there what we needed at the time. We should probably do that before we converge to 0.1.

One standard we could for instance use is Xilinx's SDAccel kernel memory map. It's already almost exactly like ours, except SDAccel passes all results through shared memory or streams and thus doesn't have a result register. We could just put that at 0x10/0x14 though. I think interrupts can be omitted, but we might want to add those at some point anyway. The advantage of using this interface is that we can more easily target Alveo boards with SDAccel flow in the future.

@jvanstraten jvanstraten added the enhancement New feature or request label Sep 24, 2019
@jvanstraten jvanstraten added this to the v0.1 milestone Sep 24, 2019
@johanpel johanpel modified the milestones: v0.0.10, v0.0.11 Nov 6, 2019
@johanpel johanpel removed this from the v0.0.17 milestone Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants