Fix IP28 boot chime - #178
Merged
Merged
Conversation
Report R10000 revision 2.5 in both CPU models so the IP28 PROM passes its boot-tune revision check. Support byte access to HPC3 PBUS DMA registers. The IP28 PROM starts audio DMA using four byte stores to the control register, which were previously ignored. Add regression tests for CPU identity across reset, DMA startup and completion, byte lanes, and interrupt side effects. Signed-off-by: iblowmymind <28228415+iblowmymind@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The IP28 boot chime was silent for two reasons: the emulated R10000
reported revision 0.0, causing the PROM to skip playback, and the
PROM's byte stores to PBUS DMA control registers were ignored.
Report R10000 revision 2.5 in both CPU models and support PBUS DMA
register byte access, including activation and status polling.
Validation:
silence gaps.