-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[WIP] HW/DSPHLE: Refactor Old AX Wii into a separate class #8529
base: master
Are you sure you want to change the base?
Conversation
This fixes Old AX Wii games having no audio when compiled under VS2019. This also includes some minor code cleanup and moving a function to avoid duplication.
a1cf193
to
40b6c93
Compare
What's the status of this PR, now that #8525 is merged? |
@@ -113,6 +113,14 @@ struct PBUpdates | |||
u16 data_lo; | |||
}; | |||
|
|||
// Old AX Wii uses update data, but a block is 3ms. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's been a while that I worked on the DSP, but this comment doesn't really make sense to me in its current form?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not really clear to me either what update data is. If someone could provide a better explanation I'd be happy to put it in the comment.
{ | ||
volume = m_cmdlist[curr_idx++]; | ||
u32 addresses[6] = { | ||
(u32)(m_cmdlist[curr_idx + 0] << 16) | m_cmdlist[curr_idx + 1], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we should clean those up into proper casts while we're at it.
Based on #8525