Skip to content

Conversation

@rivos-eblot
Copy link

No description provided.

@rivos-eblot rivos-eblot changed the title tools: update file parsing to support latest HW defitinions. tools: update file parsing to support latest HW definitions. Dec 4, 2025
Copy link

@loiclefort loiclefort left a comment

Choose a reason for hiding this comment

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

LGTM

Invalid default values are only used in RTL and QEMU for buffered partitions.
However in the RTL and HJSON files, they are defined for -all- partitions.

Up to know, a heuristic was used to known whether these invalid default values
need to be converted into a QEMU property, as for all non-buffered partition,
the only non-zero field was the digest field.

However in some recent RTL versions, some multibit bool values are made their
way to the unused invalid default value constants definitions, which means
relying on the default values (non-zero) is no enough. The actual type of the
partition needs to be considered.

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
… are supported

This is a new syntax introduced in recent HW files.

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
@rivos-eblot rivos-eblot merged commit ead5323 into lowRISC:ot-10.1.0 Dec 5, 2025
8 of 10 checks passed
@rivos-eblot rivos-eblot deleted the dev/ebl/tools_update branch December 5, 2025 12:30
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.

3 participants