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

nmk/ddealer.cpp: upgraded interrupts trigger system based on PROM contents #12620

Merged
merged 2 commits into from
Sep 5, 2024

Conversation

sergiopolog
Copy link
Contributor

Same work done on this PR for nmk/nmk16.cpp driver:

#12562

is now applied in nmk/ddealer.cpp

Double Dealer uses the same "hires" screen resolution than several games in the nmk16 driver, and the same interrupt system based on PROM contents, so the same config is applied to it.

Pixel Clock was confirmed on real hw, too.

@cam900
Copy link
Contributor

cam900 commented Jul 27, 2024

nmk/quizpani.cpp uses similar hardware, so it's possibly applied a PROM based interrupt system too?

@sergiopolog
Copy link
Contributor Author

nmk/quizpani.cpp uses similar hardware, so it's possibly applied a PROM based interrupt system too?

I wasn’t aware that quizpani has similar hw. By a quick look at the code, it apparently uses the same screen resolution (“hi-res”) and same interrupts: IRQ1 for music and IRQ4 each VBlank.
Also PROMs are dumped, so I could definitely check it and apply if possible.

Thanks for pointing it out!

@sergiopolog
Copy link
Contributor Author

sergiopolog commented Sep 5, 2024

I think this could be merged as is, because quizpani board doesn't use NMK902 chip (PCB picture: https://i.imgur.com/kZ7sKBI.png ), in which rev-eng work the previous PR is based, so further investigation needs to be done for that case, and that requires to check actual pcb to see how the counters are implemented in this board without NMK902.

@cuavas cuavas merged commit 60a36e1 into mamedev:master Sep 5, 2024
5 checks passed
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