Skip to content

Commit

Permalink
ci: Fix elaboration flow (#188)
Browse files Browse the repository at this point in the history
* deps: Use iDMA rev with generated sources

* ci: Enable elaboration again

* ci: Don't install iDMA python dependencies
  • Loading branch information
fischeti authored Aug 28, 2024
1 parent 198e990 commit a58a803
Show file tree
Hide file tree
Showing 8 changed files with 9 additions and 23 deletions.
1 change: 0 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@ jobs:
- name: Build Hardware
working-directory: target/snitch_cluster
run: |
pip install -r $(bender path idma)/requirements.txt
make CFG_OVERRIDE=cfg/github-ci.hjson VLT_JOBS=1 bin/snitch_cluster.vlt
- name: Run Tests
working-directory: target/snitch_cluster
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ jobs:
- name: Generate RTL sources
working-directory: target/snitch_cluster
run: |
pip install -r $(bender path idma)/requirements.txt
make rtl
# For some reason, the checkout is done by a different user,
# than that running `git diff` (root, possibly due to Docker).
Expand Down
9 changes: 4 additions & 5 deletions .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ variables:

before_script:
source iis-setup.sh
pip install $($BENDER path idma)/requirements.txt

##############
# Build docs #
Expand Down Expand Up @@ -159,7 +158,7 @@ snitch-cluster-mchan-vsim:
# Non-free #
############

# nonfree:
# script:
# - make nonfree
# - make elab
nonfree:
script:
- make nonfree
- make elab
6 changes: 3 additions & 3 deletions Bender.lock
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ packages:
dependencies:
- common_cells
idma:
revision: 0a0c113434aee743923d9e85631de009b4f00847
revision: 1a42da9d5c76f1cff0bcdaaacc474ed85631d734
version: null
source:
Git: https://github.com/pulp-platform/iDMA
Expand All @@ -83,8 +83,8 @@ packages:
- obi
- register_interface
obi:
revision: c2141a653c755461ff44f61d12aeb5d99fc8e760
version: 0.1.3
revision: 5321106817e177d6c16ecc4daa922b96b1bc946b
version: 0.1.5
source:
Git: https://github.com/pulp-platform/obi.git
dependencies:
Expand Down
2 changes: 1 addition & 1 deletion Bender.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ dependencies:
tech_cells_generic: { git: https://github.com/pulp-platform/tech_cells_generic, version: 0.2.11 }
riscv-dbg: { git: https://github.com/pulp-platform/riscv-dbg, version: 0.8.0 }
cluster_icache: { git: https://github.com/pulp-platform/cluster_icache.git, version: 0.1.0 }
idma: { git: https://github.com/pulp-platform/iDMA, rev: snitch-tracing }
idma: { git: https://github.com/pulp-platform/iDMA, rev: __deploy__9cbcd30__snitch-tracing }

export_include_dirs:
- hw/reqrsp_interface/include
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ ROOT = $(dir $(abspath $(lastword $(MAKEFILE_LIST))))
############

NONFREE_REMOTE ?= [email protected]:pulp-restricted/snitch-cluster-nonfree.git
NONFREE_COMMIT ?= 214380a8c6af4cdd3d53a5b74cfaf462da877be2
NONFREE_COMMIT ?= e30961e20a23a76442da27d2ba07c9fe20f3b575
NONFREE_DIR = $(ROOT)/nonfree

all: nonfree
Expand Down
1 change: 0 additions & 1 deletion iis-setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ source .venv/bin/activate
# occurring when the /tmp folder is filled by other processes.
mkdir tmp
TMPDIR=tmp pip install -r python-requirements.txt
TMPDIR=tmp pip install -r $($BENDER path idma)/requirements.txt
rm -rf tmp

# Install local packages in editable mode.
Expand Down
10 changes: 0 additions & 10 deletions target/snitch_cluster/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -151,19 +151,9 @@ include $(ROOT)/target/snitch_cluster/sw.mk
# RTL #
#######

# Include iDMA Makefile to generate prerequisite iDMA sources
include $(shell $(BENDER) path idma)/idma.mk

GENERATED_RTL_SOURCES = $(PERIPH_DIR)/snitch_cluster_peripheral_reg_top.sv
GENERATED_RTL_SOURCES += $(PERIPH_DIR)/snitch_cluster_peripheral_reg_pkg.sv
GENERATED_RTL_SOURCES += $(GENERATED_DIR)/snitch_cluster_wrapper.sv
GENERATED_RTL_SOURCES += $(IDMA_ROOT)/target/rtl/idma_inst64_top.sv
GENERATED_RTL_SOURCES += $(IDMA_ROOT)/target/rtl/include/idma/tracer.svh

# Add dependency on DMA header files
VSIM_SOURCES += $(IDMA_ROOT)/target/rtl/include/idma/tracer.svh
VLT_SOURCES += $(IDMA_ROOT)/target/rtl/include/idma/tracer.svh
VCS_SOURCES += $(IDMA_ROOT)/target/rtl/include/idma/tracer.svh

.PHONY: rtl clean-rtl

Expand Down

0 comments on commit a58a803

Please sign in to comment.