Skip to content

Add skeleton for Casio CPS-2000 #13591

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

Merged
merged 3 commits into from
Apr 16, 2025
Merged

Add skeleton for Casio CPS-2000 #13591

merged 3 commits into from
Apr 16, 2025

Conversation

devinacker
Copy link
Contributor

Also some fixes to the uPD934 and MSM6200.

New NOT_WORKING machine

Casio CPS-2000 [BCM, Devin Acker]

----------
Casio CPS-2000 [BCM, Devin Acker]
MSM6200(config, "kbd").irq_cb().set_inputline(m_mcu, MCS48_INPUT_IRQ);
MSM6200(config, "kbd", 2'470'000).irq_cb().set_inputline(m_mcu, MCS48_INPUT_IRQ);
Copy link
Member

Choose a reason for hiding this comment

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

Where’s this frequency coming from?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There's an oscillator on the main board that goes out to the keyboard PCB (it's hard to make out in the schematic because of the poor scan quality, but the parts list makes it more clear)
image
image

Copy link
Member

Choose a reason for hiding this comment

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

You’re supposed to use _XTAL markup for those. I’ll fix it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Even for ceramics?

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, if it’s a crystal/resonator, use _XTAL markup. If it’s an RC/LC circuit or something, just put in the frequency.

Copy link
Member

Choose a reason for hiding this comment

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

Yes xtal.cpp includes ceramic oscillators.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good to know, thanks.

@cuavas
Copy link
Member

cuavas commented Apr 15, 2025

The code looked fine after a quick skim. I’d just like clarification on where that MSM6200 clock frequency comes from.

@cuavas cuavas merged commit ec7eb50 into mamedev:master Apr 16, 2025
5 checks passed
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