Open colluca opened 2 months ago
LGTM! What's the problem exactly with the overlapping snitch_cluster
target in this repo and the DMA?
LGTM! What's the problem exactly with the overlapping
snitch_cluster
target in this repo and the DMA?
Basically the iDMA uses snitch_cluster
as a Bender target for the Snitch DMA sources. We were using the same target for the snitch_cluster_wrapper
sources. Occamy, and derived systems, need Snitch's iDMA sources and will thus need to include the snitch_cluster
Bender target, but they should not include the wrapper (this failed to compile in Occamy). Thus we need distinct Bender targets.
Implements several changes to bump the Snitch cluster dependency in Occamy:
snitch_cluster
Bender target, since this is now used by the DMA and required in Snitch-based systems, tosnitch_cluster_wrapper
containing the sources which are only needed to simulate/synthesize thesnitch_cluster
out of contextROOT
toSNITCH_ROOT
, renamePLATFORM_HEADERS
toSNRT_HAL_HDRS
, renamesnitch_cluster_start.S
tosnrt.S
, makeMEMORY_LD
overridable)/*/*
to all include directories would lead to "No rule to make target" errors.verible
, which was not correctly installed in container, and is anyways unused in Snitchhw/snitch_dma
fromCODEOWNERS
, outdated after iDMA integrationdeploy-docs
job to CI workflow, to properly encode thebuild-docker
->deploy-docs
dependency