Skip to content

Soft HPQM and Rx buffer programming support #52

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

ajayparida
Copy link
Contributor

Soft HPQM and Rx host buffer programming support.

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]>
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 ?

@@ -97,6 +97,7 @@ ccflags-y += -DNRF_WIFI_RPU_RECOVERY_PS_ACTIVE_TIMEOUT_MS=50000
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.

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