-
Notifications
You must be signed in to change notification settings - Fork 56
Support for NUCLEO_F401RE + X-NUCLEO_IDW01M1 WiFi #323
Conversation
Includes update of `easy-connect.lib` (cherry picked from commit af2dbc6) Signed-off-by: Janne Kiiskilä <[email protected]>
(cherry picked from commit f810786) Signed-off-by: Janne Kiiskilä <[email protected]>
63eee65
to
6d21b75
Compare
easy-connect.lib
Outdated
@@ -1 +1 @@ | |||
https://github.com/ARMmbed/easy-connect/#bf821b0695ccada3620a6091503eb53a2a1ac3c8 | |||
https://github.com/ARMmbed/easy-connect/#adb235d929815d92083aa2abeed9f29b0386b2c3 |
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.
Update with latest easy-connect
hash code (once merged in).
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.
Yep!
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.
I would wait for the new easy-connect
and update easy-connect.lib
before merging.
Current tip of master that is, MCR20A updates at least there and now the latest X-WIFI-NUCLEO as well.
Modify `build_all.sh` - add Nucleo F401 with X-WIFI-NUCLEO Multiple .mbedignore file modifications to add the new x-wifi folder to avoid compilation problems due to missing PC_8/PC_12. - Hopefully we can get rid of these .mbedignore -files completely, but that seems to take some time. Some minor fixes to the README.MD. Wifi choices were all wrong in the example. UDP works over Ethernet IPv6 as well. Compilation info update to REAMDE.MD The WiFis have better .mbedingore files now, so advice how to use them AND also the actual mbed_app.json files we have there. Test steps: `configs/wifi_idw01m1_v4.json` as `mbed_app.json` - put in the Wifi APN + pass phrase `configs/wifi-idw01m1-mbedignore` as `.mbedignore` and it will roll. ``` Starting mbed Client example [EasyConnect] IPv4 mode [EasyConnect] Using WiFi (X-NUCLEO-IDW01M1) [EasyConnect] Connecting to WiFi jannek.iki.fi [EasyConnect] Connected to Network successfully [EasyConnect] MAC address 00:80:E1:B7:D0:AC [EasyConnect] IP address 192.168.1.163 SOCKET_MODE : TCP Connecting to coap://api.connector.mbed.com:5684 Registered object successfully! ``` Button support not in place yet, more abstract button support coming via another PR.
a505806
to
edb243a
Compare
Latest easy-connect has been updated. |
@betzw - there is now some problem with the latest one, it doesnt' connect at all. With the previous easy-connect it was still able to connect - I think I need to have something more now as you have multiple models, but how do you know what? I think the README.md has to be updated a bit to detail this out. |
@JanneKiiskila - could you pls. elaborate on this a bit more? |
With the latest x-nucleo-wifi-idw01m1 repo -> it fails the connection. I've double checked by wifi APN + passphrase and they are OK, but it fails.
The text on my Wifi module is stating X-NUCLEO-IDW01M1, so the default value in the |
Two expansion boards are supported, the Arduino form factor boards needs a target override - offer the place where to put it & info how to do that.
@JanneKiiskila, you might want to compare with PR #327 and picky-pick necesary modifications. |
Any progress on this or PR #327? |
Bump @JanneKiiskila |
@@ -1,3 +1,5 @@ | |||
easy-connect/atmel-rf-driver/* | |||
easy-connect/mcr20a-rf-driver/* | |||
easy-connect/stm-spirit1-rf-driver/* | |||
easy-connect/wifi-x-nucleo-idw01m1/* |
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.
at least esp8266 / k64f seems to compile fine without this. is there some known combination that needs this?
@@ -0,0 +1,4 @@ | |||
easy-connect/atmel-rf-driver/* |
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.
at least esp8266 / k64f seems to compile fine without this. is there some known combination that needs this?
@@ -0,0 +1,5 @@ | |||
easy-connect/atmel-rf-driver/* |
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.
at least esp8266 / k64f seems to compile fine without this. is there some known combination that needs this?
@JanneKiiskila functionality works for me, but i'd remove the (afaik) unnecessary mbedignores unless there are some known cases where they are needed. |
In general I would consider this PR as a bit stale and prefer PR #327 to be considered for merging (at the place of this PR). @JanneKiiskila, what do you think? Regarding the |
Agree with @betzw - turns out that for example ATParser leaves 35 bytes of code there, if it's compiled and not used. |
Closing in favor of: #327 |
Combination of commits by betwz and some changes on top of that by me to get the NUCLEO F401RE + X-Nucleo WiFi working.