Skip to content

Removed RI5CY and re-enabled CV32E40X in memory-mapped mode - #92

Open
luca24balboni wants to merge 3 commits into
mainfrom
lb/core-rework
Open

Removed RI5CY and re-enabled CV32E40X in memory-mapped mode#92
luca24balboni wants to merge 3 commits into
mainfrom
lb/core-rework

Conversation

@luca24balboni

Copy link
Copy Markdown
Contributor

This PR

  • Removes the RI5CY core option: core= is now CV32E40P (OpenHW, FondazioneChipsIT/cv32e40p ) or CV32E40X
  • Removes the XIF accelerator path (xif_inst_dispatcher, the iDMA and FractalSync instruction decoders, idma_ctrl, and the six sw/tests/xif_tests): RedMulE, iDMA and FractalSync are now memory-mapped for both cores, so with CV32E40X the OBI crossbar grows from 5 to 7 subordinates and from 6 to 9 address rules and obi_mem_array_idx_e collapses into a single enum; XIF survives only as the link to fpu_ss.
  • Splits the core data-port converters and their request/response structs per core flavour (cv32e40p_* / cv32e40x_*) and types the cluster data channel on CV32E40P: the cluster cores are always CV32E40P even when the control core is CV32E40X

Comment thread hw/tile/magia_tile_pkg.sv Outdated
Comment thread hw/tile/magia_tile_pkg.sv Outdated

@AlessandroNadalini AlessandroNadalini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall legitimate, just a couple of comments to be addressed.

Comment thread sw/tests/fpu_test.c
Comment thread Makefile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants