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

fix: Use the pcnet ethernet device for FreeDOS #1587

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheMuso
Copy link
Contributor

@TheMuso TheMuso commented Feb 11, 2025

Description

FreeDOS does ship the Crynwr packet driver package, which does contain an RTL8139 driver. However, you have to install the package after installing FreeDOS, even if you performed a full install. It also seems to lock up for me when attempting to load it.

The PCNet packet driver on the other hand, is installed with a full FreeDOS install, is open source, and does work, at least for me.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have performed a self-review of my code
  • I have tested my code in common scenarios and confirmed there are no regressions
  • I have added comments to my code, particularly in hard-to-understand sections

@TheMuso TheMuso force-pushed the use-pcnet-for-freedos branch from 1baeb94 to 78cdf04 Compare February 11, 2025 09:44
@TheMuso TheMuso changed the title Fix: Use the pcnet ethernet device for FreeDOS fix: Use the pcnet ethernet device for FreeDOS Feb 11, 2025
FreeDOS does ship the Crynwr packet driver package, which does contain an
RTL8139 driver. However, you have to install the package after installing
FreeDOS, even if you performed a full install. It also seems to lock up for
me when attempting to load it.

The PCNet packet driver on the other hand, is installed with a full FreeDOS
install, is open source, and does work, at least for me.
@TheMuso TheMuso force-pushed the use-pcnet-for-freedos branch from 78cdf04 to 4960dde Compare February 11, 2025 20:34
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.

1 participant