Skip to content

Conversation

deltecent
Copy link
Contributor

@deltecent deltecent commented Sep 1, 2025

This PR contains the following:

Adds a new SD Systems SBC200 device.
Adds a new SD Systems VFII VersaFloppy II device.
Adds ability to load HEX files at a specified address.
Adds support for SD Systems ExpandoRAM banked memory.
Adds improved DSK image support to the WD179X device.
Adds support for formats that don't include the FC index mark to WD179X WRITE_TRACK command.
Fixed WD179X VERIFY register.

SD Systems uses the high bit of the SIDE byte of a disk's 6-byte sector ID field to identify a double-sided disk. A DSMASK register was added to WD179X device. When a READ ADDRESS command is issued, the value of this register is ORed with the SIDE number. For example, if DSMASK is set to 80H, 80H is returned for Side 0 and 81H is returned for Side 1.

This PR passes Howard's AltairZ80 tests.

Additional tests were added here:

https://github.com/deltecent/altairz80-tests/

Additional packages were added here:

https://github.com/deltecent/altairz80-packages/

@deltecent deltecent force-pushed the sdsys branch 7 times, most recently from f3c3cf9 to 48e1673 Compare September 2, 2025 01:11
@markpizz
Copy link
Contributor

markpizz commented Sep 2, 2025

Patrick,

As you keep force pushing your changes, please change the commit message to something more appropriate.

Thanks,

Mark

@deltecent
Copy link
Contributor Author

Sorry about that... trying to fix stuff failing in the automatic build checks.

@deltecent
Copy link
Contributor Author

Included PR #492 in scp.c
Replaced binary notation with hex notation in s100_scb200.c

This PR contains the following:

Adds a new SD Systems SBC200 device.
Adds a new SD Systems VFII VersaFloppy II device.
Adds ability to load HEX files at a specified address.
Adds support for SD Systems ExpandoRAM banked memory.
Adds improved DSK image support to the WD179X device.
Adds support for formats that don't include the FC index mark to WD179X WRITE_TRACK command.
Fixed WD179X VERIFY register.
Updated CMakeList.txt
Included PR open-simh#492 in scp.c
Replaced binary notation with hex notation in s100_scb200.c
Fixed Warnings
Fixed DTR comments

This PR passes Howard's AltairZ80 tests.

Additional tests were added here:

https://github.com/deltecent/altairz80-tests/

Additional packages were added here:

https://github.com/deltecent/altairz80-packages/
@pmetzger
Copy link

Why did this get closed?

@deltecent
Copy link
Contributor Author

This PR got stalled. I think it is too many changes touching too much stuff for one PR. It also touches the same code as
#484 which is also stalled and generated another that hasn't made it through consensus yet. I am going to wait for some of these other PRs to make it through the pipe and then submit separate PRs. Sometimes they want only one commit. Sometimes more than one commit. I'm just not that good at Git to keep things in order and work on other branches while this one is in limbo. It is much easier to wait it out and start over.

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