Skip to content

drivers: nrf_wifi: Fix TX rate in Mbps #93085

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 2 commits into
base: main
Choose a base branch
from

Conversation

kapbh
Copy link
Contributor

@kapbh kapbh commented Jul 14, 2025

The current_tx_rate will be in Kbps. It is showing wrong TX rate in wifi status. Need to convert tx_birate to Kbps.

@github-actions github-actions bot added area: Wi-Fi Wi-Fi size: XS A PR changing only a single line of code labels Jul 14, 2025
rlubos
rlubos previously approved these changes Jul 14, 2025
rlubos
rlubos previously approved these changes Jul 14, 2025
kapbh added 2 commits July 15, 2025 06:56
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.

Signed-off-by: Kapil Bhatt <kapil.bhatt@nordicsemi.no>
Update hostap revision to fix tx rate.

Signed-off-by: Kapil Bhatt <kapil.bhatt@nordicsemi.no>
Copy link

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

Name Old Revision New Revision Diff
hostap zephyrproject-rtos/hostap@e942f86 (main) zephyrproject-rtos/hostap#93 zephyrproject-rtos/hostap#93/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hostap DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Jul 15, 2025
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Networking area: Wi-Fi Wi-Fi DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hostap size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants