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

Added Pencil II support. #1637

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

Added Pencil II support. #1637

wants to merge 1 commit into from

Conversation

cdoty
Copy link

@cdoty cdoty commented Sep 5, 2024

Created a Pencil II core based on the ColecoVision core. The timings are different as the system uses a 10.738 mhz crystal instead of the 7 mhz on the Colecovision.

@LukeUsher
Copy link
Member

LukeUsher commented Sep 5, 2024

Given how similar these platforms are, I can't help but think it would it be better to make it an alternate configuration of the ColecoVision core rather than an entirely new one.

Take a look at how Pocket Challenge V2 is an alternate configuration in the WonderSwan core,

Thoughts/feedback welcome

Also this seems to lack support for the keyboard? Given that the Pencil II is a computer, don't you think that would be important to add?

@cdoty
Copy link
Author

cdoty commented Sep 5, 2024

That would be better. Let me have a look and I'll see if I can adapt it.

@LukeUsher
Copy link
Member

Did you manage to take a look at this (merged cores)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants