Introducing Ne2000 driver - #118
Open
sirbu12 wants to merge 2 commits into
Open
Conversation
Author
|
forgot to say: before the PR fixing the compilation error i fixed it earlier so i can test the driver. only warning |
Author
|
wait wait wait i'm not sure if the compiled binaries got here from my testings (i forgot to remove them). |
Author
|
Default gateway to host: 10.0.2.2 |
Author
|
forgot to say: update todo.txt if this PR is accepted pls |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
i had also made the documentation for it located at x16-PRos/docs/Drivers/Ne2000.md
and a ping command too:
ping.asm: the ping command. usage: ping IPv4 address
ne2000.inc: included file inside all apps that has to do with networking (rignt now only ping.asm), includes the driver itself inside the app source code as a set of functions to be called from the app source code.
also please note: i am still a begginer at git because the cloned repo i worked from includes a splash screen but the original repo ( github.com/prox2011/x16-pros ) has another splash screen.
for now, all the networking protocols is needed to be implemented manually in the app because the driver only has the code to talk with the NIC itself, not any protocol.
In the future:
i will try to add IPv6 ( if possible in 16 bit )
protocols so each app won't have to
In case of changes needed, i hope the reviewer ( PRoX2011 ) will text me in telegram or right here in the comments