Skip to content
This repository was archived by the owner on Apr 24, 2019. It is now read-only.

Support for NUCLEO_F401RE + X-NUCLEO_IDW01M1 WiFi #323

Closed
wants to merge 6 commits into from

Conversation

JanneKiiskila
Copy link
Contributor

Combination of commits by betwz and some changes on top of that by me to get the NUCLEO F401RE + X-Nucleo WiFi working.

betzw added 2 commits October 10, 2017 10:19
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]>
easy-connect.lib Outdated
@@ -1 +1 @@
https://github.com/ARMmbed/easy-connect/#bf821b0695ccada3620a6091503eb53a2a1ac3c8
https://github.com/ARMmbed/easy-connect/#adb235d929815d92083aa2abeed9f29b0386b2c3
Copy link
Contributor

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).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep!

Copy link
Contributor

@betzw betzw left a 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.

Janne Kiiskilä added 3 commits October 10, 2017 23:30
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.
@JanneKiiskila
Copy link
Contributor Author

Latest easy-connect has been updated.

@JanneKiiskila
Copy link
Contributor Author

@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.

@betzw
Copy link
Contributor

betzw commented Oct 11, 2017

@JanneKiiskila - could you pls. elaborate on this a bit more?

@JanneKiiskila
Copy link
Contributor Author

JanneKiiskila commented Oct 11, 2017

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.

Starting mbed Client example
[EasyConnect] IPv4 mode
[EasyConnect] Using WiFi (X-NUCLEO-IDW01M1)
[EasyConnect] Connecting to WiFi jannek.iki.fi
[EasyConnect] MAC address 00:80:E1:B7:D0:AC
[EasyConnect] Connection to Network Failed -3012!

Connection to Network Failed - exiting application...

The text on my Wifi module is stating X-NUCLEO-IDW01M1, so the default value in the mbed_lib.json (https://github.com/ARMmbed/wifi-x-nucleo-idw01m1/blob/master/mbed_lib.json) should match the HW.

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.
@betzw
Copy link
Contributor

betzw commented Oct 12, 2017

@JanneKiiskila, you might want to compare with PR #327 and picky-pick necesary modifications.

@betzw
Copy link
Contributor

betzw commented Nov 3, 2017

Any progress on this or PR #327?

cc @screamerbg, @nikapov-ST

@betzw
Copy link
Contributor

betzw commented Nov 7, 2017

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/*
Copy link
Contributor

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/*
Copy link
Contributor

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/*
Copy link
Contributor

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?

@JammuKekkonen
Copy link
Contributor

@JanneKiiskila functionality works for me, but i'd remove the (afaik) unnecessary mbedignores unless there are some known cases where they are needed.

@betzw
Copy link
Contributor

betzw commented Nov 7, 2017

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 mbedignore files, I do not know what ARM's policy is there, but I would prefer to exclude as much unnecessary code as possible from builds ...

@JanneKiiskila
Copy link
Contributor Author

Agree with @betzw - turns out that for example ATParser leaves 35 bytes of code there, if it's compiled and not used.

@teetak01
Copy link
Contributor

Closing in favor of: #327

@teetak01 teetak01 closed this Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants