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

technos/ddragon.cpp: Added Double Dragon II layer priority PROM to clones #10784

Merged
merged 3 commits into from
Jan 5, 2023

Conversation

jotego
Copy link
Contributor

@jotego jotego commented Jan 4, 2023

Double Dragon II clones (and alternate versions) are missing one of the PROMs in the ROM section. I have checked that the systems work correctly using the same dump as in ddragon2.

In this commit, I have added that PROM to the ROM section. This matches what other drivers do in these circumstances. The comments suggest that a better dump may exist (although I doubt it).

src/mame/technos/ddragon.cpp Outdated Show resolved Hide resolved
@cuavas cuavas changed the title Double Dragon II clones use the same layer priority PROM technos/ddragon.cpp: Added Double Dragon II layer priority PROM to clones Jan 4, 2023
@cuavas
Copy link
Member

cuavas commented Jan 4, 2023

I have checked that the systems work correctly using the same dump as in ddragon2.

What do you mean by this? Have you confirmed that a PROM with the same contents is present on original boards for the clone versions? Or did you convert a board to the clone version and confirm that it works with the same priority PROM? In the latter case, these should really still be marked as bad dumps as they haven’t been confirmed on original boards.

@jotego
Copy link
Contributor Author

jotego commented Jan 5, 2023

Good point. It was the latter case. I have added that. I think the PR covers both @cuavas and @angelosa 's reviews now.

@cuavas cuavas merged commit 4a1b418 into mamedev:master Jan 5, 2023
@cuavas
Copy link
Member

cuavas commented Jan 5, 2023

Thanks.

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