Skip to content
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

Fixes for MACHINE_XT build #32

Merged
merged 2 commits into from
Feb 23, 2023
Merged

Fixes for MACHINE_XT build #32

merged 2 commits into from
Feb 23, 2023

Conversation

640-KB
Copy link
Contributor

@640-KB 640-KB commented Feb 23, 2023

Fixes a few small issues for IBM PC/XT build:

  1. Standard PPI equipment switch is 1 to read SW5-8 (opposite of FE2010A).
  2. Some option ROMs can trash IMR (including IBM Xebec FD controller). Preserve and restore after ROM scan.
  3. MDA can only be video mode 7. Some "badly behaved" programs may attempt to change to a video mode other than 7 with MDA installed (causing what appears to be a crash).

mov ch,0B0h ; MDA video memory segment
mov dl,0B4h ; port for MC6845 CRTC address register
inc al
inc ax
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saves 1 byte :)

@skiselev
Copy link
Owner

That's great stuff! Getting the XT version working was on my "to do" list already :)

I am wondering if you've actually tried running the code on a real XT machine?

@skiselev skiselev merged commit 4ec3c1c into skiselev:master Feb 23, 2023
@640-KB
Copy link
Contributor Author

640-KB commented Feb 23, 2023

That's great stuff! Getting the XT version working was on my "to do" list already :)

Yes, saw your post on vcfed about it and thought I'd give it a try and see!

I am wondering if you've actually tried running the code on a real XT machine?

I did. I think there are still some things, potentially related to DRAM refresh. I'll keep plugging and get back to you.

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.

None yet

2 participants