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

Any plans to extend the WIZnet550 driver? #1806

Closed
MaBecker opened this issue Apr 24, 2020 · 9 comments
Closed

Any plans to extend the WIZnet550 driver? #1806

MaBecker opened this issue Apr 24, 2020 · 9 comments

Comments

@MaBecker
Copy link
Contributor

@MaBecker MaBecker commented Apr 24, 2020

Would be cool to have

  • getHostname() and setHostname()

  • reset() to release ip given with setIP() only for WS550 because it has a default ip after reset/powerup

  • ......

To Be extended ...

@gfwilliams
Copy link
Member

@gfwilliams gfwilliams commented Apr 24, 2020

getHostname

If the hostname is actually configurable, then yes. I'm not sure if the driver does mDNS, so if it doesn't there's be no point having the hostname.

reset()

What's the use for this one? Can you not just call setIP with 0,0,0,0?

One thing I've started doing is adding a function that allows the user to send command straight to the chip. Might be a way to allow some of the more interesting functionality to be exposed...

@MaBecker
Copy link
Contributor Author

@MaBecker MaBecker commented Apr 26, 2020

@opichals do you have any whishes or needs to enhance WIZnet lib?

@MaBecker
Copy link
Contributor Author

@MaBecker MaBecker commented Apr 26, 2020

I'm not sure if the driver does mDNS

Look's like mDNS and uses a fixed hostname.

This is hows it could be done.

What's the use for this one? Can you not just call setIP with 0,0,0,0?

Some boards have a reset pin and if there is a reset they startup with a default like this

so just a optional pin {rst:
WIZ550io

={
  ip: "192.168.1.2",
  subnet: "255.255.255.0",
  gateway: "192.168.1.1",
  dns: "0.0.0.0",
  mac: "00:08:dc:57:c1:f0"
 }

Ok, for other board a setIP({values}) is ok.

@gfwilliams
Copy link
Member

@gfwilliams gfwilliams commented Apr 27, 2020

Ahh, So I'm not sure that's mDNS - it just gives a hostname when it configures itself via DHCP? That sounds like a great thing to add.

I'm not too sure about the reset - for the cases where you need it, can't you just toggle the reset pin yourself?

@MaBecker
Copy link
Contributor Author

@MaBecker MaBecker commented Apr 30, 2020

What about adding this too?

  • ping(cb)

  • getState() return value of JsNetworkState as string

@gfwilliams
Copy link
Member

@gfwilliams gfwilliams commented Apr 30, 2020

That'd be cool if you want to add it.

Although with ping - maybe be careful it doesn't add too much extra size to the binary?

@MaBecker
Copy link
Contributor Author

@MaBecker MaBecker commented May 29, 2020

>eth.getStatus()
={
  state: "online",
  id: "W5500",
  phylink: "up",
  phypowmode: "normal"
 }
> 
@MaBecker
Copy link
Contributor Author

@MaBecker MaBecker commented Jul 1, 2020

calling eth.setIP()has some extra jsiConsolePrintf() like:

jsiConsolePrintf("> Send DHCP_DISCOVER\r\n");

jsiConsolePrintf("> Send DHCP_Request\r\n");

and so on...

I can't remember which function to use so those are not printed when build with RELEASE=1.

@MaBecker MaBecker mentioned this issue Jul 1, 2020
@gfwilliams
Copy link
Member

@gfwilliams gfwilliams commented Jul 2, 2020

That sounds good - there isn't actually a specific function for debug messages, so ifndef RELEASE is probably the best bet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants