-
Notifications
You must be signed in to change notification settings - Fork 679
Wi-Fi upmerge 2 for 3.1.0 #3066
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
base: main
Are you sure you want to change the base?
Conversation
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
@@ -243,7 +243,7 @@ following example simply prints when an event occurs. | |||
#define COAP_EVENTS_SET (NET_EVENT_COAP_OBSERVER_ADDED | NET_EVENT_COAP_OBSERVER_REMOVED | \ | |||
NET_EVENT_COAP_SERVICE_STARTED | NET_EVENT_COAP_SERVICE_STOPPED) | |||
void coap_event_handler(uint32_t mgmt_event, struct net_if *iface, | |||
void coap_event_handler(uint64_t mgmt_event, struct net_if *iface, |
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 don't think we can pull this, @rlubos tried this earlier in the full upmerge but abandoned due to some limitations.
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 need to take because of conflicts with @ajayparida commit.
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.
Depends how much you need this, for sure pulling the commit alone is not enough. I have a separate PR (nrfconnect/sdk-nrf#22993) which should be ready I believe, although since we were not doing another upmerge before the release, I've decided to postpone it not to bring a stable API change to the release w/o a need.
But if you need this change, I can refresh my PR an push it forward.
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.
Either we pull this change in or do a noup
for the conflicting commit?
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.
Temporary noup is also fine, just leave a note in the commit message it should be reverted on the next upmerge.
Update hostap revision in manifest. Upstream PR #: 93085 Signed-off-by: Kapil Bhatt <[email protected]>
…float The Tx rate was previously stored as an integer, which caused loss of precision for rates like 8.6 Mbps or 34.4 Mbps due to integer division. The change will update data type to float. Upstream PR #: 93085 Signed-off-by: Kapil Bhatt <[email protected]>
Create dedicated heap for supplicant operations and define the heap size. Upstream PR #: 93085 Signed-off-by: Ravi Dondaputi <[email protected]>
…nto common file Refactor certificate processing code to eliminate duplication and enable reuse across modules that require enterprise support. Upstream PR #: 93085 Signed-off-by: Triveni Danda <[email protected]>
the nRF connect device tree extension shows below warning: 'Property not mentioned in "nordic,nrf-pinctrl:child"'. Fix this by applying the property to both groups separately. Upstream PR #: 93465 Signed-off-by: Chaitanya Tata <[email protected]>
Support to configure BSS max idle period at runtime. This commit should be reverted on the next upmerge. Signed-off-by: Kapil Bhatt <[email protected]>
manifest-pr-skip