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

Arcade mappers #7

Open
chmcarro opened this issue Feb 1, 2021 · 14 comments
Open

Arcade mappers #7

chmcarro opened this issue Feb 1, 2021 · 14 comments

Comments

@chmcarro
Copy link

chmcarro commented Feb 1, 2021

Unsupport mapper error for the following ROMs:
VS. Excitebike
VS. The Goonies
VS. Ice Climbers
VS. Ladies Golf
PlayChoice-10 The Goonies

@hcs64
Copy link
Owner

hcs64 commented Feb 2, 2021

I haven't thought much about the arcade hardware. Vs. System sounds like it has some significant differences from the NES, so this is probably out of scope. PlayChoice-10 sounds like it would be simpler, but there would be at least palette issues, DIP switches, coins, along with implementing a new mapper.

I don't have plans for either of these, but patches are welcome!

@hcs64 hcs64 changed the title Unsupported mapper error Arcade mappers Feb 2, 2021
@hcs64
Copy link
Owner

hcs64 commented Feb 2, 2021

Er, never mind, I hadn't read the PlayChoice-10 article carefully, there's a whole separate CPU and monitor involved. This is pretty unlikely to happen.

@hcs64 hcs64 closed this as completed Feb 2, 2021
@chmcarro
Copy link
Author

chmcarro commented Feb 2, 2021

These are actually converted to NES roms playable on NES.

https://nes.goondocks.se/patches2.php

@hcs64
Copy link
Owner

hcs64 commented Feb 2, 2021

I think they're still using mapper 99, though. Could you check? I'm not super enthusiastic about adding partial mapper 99 support for hacks.

@oRBIT2002
Copy link

Nice that you mentioned my website. :) I'm pretty sure I patched these to something else than mapper 99.

@hcs64
Copy link
Owner

hcs64 commented Feb 2, 2021

The readme for Excitebike_(VS)_[NES-hack_v1.2].zip says "It's still mapper 99 and uses four-screen VRAM layout," I don't know about the others.

@chmcarro
Copy link
Author

chmcarro commented Feb 2, 2021

Ice climbers change log:
- Mapper 99 removed! Patched to use Mapper 1 (MMC1)

@hcs64
Copy link
Owner

hcs64 commented Feb 2, 2021

Mapper 1 should work fine, thanks for the info. @chmcarro you may have an older version of the hack.

@chmcarro
Copy link
Author

chmcarro commented Feb 2, 2021

OK, interesting. I have the latest patch, but VS Ice climbers throws a different error, not the unsupported mapper error.

Bad opcode: $4b at PC=$c0a2, P=$27, A=$00, X=$00, Y=$00, S=$fd, cycle=216624

@hcs64
Copy link
Owner

hcs64 commented Feb 2, 2021

I opened another bug for the VS Ice Climber issue.

@oRBIT2002
Copy link

Make sure you're patching the right ROM, patching the wrong/bad ROM can perhaps cause this. Does it work in Mesen for example?

@chmcarro
Copy link
Author

chmcarro commented Feb 2, 2021

Name: VS. Ice Climber (VS) NES Patched v2.0b [FIX] Morgan Johansson.nes
Size: 81936 bytes (80 KiB)
CRC32: D732018E

Runs great in Mesen v0.9.9

@chmcarro
Copy link
Author

chmcarro commented Feb 2, 2021

@oRBIT2002 please disregard the Ice Climbers issue. It's a non-issue. I was mistakenly using your previous patch from 2013 (v2.0b). You have a patch from 2017 v2.1 that works great on this emulator. Sorry for the trouble.

I suppose the others use the incompatible mapper 99?

@hcs64
Copy link
Owner

hcs64 commented Feb 2, 2021

I'll leave this open, it may be feasible to add mapper 99 for hacks like this, but skip supporting the bits in the header that say the rest of the VS Unisystem or PlayChoice-10 is present.

@hcs64 hcs64 reopened this Feb 2, 2021
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

No branches or pull requests

3 participants