Spatz Core Complex Integration#51
Merged
Merged
Conversation
ad26d17 to
9915112
Compare
AlessandroNadalini
requested changes
Apr 17, 2026
AlessandroNadalini
left a comment
Member
There was a problem hiding this comment.
The PR is in good shape, however it requires some changes: most of them are related to the copyright headers, but there are also some points to be clarified, e.g. the need of a function automatic within the magia_tile_pkg.
After the suggested changes, it will be green light from my side.
AlessandroNadalini
approved these changes
Apr 17, 2026
AlessandroNadalini
left a comment
Member
There was a problem hiding this comment.
Comments have been implemented, green light from my side.
11e994f to
06d4f3f
Compare
FrancescoConti
requested changes
May 5, 2026
FrancescoConti
left a comment
Member
There was a problem hiding this comment.
Ultra-minor change requested: non-new files should extend, not replace the year in the copyright header
FrancescoConti
approved these changes
May 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Spatz Core Complex Integration
TODO List before merging
mainbefore mergingHardware Changes
New RTL Modules
spatz_cc_wrapper.sv: Main wrapper instantiating Snitch scalar core + Spatz vector coprocessor with HCI/OBI connectivityobi_slave_ctrl_spatz.sv: OBI slave interface with 7 memory-mapped control registers (CLK_EN, READY, START, TASKBIN, DATA, RETURN, DONE) at base 0x1700spatz_bootrom.sv: Minimal 3-instruction bootROM (load from TASKBIN, jump to dispatcher, NOT combinatorial)eu_direct_cut.sv: Direct connection cut to Event UnitProtocol Converters (hw/tile/converters/)
reqrsp2obi.sv,reqrsp64_to_obi32.sv: Req/Rsp to OBI conversiontcdm2hci.sv,tcdm2hci_atomic.sv: TCDM to HCI conversion with/without atomicstcdm2obi.sv: TCDM to OBI conversiontcdm64_to_dual_hci32.sv,tcdm64_to_dual_hci32_atomic.sv: 64-bit to dual 32-bit HCItcdm64_to_dual_tcdm32.sv: 64-bit to dual 32-bit TCDMModified RTL
magia_tile.sv: Integrated Spatz CC, added HCI port array, OBI slave interface, clock gating for Spatz CCmagia_tile_pkg.sv: Added Spatz configuration parameters (VLEN, N_FPU, RVD, RVF, XDIVSQRT), address definitionsmagia_event_unit.sv: Added SPATZ_DONE event input and EU_SPATZ_DONE_MASK (bit 16)Icache: Switched to version without non-resettable flip-flopsBuild System
Makefile Changes
*_TASKsymbols, automatically builds Spatz binary when neededSPATZ_BINARY_STARTand task offsets*_task.c→int *_task(void)→*_TASKsymbol in headerScripts
Bender
Note: This is a large PR touching many files with significant architectural changes. The
spatz/directory contains all Spatz-specific software (bootrom, runtime, tasks, build system). Additional minor modifications and cleanups may be present throughout the codebase.