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

Network settings should be loaded from EEPROM #10

Open
JayFoxRox opened this issue Jul 8, 2017 · 3 comments
Open

Network settings should be loaded from EEPROM #10

JayFoxRox opened this issue Jul 8, 2017 · 3 comments

Comments

@JayFoxRox
Copy link
Member

This is more of an issue with nxdk itself, however, nxdk-rdt is directly affected by it.

Currently, the network settings are hardcoded in net.c
For me, this meant that whenever I wanted to run nxdk-rdt in XQEMU, I had to go in and change the source code to use DHCP. When running on Xbox, I manually had to change it to use hardcoded addresses.

I've locally solved the issue by setting up a DHCP server for the ethernet port on my laptop, so the connected Xbox would also work with DHCP.
However, other people should not have to go through this.

We should fix this by loading the systems network settings from EEPROM.
The implementation should probably be in nxdk, but nxdk-rdt is currently the primary stakeholder. nxdk-rdt will also require some code changes. Hence the issue on this repository.

@mborgerson
Copy link
Member

Good idea...configure your network settings for Xbox, let the titles use them as expected. I think the code really belongs in some kind of generic network startup function in nxdk. We can do it here for now since this is the only real application, but down the road it'll probably just get copy-pasted, so I'd rather it be in the common code.

@JayFoxRox
Copy link
Member Author

JayFoxRox commented Jul 10, 2017

This is also what I meant in my first post (see the first line and the last sentence). = This is an nxdk issue, but the issue is more likely to be fixed if it's listed here. It should still be implemented directly in nxdk. However nxdk-rdt also needs a PR to support the changes in nxdk when this happens.

@JayFoxRox
Copy link
Member Author

JayFoxRox commented Jul 16, 2018

I've looked into this, my prototype (including a handful of problems) can be found at JayFoxRox#11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants