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

Not all OBI configurations support rready #48

Open
micprog opened this issue Jun 21, 2024 · 0 comments
Open

Not all OBI configurations support rready #48

micprog opened this issue Jun 21, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@micprog
Copy link
Member

micprog commented Jun 21, 2024

The obi backend currently requires an rready signal to be present, however this is an optional signal not supported by all OBI uses.

https://github.com/pulp-platform/iDMA/blob/master/src/backend/idma_obi_read.sv#L120
https://github.com/pulp-platform/iDMA/blob/master/src/backend/idma_obi_write.sv#L167

Ideally, all requests coming from the obi read are directly acknowledged and absorbed, so the internal buffer needs to be capable of accepting all outstanding transactions. This can be implemented with a credit counter.
The write backend should be slightly easier to manage, as no data needs to be absorbed (only error information).

micprog added a commit to pulp-platform/safety_island that referenced this issue Jun 21, 2024
@thommythomaso thommythomaso self-assigned this Jul 4, 2024
@thommythomaso thommythomaso added the enhancement New feature or request label Jul 4, 2024
micprog added a commit to pulp-platform/safety_island that referenced this issue Jul 8, 2024
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