Skip to content

Conversation

@Lore0599
Copy link
Contributor

@Lore0599 Lore0599 commented Jul 5, 2025

System RDL support

This PR introduces initial support for generating SoC configuration registers using the SystemRDL tool. The goal is to replace the existing reggen tool with SystemRDL.

πŸ†• Added

  • βš™οΈ HARDWARE:
    • rdl : The rdl folder contains all the RDL files needed to correctly generate the SoC address map. It includes the SoC control registers (chimera_regs) as well as the top-level address map (chimera.rdl), which is used to generate the C-header with the full address map and helper structs.
    • chimera_top_wrapper: The top-level wrapper has been updated to convert the internal Cheshire register protocol to the APB protocol used to connect to the SystemRDL-generated registers.
  • πŸ–₯️ SOFTWARE:
    • chimera_addrmap.h: With the C-header support of SystemRDL, a new header file containing the full Chimera SoC address map has been generated.
    • offload.c: The offload helper functions have been refactored to use the new generated structs.
    • Regression Tests: All regression tests have been adapted to the new offload library.
  • 🎨 MAKEFILE:
    • chimera.mk: The makefile now includes a target to regenerate the SoC registers if necessary.
    • sw.mk: A new target has been introduced to regenerate the address map header.

@Lore0599 Lore0599 requested a review from micprog July 5, 2025 23:02
@Lore0599 Lore0599 force-pushed the lleone/system-rdl branch 5 times, most recently from 96c78fa to 2979c8d Compare July 7, 2025 19:27
@Lore0599 Lore0599 requested review from Xeratec and fischeti July 8, 2025 08:33
@Lore0599 Lore0599 force-pushed the lleone/system-rdl branch from ec07ad9 to ac9f120 Compare July 8, 2025 12:15
@Lore0599 Lore0599 force-pushed the lleone/system-rdl branch from f1a6e79 to d990517 Compare July 8, 2025 12:17
@Lore0599 Lore0599 marked this pull request as ready for review July 8, 2025 12:20
Copy link
Member

@Xeratec Xeratec left a comment

Choose a reason for hiding this comment

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

Looks good to me.

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