Skip to content

tools: autoreg.py: add initial handling of compact multireg #243

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Oct 17, 2025
Merged

tools: autoreg.py: add initial handling of compact multireg #243

merged 2 commits into from
Oct 17, 2025

Conversation

@rivos-eblot
Copy link

@rivos-eblot rivos-eblot commented Oct 16, 2025 •
edited
Loading

Depending on an optional compact boolean, multireg can either be generated as multiple registers or multiple fields.

Ex with a multireg with count: 3

REG32(REGISTER_0, 0x00)
    SHARED_FIELD(REGISTER_FIELD, 0u, 1u)
REG32(REGISTER_1, 0x04)
REG32(REGISTER_2, 0x04)

or

REG32(REGISTER, 0x00)
    FIELD(REGISTER, FIELD_0, 0u, 1u)
    FIELD(REGISTER, FIELD_1, 1u, 1u)
    FIELD(REGISTER, FIELD_2, 2u, 1u)

@rivos-eblot rivos-eblot changed the title [ot] scripts/opentitan: autoreg.py: add initial handling of compact multireg tools: autoreg.py: add initial handling of compact multireg Oct 16, 2025
@rivos-eblot rivos-eblot force-pushed the dev/ebl/autoreg_compact branch 2 times, most recently from 590e062 to 3d82eab Compare October 17, 2025 07:59
AlexJones0
AlexJones0 approved these changes Oct 17, 2025
View reviewed changes
Copy link

@AlexJones0 AlexJones0 left a comment

Choose a reason for hiding this comment

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

LGTM, the CI failures in the regression should be unrelated (marked flaky by my separate PR), feel free to ignore.

@rivos-eblot
[ot] scripts/opentitan: autoreg.py: add initial handling of compact m…
8e46c0b 
…ultireg

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
@rivos-eblot
[ot] scripts/opentitan: autoreg.py: add missing padding for Tock regi…
99511e6 
…sters

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
@rivos-eblot rivos-eblot force-pushed the dev/ebl/autoreg_compact branch from 3d82eab to 99511e6 Compare October 17, 2025 10:44
@rivos-eblot rivos-eblot merged commit 9631037 into lowRISC:ot-9.2.0 Oct 17, 2025
9 checks passed
@rivos-eblot rivos-eblot deleted the dev/ebl/autoreg_compact branch October 17, 2025 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AlexJones0 AlexJones0 AlexJones0 approved these changes

@jwnrt jwnrt Awaiting requested review from jwnrt

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rivos-eblot @AlexJones0