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

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

Merged
merged 3 commits into from
Jul 5, 2019

Conversation

d-a-v
Copy link
Collaborator

@d-a-v d-a-v commented Jul 4, 2019

Optional and only available for generic esp8266 board.

d-a-v added 2 commits July 4, 2019 15:19

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@torntrousers
Copy link
Contributor

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

Cool, will try tonight

@torntrousers
Copy link
Contributor

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.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@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.

@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
4 tasks
@torntrousers
Copy link
Contributor

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

You may tell them !

@aerlon aerlon mentioned this pull request Jul 10, 2019
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants