Skip to content

ql1xxx: Fix various PCI emulation issues#6949

Merged
OBattler merged 1 commit into
86Box:masterfrom
disean:ql_pci_fix
Mar 17, 2026
Merged

ql1xxx: Fix various PCI emulation issues#6949
OBattler merged 1 commit into
86Box:masterfrom
disean:ql_pci_fix

Conversation

@disean

@disean disean commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Use the system ID [1077:0005] for the 1080 boards, fixes boot with NT4 driver v7.13.
  • Fix a bug that caused the isp_rev and flash_type variables to be invalid.
  • Fix incorrect PCI status indication for the 1020/1040 chips.
  • Implement the PCI configuration space decode logic for the 1020/1040 chips.
  • Rename ISP1020 to ISP1020A to be more accurate.

Checklist

  • Closes #xxx
  • I have tested my changes locally and validated that the functionality works as intended
  • I have discussed this with core contributors already
  • This pull request requires changes to the ROM set
    • I have opened a roms pull request
  • This pull request requires changes to the asset set
    • I have opened an assets pull request

References

- Use the system ID [1077:0005] for the 1080 boards, fixes boot with NT4 driver v7.13.
- Fix a bug that caused the isp_rev and flash_type variables to be invalid.
- Fix incorrect PCI status indication for the 1020/1040 chips.
- Implement the PCI configuration space decode logic for the 1020/1040 chips.
- Rename ISP1020 to ISP1020A to be more accurate.
@OBattler OBattler merged commit 322a2b3 into 86Box:master Mar 17, 2026
43 of 44 checks passed
@disean disean deleted the ql_pci_fix branch March 17, 2026 15:34
@sskras

sskras commented Mar 19, 2026

Copy link
Copy Markdown
  • fixes boot with NT4 driver v7.13

@disean, since you mention the booting process, maybe you know the answer:

Can NT4 boot from the CD-ROM attached to a QL10x0 board (or whatever it supports) by default / out of the box / without installing any additional drivers?

EDIT: I mean the physical machines.

@disean

disean commented Mar 19, 2026

Copy link
Copy Markdown
Contributor Author

@sskras Yes, because it has the ISP1020/ISP1040 driver built-in.

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