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

Namco system 246/256 support #6

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

israpps
Copy link
Member

@israpps israpps commented Apr 12, 2024

Pull Request checklist

Note: these are not necessarily requirements

  • I reformatted the code with clang-format
  • I checked to make sure my submission worked
  • I am the author of submission or have permission from the original author
  • Requires update of the PS2SDK or other dependencies
  • Others (please specify below)

TODO:

  • fix issue with font text not rendering
  • fix detection of almost everything
  • fix issue with rom dumping (Arcade ROMDRV Has some special stuff inside?)
  • whatever i'm missing (and I know there will be)

@israpps israpps marked this pull request as draft April 12, 2024 16:40
@AKuHAK
Copy link
Member

AKuHAK commented Apr 12, 2024

@israpps I am heavily rewriting ps2ident, better use this branch as a base: https://github.com/AKuHAK/PS2Ident/tree/new

@AKuHAK
Copy link
Member

AKuHAK commented Apr 12, 2024

Fonts are embedded now, also dumping is done thrugh EE memory, so it should be more reliable. Also there were some random memory leaks, maybe some problems were due to that leaks.

@israpps
Copy link
Member Author

israpps commented Apr 12, 2024

Very well.

I was worried at first of differences on ROMDRV. but there seem to be none. will test on the new branch soon

@israpps
Copy link
Member Author

israpps commented May 24, 2024

I will leave this as a note here.

Since DVDPLAYER does not exist on COH, make PS2IDENT load rom0:ACDEV. so the alreay existing code dumps the arcade flash chip from the arcade board when possible

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.

2 participants