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

sound/nes_apu.cpp: Can no longer read from anything but status register. #10644

Merged
merged 3 commits into from Dec 4, 2022

Conversation

0kmg
Copy link
Contributor

@0kmg 0kmg commented Dec 4, 2022

nintendo/nes_vt*: Aligned PnP APU code with core changes.
nintendo/dkong.cpp: Fixed dkong3b crash regression.

nintendo/nes_vt*: Aligned PnP APU code with core changes.
nintendo/dkong.cpp: Fixed dkong3b crash regression.
@0kmg
Copy link
Contributor Author

0kmg commented Dec 4, 2022

@happppp Thanks for fixing the dkong3 sound regression. I was working on doing something very similar that included the changes here. Also note dkong3b uses dkongjr hardware, so it doesn't need the new dkong3 init function.

@happppp
Copy link
Member

happppp commented Dec 4, 2022

dkong3b: Ah I missed that.
Does anything need the apu_w trampoline from this and rp2a03? If not, that can be removed too, instead do a m_apu->write directly.

@0kmg
Copy link
Contributor Author

0kmg commented Dec 4, 2022

How do we get the proper 0x15/0x17 offsets to nesapu::write without the trampoline? That's the only reason I couldn't remove it I think.

@happppp
Copy link
Member

happppp commented Dec 4, 2022

m_apu->write(0x15, data); ?

@0kmg
Copy link
Contributor Author

0kmg commented Dec 4, 2022

I'll try to see what I can do. I guess I forgot these VT PnPs are using 6502s and so they don't have to deal with the internal 2A03 map issue.

@0kmg
Copy link
Contributor Author

0kmg commented Dec 4, 2022

Ok, that's better. Sorry, I still need hand holding for simple things sometimes :)

@0kmg 0kmg marked this pull request as draft December 4, 2022 20:58
@0kmg 0kmg marked this pull request as ready for review December 4, 2022 21:08
@happppp
Copy link
Member

happppp commented Dec 4, 2022

I was looking for github actions, but I don't see it on your PRs.

@happppp happppp merged commit bfc3075 into mamedev:master Dec 4, 2022
@0kmg
Copy link
Contributor Author

0kmg commented Dec 4, 2022

Yeah, sorry. GitHub hobbled my account, disabling actions, and then refused to tell me why despite repeated questions. This account has never done anything but MAME PRs, so go figure.

@0kmg 0kmg deleted the apu-cleanup branch December 4, 2022 23:52
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.

None yet

2 participants