Skip to content

nonos-sdk: update to latest version 2.2.x branch, commitlog added in tree#6257

Merged
devyte merged 3 commits intoesp8266:masterfrom
d-a-v:sdk222y
Jul 5, 2019
Merged

nonos-sdk: update to latest version 2.2.x branch, commitlog added in tree#6257
devyte merged 3 commits intoesp8266:masterfrom
d-a-v:sdk222y

Conversation

@d-a-v
Copy link
Collaborator

@d-a-v d-a-v commented Jul 4, 2019 •
edited
Loading

Optional and only available for generic esp8266 board.

@d-a-v d-a-v mentioned this pull request Jul 4, 2019
Closed
6 tasks
@torntrousers
Copy link
Contributor

torntrousers commented Jul 4, 2019

Sounds interesting - do you know does this include the ESP-Now Broadcast fix from 3.0?

espressif/ESP8266_NONOS_SDK#8

@d-a-v
Copy link
Collaborator Author

d-a-v commented Jul 4, 2019

About espnow, there is a mention in the current testing sdk's commitlog (the commitlog is added in this PR):

$ zgrep -i now NONOSDK22*/commitlog*gz
NONOSDK22x/commitlog-from-221.txt.gz:    Merge branch 'feature/add_broad_support_for_espnow' into 'release/v2.2.x'
NONOSDK22x/commitlog-from-221.txt.gz:    feat(espnow): Add broadcast packets support for espnow

So I guess you're good to test this right now from master with generic esp8266 board for the sdk selection menu to appear.

@torntrousers
Copy link
Contributor

torntrousers commented Jul 4, 2019

Cool, will try tonight

@torntrousers
Copy link
Contributor

torntrousers commented Jul 4, 2019

Still doesn't seem to work with ESP-Now broadcasts, running the sketch from #6174 on two ESP's don't receive each other anyway.

@devyte devyte merged commit 403001e into esp8266:master Jul 5, 2019
@devyte
Copy link
Collaborator

devyte commented Jul 5, 2019

@torntrousers you may want to rrquest at Espressif's repo a backport of the esp now fix to their 2.2.x branch.

@torntrousers torntrousers mentioned this pull request Jul 5, 2019
Closed
@d-a-v
Copy link
Collaborator Author

d-a-v commented Jul 5, 2019

Running #6174 with any of the two testing firmwares
(one with 2.2.1+100 and the other with 2.2.1+100(or +61) gives

10:51:14.669 -> [1224298] Received Message. Len=10[1227043] Sending new Message -- Result Code: 0
10:51:19.677 -> [1229305] Received Message. Len=10[1232051] Sending new Message -- Result Code: 0
10:51:24.681 -> [1234312] Received Message. Len=10[1237059] Sending new Message -- Result Code: 0
10:51:29.685 -> [1239320] Received Message. Len=10[1242067] Sending new Message -- Result Code: 0
10:51:34.726 -> [1244329] Received Message. Len=10[1247075] Sending new Message -- Result Code: 0
10:51:39.731 -> [1249336] Received Message. Len=10[1252084] Sending new Message -- Result Code: 0
10:51:44.739 -> [1254345] Received Message. Len=10[1257092] Sending new Message -- Result Code: 0
10:51:49.744 -> [1259352] Received Message. Len=10[1262100] Sending new Message -- Result Code: 0
10:51:54.753 -> [1264359] Received Message. Len=10[1267108] Sending new Message -- Result Code: 0
10:51:59.760 -> [1269369] Received Message. Len=10[1272117] Sending new Message -- Result Code: 0
10:52:04.768 -> [1274379] Received Message. Len=10[1277125] Sending new Message -- Result Code: 0
10:52:09.776 -> [1279389] Received Message. Len=10[1282133] Sending new Message -- Result Code: 0

(default 2.2.1(legacy) wont't receive)

@d-a-v d-a-v deleted the sdk222y branch July 5, 2019 08:58
@d-a-v d-a-v mentioned this pull request Jul 5, 2019
Closed
4 tasks
@torntrousers
Copy link
Contributor

torntrousers commented Jul 5, 2019

As it worked for you I tried with master now that this is merged and it works for me too. Great!
(don't know why it didn't work yesterday, I was trying with the d-a-v:sdk222y branch, maybe I'd messed up the install or something)

@d-a-v
Copy link
Collaborator Author

d-a-v commented Jul 5, 2019 •
edited
Loading

You may tell them !

@aerlon aerlon mentioned this pull request Jul 10, 2019
Merged
7 tasks
@OttoWinter OttoWinter mentioned this pull request Nov 9, 2019
Closed
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@d-a-v @torntrousers @devyte