Skip to content
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.

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

Darjeeling updates #71

Merged

Conversation

loiclefort
Copy link

This batch includes updates to reset management and several bug fixes (OTP DAI, SPI Device busy bit, MISA value, CSRNG cmd.flag0)

rivos-eblot and others added 30 commits July 5, 2024 15:52
…t functions

These functions enables printing the AS and/or the flatview of a single AS.
For example, this can be used to report the flatview exposed to a single vCPU.

Signed-off-by: Emmanuel Blot <[email protected]>
There is no way to disable them from the VM itself, and those are mostly
useless messages

Signed-off-by: Emmanuel Blot <[email protected]>
Log message classification can now be configured with regular expressions.

Signed-off-by: Emmanuel Blot <[email protected]>
…IRQs

IRQs that are not connected to a sink can now be intercepted.

Signed-off-by: Emmanuel Blot <[email protected]>
Also add a new verbosity option for increasing QEMU virtual comm port
verbosity.

Signed-off-by: Emmanuel Blot <[email protected]>
…-bit

Don't use MemoryRegionSection.size directly, use int128_getlo to convert it to a
64-bit value.

Signed-off-by: Loïc Lefort <[email protected]>
Use memory_region_size to get a 64-bit memory region size instead of calling
int128_getlo.

One int128_getlo(mr->size) remains in ot_ibex_wrapper_dj.c because
memory_region_size argument is not declared const.

Signed-off-by: Loïc Lefort <[email protected]>
Ibex sets MISA.X if any zb* extension is enabled.

Signed-off-by: Loïc Lefort <[email protected]>
The cast to bool takes precedence over the &, so we need parens.
Otherwise, the busy bit is always evaluated as false.

Signed-off-by: Noah Moroze <[email protected]>
Payload length should be reset even if the incoming command does not have
a payload.

Signed-off-by: Emmanuel Blot <[email protected]>
Verbose logging messages are only reported if an error is detected

Signed-off-by: Emmanuel Blot <[email protected]>
… back-end

Also fix buffered partition digest computation with DAI DIGEST

Signed-off-by: Emmanuel Blot <[email protected]>
Non-multibitbool values should not be considered as an error.

Signed-off-by: Emmanuel Blot <[email protected]>
…RQ line

Reset request is now managed with an IRQ line, not a dedicated function call.

Signed-off-by: Emmanuel Blot <[email protected]>
It is up to the machine to manage the reset signal.

Signed-off-by: Emmanuel Blot <[email protected]>
Copy link

@rivos-eblot rivos-eblot left a comment

Choose a reason for hiding this comment

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

LGTM (reviewed internally)

@loiclefort loiclefort merged commit 35cbf60 into lowRISC:ot-darjeeling-9.0.1 Jul 5, 2024
5 checks passed
@loiclefort loiclefort deleted the ot-dj-9.0.1-updates branch July 5, 2024 15:24
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