search

pulp-platform / cheshire

A minimal Linux-capable 64-bit RISC-V SoC built around CVA6
Other
190 stars 42 forks source link

hw: Add native iDMA 2D capabilities, tune interconnect #73

Closed thommythomaso closed 1 year ago

thommythomaso commented 1 year ago

LGTM.

If I may nitpick, the DIF header looks a bit verbose, boiler-platey, and maintenance-unfriendly for my taste. Since it can be prefixed through macros, maybe it makes sense to include it in and get it from the iDMA repo directly?

Not a pressing issue though.

Wilco. I will move as much as I can from the systems into the iDMA with the next release.