search

arduino / ArduinoCore-renesas

MIT License
110 stars 75 forks source link

WiFi.ping() function on WiFiS3 library doesn't exist #251

Open gpb01 opened 8 months ago

gpb01 commented 8 months ago

The WiFi.ping() function, that exists in the library WiFiNINA, doesn't exist in the WiFiS3 library and there is no way to ICMP ping network devices.

Can you please add such a function?

Additional context

Additional requests

TeubyDE commented 7 months ago

would be really nice to have this feature!

paulvha commented 7 months ago

I have spend time to add PING. it requires changes to both the UNOR4Bridge software as well as the WiFiS3 library. In the attached zip is a document with the changes to be applied. Also the files that need changes are included for reference. WiFiPing.zip

regards, Paul

gpb01 commented 7 months ago

In the attached zip is a document with the changes to be applied.

Hi Paul, thanks a lot for your work, but ... if you don't create a "fork" and do an official "pull request" ... I don't know if the Arduino team will consider it ... :-\

Guglielmo

paulvha commented 7 months ago

hi Guglielmo, I had considered doing that, but I am not sure the Arduino Team is interested. Several other pending bridge issues could be solved and some don't look that difficult to solve. I was doing this exercise to understand how the bridge software is working. It took longer time to get started as good documentation was missing and different links & scripts were outdated.

regards, Paul

JAndrassy commented 7 months ago

@paulvha they are interested. I have now 22 PR merged in core and 3 in the firmware

paulvha commented 7 months ago

I have made a fork and pull request now. I also saw noticed update to the partition table for the ESP32 (i used the latest version from yesterday). I have made a pull request for the firmware and will add one for the core. Did you also solve the getTime() issue 206?

regards Paul

privitera commented 4 months ago

Wondering, what is the status of this?

voidzero commented 3 months ago

I just got my first R4 and am very excited to try this out. Thanks for the pull request, hope it will be accepted.