search

zeroasiccorp / umi

Universal Memory Interface (UMI)
Apache License 2.0
140 stars 9 forks source link

UMI to APB converter #147

Closed azaidy closed 3 months ago

azaidy commented 3 months ago

Sorry, my bad. I agree with Wenting, the old methodology of using "2" is a nice directional hint. I suggest the namge is changed to umi2apb

umi2apb apb2umi tl2umi...

Naming change done

azaidy commented 3 months ago

@aolofsson, @zephray A bit of detail on the latest commit:

  1. Removed UMI atomic support. We can add it at a later date if needed.
  2. Removing atomics resulted in pvalid_r looking identical to penable - so just eliminated pvalid_r and replaced instances of it with penable.
  3. pready was unqualified before which meant that anytime pready went high an outgoing resp was issued. This was fixed by qualifying pready with penable.