search

lowRISC / opentitan

OpenTitan: Open source silicon root of trust
https://www.opentitan.org
Apache License 2.0
2.53k stars 754 forks source link

[top, edn] Future enhancement to edn interface to various blocks #7667

Open tjaychen opened 3 years ago

tjaychen commented 3 years ago

Right now, most blocks assume they are async to edn (see prim_edn_req).

This is a simplification since modules themselves don't really know if it is async to the edn (the top level integration information can be different).

This means for some modules (kmac, keymgr) the async handling is redundant, since their primary clock may actually be synchronous to edn.

It's worth adding some top level tooling support in the future that allows modules to bypass this async handling when it is not needed.

tjaychen commented 3 years ago

@dsandrs

tjaychen commented 3 years ago

@mwbranstad just FYI. We don't need to change anything for edn, but just wanted you to know.

andreaskurth commented 1 year ago

Triaged for edn, assigning https://github.com/lowRISC/opentitan/labels/Type%3AFutureRelease because I think this is out of scope for the current release.