-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[EFR32] Adds changes in the order of connected and IP notify events #21219
[EFR32] Adds changes in the order of connected and IP notify events #21219
Conversation
PR #21219: Size comparison from c32f137 to 7564293 Increases (9 builds for bl602, cc13x2_26x2, cyw30739, efr32, telink)
Decreases (5 builds for cc13x2_26x2, cyw30739, nrfconnect)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
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.
blocking the PR until we fix the commit history oh the submodule.
df7bf56
to
27ac6a8
Compare
PR #21219: Size comparison from 750eedf to 27ac6a8 Increases (5 builds for cc13x2_26x2, cyw30739, efr32, esp32)
Decreases (3 builds for bl602, cc13x2_26x2, telink)
Full report (32 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
can this be pulled into SVE ? |
@rosahay-silabs Can you create the PR for that? The bot failed to because of merge conflicts: see https://github.com/project-chip/connectedhomeip/runs/7555113528?check_suite_focus=true |
…roject-chip#21219) * Adds changes in the order of connected and IP notify events * Update SDK support
Problem
What is being fixed? Examples:
Change overview
Changes the order of the event notifications around access point connected and IP assignment.
Testing
How was this tested? (at least one bullet point required)