Skip to content

(WIP, DO NOT MERGE, breaks Dune music) Tried to fix DOTT having no music half the time (OPL3 issue)#2119

Draft
maximilien-noal wants to merge 1 commit intomasterfrom
fix/opl_delays
Draft

(WIP, DO NOT MERGE, breaks Dune music) Tried to fix DOTT having no music half the time (OPL3 issue)#2119
maximilien-noal wants to merge 1 commit intomasterfrom
fix/opl_delays

Conversation

@maximilien-noal
Copy link
Copy Markdown
Member

Description of Changes

Simulate 1.5us hardware latency in Opl3Fm port reads.

Rationale behind Changes

Makes OPL FM drivers from many games detect the OPL3 chip successfully.

In other words, DOTT starts consistently with music:

Suggested Testing Steps

Manually tested with DOTT. Covered with a unit test.

Spice86.C__Jeux_DOTT_TENTACLE.EXE.-.cycles_ms_.0.2026-04-27.23-18-52.mp4

@maximilien-noal maximilien-noal self-assigned this Apr 27, 2026
@maximilien-noal maximilien-noal added Music Music emulation related. Can be any form of Music (FM Synth, MT-32, or General MIDI, ...) compatibility Emulator compatibility with DOS apps hardware hardware device (can be virtual) emulation labels Apr 27, 2026
@maximilien-noal maximilien-noal force-pushed the fix/opl_delays branch 2 times, most recently from 8a34c30 to ec91d92 Compare April 28, 2026 06:32
Simulate 1.5us hardware latency in Opl3Fm port reads
@maximilien-noal maximilien-noal marked this pull request as draft April 28, 2026 18:43
@maximilien-noal maximilien-noal changed the title fix: OPL3 port read delay and remove legacy ASM tests (WIP, DO NOT MERGE, breaks Dune music) Tried to fix DOTT having no music half the time (OPL3 issue) Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility Emulator compatibility with DOS apps hardware hardware device (can be virtual) emulation Music Music emulation related. Can be any form of Music (FM Synth, MT-32, or General MIDI, ...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant