Skip to content

Commit

Permalink
[nrfconnect] Defined default MRP retry intervals for the platform
Browse files Browse the repository at this point in the history
The nrfconnect platform requires some custom MRP retry intervals
that are proven by tests to work for Wi-Fi and Thread.

Added defining MRP retry values to the nrfconnect platform code.
  • Loading branch information
kkasperczyk-no committed Feb 22, 2023
1 parent cad990f commit b9a6221
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,3 @@
// Use a default pairing code if one hasn't been provisioned in flash.
#define CHIP_DEVICE_CONFIG_USE_TEST_SETUP_PIN_CODE 20202021
#define CHIP_DEVICE_CONFIG_USE_TEST_SETUP_DISCRIMINATOR 0xF00

// Switching from Thread child to router may cause a few second packet stall.
// Until this is improved in OpenThread we need to increase the retransmission
// interval to survive the stall.
#define CHIP_CONFIG_MRP_LOCAL_ACTIVE_RETRY_INTERVAL (1000_ms32)
17 changes: 17 additions & 0 deletions src/platform/nrfconnect/CHIPPlatformConfig.h
Original file line number Diff line number Diff line change
Expand Up @@ -95,3 +95,20 @@
#define CHIP_CONFIG_LOG_MODULE_Support_PROGRESS 0
#define CHIP_CONFIG_LOG_MODULE_DeviceLayer_DETAIL 0
#endif

// Set MRP retry intervals for Thread and Wi-Fi to test-proven values.
#ifndef CHIP_CONFIG_MRP_LOCAL_ACTIVE_RETRY_INTERVAL
#if CHIP_ENABLE_OPENTHREAD
#define CHIP_CONFIG_MRP_LOCAL_ACTIVE_RETRY_INTERVAL (800_ms32)
#else
#define CHIP_CONFIG_MRP_LOCAL_ACTIVE_RETRY_INTERVAL (500_ms32)
#endif
#endif // CHIP_CONFIG_MRP_LOCAL_ACTIVE_RETRY_INTERVAL

#ifndef CHIP_CONFIG_MRP_LOCAL_IDLE_RETRY_INTERVAL
#if CHIP_ENABLE_OPENTHREAD
#define CHIP_CONFIG_MRP_LOCAL_IDLE_RETRY_INTERVAL (800_ms32)
#else
#define CHIP_CONFIG_MRP_LOCAL_IDLE_RETRY_INTERVAL (500_ms32)
#endif
#endif // CHIP_CONFIG_MRP_LOCAL_IDLE_RETRY_INTERVAL

0 comments on commit b9a6221

Please sign in to comment.