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

Fix wrong pitch for newhunterb, newhunterc, ndongmul, newhunterd, luckyseven sevenlnd #12584

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

flama12333
Copy link
Contributor

@flama12333 flama12333 commented Jul 16, 2024

will fix the wrong pitch

increase ymsnd clock to 12'000'000  and guessed divide to 3.
this will solve wrong pitch for New Hunter, New Dongmul Dongmul, Seven land and Lucky seven.
@flama12333 flama12333 closed this Jul 16, 2024
@flama12333 flama12333 reopened this Jul 16, 2024
@flama12333
Copy link
Contributor Author

Im gonna to revert change for original treasure island and super treasure island except for bootleg.
the fix for wrong pitch will be applied for new hunter. seven land and lucky seven.

@cuavas
Copy link
Member

cuavas commented Jul 16, 2024

What are you basing this on? What oscillators/crystals/resonators are present on the boards, and where are they located?

@flama12333
Copy link
Contributor Author

sorry for late reply
this is for newhunter newhunterb, newhunterc, newhunterd, ndongmul, sevenlnd, luckyseven
in the pcb recording seem to sound diferent pitch.
first pcb there are two xtal 12.000 mhz at x1
at x2 im not sure about second xtal since is covered.
second and third pcb has 1x 12 mhz xtal at y1

only for newhunterb, newhunterc, ndongmul, newhunterd, luckyseven sevenlnd.
@flama12333 flama12333 changed the title Fix wrong pitch for tisub and stbsub Fix wrong pitch for newhunterb, newhunterc, ndongmul, newhunterd, luckyseven sevenlnd Jul 16, 2024
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