-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
WIP: Circuitpython nickzoic 1800 wiznet socket #1837
Conversation
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! |
Has anyone tested this for you? |
shared-bindings/wiznet/wiznet5k.c
Outdated
@@ -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? |
There was a problem hiding this comment.
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!
@gvcp has been doing some testing down the bottom of #1800 ... still got a couple of little issues which I'll look at tomorrow.
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. |
yah if they set an address, dont use dhcp by default |
Is the uf2 image for the feather M4 express with the changes blow available anywhere for further testing ? @nickzoic pushed 3 commits.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 ;) |
On Sat, 11 May 2019, at 03:16, gvcp wrote:
Is the uf2 image for the feather M4 express with the changes blow available anywhere for further testing ?
It's in the latest Release Candidate: https://github.com/adafruit/circuitpython/releases/tag/4.0.0-rc.2 (click on "Assets" down the very bottom of that page)
Doesn't mean we won't keep fixing stuff ... thanks for your help with testing and keep reporting issues if you find them!
For starters, I'll raise a separate issue for fixing up the exceptions, eg: something better than numbers `OSError`s.
…-----N
|
Fixes problems experienced in Ethernet FeatherWing Library #703 W5500 problem #1500 and no socket communication with W5500 ethernet wing and feather m4 express #1800 with DHCP interfering with open sockets.
Adds "dhcp" as a boolean argument to wiznet constructor (keyword only, default True)
Setting "dhcp=False" will prevent DHCP from starting. It can still be started by setting the .dhcp
property.