-
Notifications
You must be signed in to change notification settings - Fork 118
higan: preliminary support for ZX Spectrum 48k & ZX Spectrum 128 #144
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did a preliminary, mostly-superficial sweep over the code. Looks pretty good!
|
||
auto in(uint16 address) -> uint8; | ||
auto out(uint16 address, uint8 data) -> void {} | ||
}; No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing newline. Also, shouldn’t headers in Higan be .hpp?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good spot, will fix shortly. Old habits :p
higan/spec/tape/tape.hpp
Outdated
|
||
auto save() -> void; | ||
|
||
//private: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Guessing this is intended to be uncommented at some point?
higan/spec/ula/color.cpp
Outdated
} | ||
|
||
unreachable; | ||
} No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another missing newline here.
Why does this PR include the ZX Spectrum BIOS ROMs? Aren't those copyrighted? |
Amstrad gave permission for these roms to be included with emulators, many years ago, see https://groups.google.com/g/comp.sys.amstrad.8bit/c/HtpBU2Bzv_U/m/HhNDSU3MksAJ?pli=1 for details They were able to do this because Amstrad purchased the rights for the ZX Spectrum family from Sinclair way back when. (The +2 and all later models are Amstrad produced) The key point is
This is why all ZX Spectrum emulators include, and are able to include the firmware/bios roms. |
Fair enough, though this should probably be stated somewhere in the README or something :P |
Status updates on this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's nice to see a new system coming to Higan :)
This implements preliminary support of the ZX Spectrum 48k and ZX Spectrum 128 computers into higan.
This core should run most ZX Spectrum games without issues but may struggle with timing-sensitive homebrew and demos.
The kempston controller interface is emulated to allow joystick input in supported games.
Known Issues:
Games must be imported with Icarus and must be mono .wav (PCM) audio.