Skip to content

Conversation

@ajayparida
Copy link
Contributor

Soft HPQM and Rx host buffer programming support.

Comment on lines 144 to 145
#define NRF_WIFI_HOST_RPU_GDRAM_START_ADDR 0x20084548
#define NRF_WIFI_HOST_RPU_TX_CMD_START_ADDR 0x200800b8
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These be part of rpu_if.h ?

Makefile Outdated
ccflags-y += -DNRF_WIFI_DISPLAY_SCAN_BSS_LIMIT=150
ccflags-y += -DNRF_WIFI_RPU_MIN_TIME_TO_ENTER_SLEEP_MS=1000
ccflags-y += -DWIFI_NRF70_LOG_LEVEL=1
#ccflags-y += -DNRF71_RX_BUFF
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be part of Kconfig ? Also needs a more representative name (Just NRF71_RX_BUFF doesn't give an indication of what this feature is about).

Copy link
Collaborator

@sachinthegreen sachinthegreen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The HOST RX Buffer programming commit doesn't have any commit message.

@ajayparida ajayparida force-pushed the combo_rx_buff_soft_hpqm branch from f8912ca to 63f92f6 Compare July 3, 2025 11:05
In nRF70, the host programmed RX buffers directly
to the RPU through hardware queues. With nRF71, hardware queues
are not accessible from the host side. This update
adds a command handling to allow the host to pass RX buffers
to UMAC firmware, which then programs them to the RPU.

Signed-off-by: Ajay Parida <[email protected]>
SOFT_HPQM is to replace existing hardware based queue used to
communicate between host and firmware. Targeted towards
nrf71 based system.  TX and CTRL commands are used to communicate
using software queue. RX buffers are programmed to firmware
at initialization are handled by firmware.

Signed-off-by: Ajay Parida <[email protected]>
@ajayparida ajayparida force-pushed the combo_rx_buff_soft_hpqm branch from 63f92f6 to 36b86fc Compare July 3, 2025 11:19
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.

3 participants