Skip to content

WIP: Circuitpython nickzoic 1800 wiznet socket #1837

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

Merged

Conversation

nickzoic
Copy link

@nickzoic nickzoic commented May 2, 2019

@nickzoic nickzoic requested a review from tannewt May 2, 2019 07:18
@nickzoic
Copy link
Author

nickzoic commented May 2, 2019

OK I've made some builds for revision d97c81b and temporarily made them available at:

[temporary location removed now that this is merged back]

If all goes well, I expect this PR will get merged back into CircuitPython proper shortly after PyCon, but these should at least let you all test the new code now and see if it solves your problem!

@tannewt
Copy link
Member

tannewt commented May 8, 2019

Has anyone tested this for you?

@@ -136,6 +160,7 @@ STATIC mp_obj_t wiznet5k_ifconfig(size_t n_args, const mp_obj_t *args) {
return mp_obj_new_tuple(4, tuple);
} else {
// set
// XXX should this automatically stop DHCP here?
Copy link
Member

Choose a reason for hiding this comment

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

What are the XXX comments for? Can you either remove them or rephrase them as TODO: <explanation of what to do?? Thanks!

@nickzoic nickzoic changed the title Circuitpython nickzoic 1800 wiznet socket WIP: Circuitpython nickzoic 1800 wiznet socket May 8, 2019
@nickzoic
Copy link
Author

nickzoic commented May 8, 2019

Has anyone tested this for you?

@gvcp has been doing some testing down the bottom of #1800 ... still got a couple of little issues which I'll look at tomorrow.

What are the XXX comments for?

That one's there to remind me to ask you whether you think dhcp should be automatically set to False when someone manually configures an address :-)

UPDATE: Argh, clicked wrong button! Need more coffee, or maybe less coffee.

@nickzoic nickzoic closed this May 8, 2019
@nickzoic nickzoic reopened this May 8, 2019
@ladyada
Copy link
Member

ladyada commented May 9, 2019

yah if they set an address, dont use dhcp by default

@gvcp
Copy link

gvcp commented May 10, 2019

Is the uf2 image for the feather M4 express with the changes blow available anywhere for further testing ?

@nickzoic pushed 3 commits.

e00a702 Stop DHCP when configuring IP address adafruit/circuitpython#1800
af0bba0 reset wiznet at network deinitialize adafruit/circuitpython#1800
0d08dde randomize tcp source port for adafruit/circuitpython#1800

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Merging these so they get in the RC. Any bugs we can fix with later versions. I won't block the stable release on networking issues though. Thanks @nickzoic and @gvcp

@tannewt tannewt merged commit 46164c6 into adafruit:master May 10, 2019
@ladyada
Copy link
Member

ladyada commented May 10, 2019

agree on not blocking on ethernet, its something i'd like to have working for those who dont/cant use wifi but most use wifi ;)

@nickzoic
Copy link
Author

nickzoic commented May 11, 2019 via email

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.

4 participants