Skip to content

AltairZ80: Corrects bug in ICOM floppy device#524

Merged
rms47 merged 1 commit intoopen-simh:masterfrom
deltecent:icom
Apr 30, 2026
Merged

AltairZ80: Corrects bug in ICOM floppy device#524
rms47 merged 1 commit intoopen-simh:masterfrom
deltecent:icom

Conversation

@deltecent
Copy link
Copy Markdown
Contributor

A hobbyist found an original FDOS for 8" drives (FD360, FD3712) that has turned up a bug in both the FDC+ and AltairZ80 implementation of the FD3712.

In the FDC+, the data pointer into the 128 byte write "shift register" is reset upon receipt of a CLEAR command. The real hardware never resets the shift register, so software should not adjust the pointer either. AltairZ80 may need the same adjustment.

A hobbyist found an original FDOS for 8" drives (FD360, FD3712) that
has turned up a bug in both the FDC+ and AltairZ80 implementation of
the FD3712.

In the FDC+, the data pointer into the 128 byte write "shift register"
is reset upon receipt of a CLEAR command. The real hardware never
resets the shift register, so software should not adjust the pointer
either. AltairZ80 may need the same adjustment.
@pmetzger
Copy link
Copy Markdown

Howdy! I'm committing this to ZIMH, but in the process of doing that, I've noticed there are no Altair integration tests in the repository at all, no attempts to boot and run anything as a smoke test even. Do you have suggestions for an integration test? Like booting an operating system disk I can commit or some such? I don't really know the old S100 machines well.

@pmetzger
Copy link
Copy Markdown

I've committed this on my fork at pmetzger/zimh@5a16b98cb4 and as I said, I would appreciate help setting up an AltairZ80 integration test for the repository.

@deltecent
Copy link
Copy Markdown
Contributor Author

@pmetzger There are test programs for AltairZ80 located here: https://github.com/hharte/altairz80-tests

Packages are here: https://github.com/hharte/altairz80-packages

@pmetzger
Copy link
Copy Markdown

@deltecent Thank you so much! I'm importing those into ZIMH now!

@rms47 rms47 merged commit f5a6fd8 into open-simh:master Apr 30, 2026
16 of 19 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