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

pacman.cpp: upgrade clubpacm and clubpacma to working #7337

Merged
merged 3 commits into from
Oct 9, 2020

Conversation

devinacker
Copy link
Contributor

These two commits add/improve a few things for clubpacm and clubpacma:

  • Multiplexed P1/P2 joystick inputs for Double Command mode in clubpacm
  • Corrections to other inputs (including dip switches)
  • Simple protection(?) latch needed by at least clubpacma

Both of these sets should now be in good shape, aside from some bugs in the games themselves (which I tried to document in comments where possible, e.g. the broken free play setting).

The only thing I'm not really sure about is whether clubpacm is supposed to support third and fourth joysticks for the other side of a cocktail table, or if it's even intended to be playable in cocktail mode in the first place...

@devinacker
Copy link
Contributor Author

As for the other set of this game (clubpacmb / "set 2"), I suspect that the program ROM for this one might just be a bad dump of the one from set 1. The two ROMs are byte-for-byte identical, except all byte values between 0x40-0xBF in set 1 have their two most significant bits missing in set 2 (i.e. they all became 0x00-0x3F), which to me suggests the EPROM might just be faulty rather than encrypted. What do you folks think?

Copy link
Member

@cuavas cuavas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The “encrypted” set is probably just bit rot, it sounds more like data loss than encryption.

src/mame/drivers/pacman.cpp Outdated Show resolved Hide resolved
@Osso13
Copy link
Member

Osso13 commented Oct 8, 2020

I looked a bit at clubpacmb some time ago and discussed it a bit with another dev. I agree it's probably a bad dump.

@cuavas cuavas merged commit 0cb876c into mamedev:master Oct 9, 2020
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

3 participants