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
nec/pce.cpp, nec/pce_cd.cpp: Various updates #11999
Conversation
cam900
commented
Feb 2, 2024
•
edited
edited
- Use device_mixer_interface rather than using hard-coded speaker tag
- Reduce defines
- Remove outdated comments
- Use lowercase hexadecimal values
- Remove nec/pce_m.cpp, Move pce_m.cpp codes into nec/pce.cpp
- Use device_mixer_interface rather than using hard-coded speaker tag - Reduce defines - Remove nec/pce_m.cpp, Move pce_m.cpp codes into nec/pce.cpp
What’s the motivation for moving it into src/devices/machine? Is it used for anything besides the PC-Engine? |
File is currently a mixup of PC8801-30 drive, the specific (and opaque) PC Engine SCSI controller and the rest of the expansion unit bus & i/f. afaik PC8801-31 SCSI isn't much different beyond relying on SEL line alone (and doing the rest internally), a preliminary core i/f (without the controller part, maybe at ID 7): Bottom line: not convinced about the move either, it's essentially a NEC i/f that can stay in |
…Remove unused macro
* nec/pce_cd.cpp: Use device_mixer_interface rather than using hard-coded speaker tags. * Merged nec/pce_m.cpp into nec/pce.cpp. * Reduced preprocessor macros, removed outdated comments, changed heaxadecimal digits to lowercase.
* nec/pce_cd.cpp: Use device_mixer_interface rather than using hard-coded speaker tags. * Merged nec/pce_m.cpp into nec/pce.cpp. * Reduced preprocessor macros, removed outdated comments, changed heaxadecimal digits to lowercase.