Skip to content

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

kapbh
Copy link
Contributor

@kapbh kapbh commented Jul 22, 2025

manifest-pr-skip

Copy link

github-actions bot commented Jul 22, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hostap zephyrproject-rtos/hostap@e942f86 zephyrproject-rtos/hostap@e7feee3 (main) zephyrproject-rtos/[email protected]

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,
Copy link
Contributor

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.

Copy link
Contributor Author

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.

Copy link
Contributor

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.

Copy link
Contributor

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?

Copy link
Contributor

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.

kapbh and others added 6 commits July 22, 2025 08:55
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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants