Skip to content

Commit

Permalink
[NXP K32W] Save RAM by reducing event queue length (#17979)
Browse files Browse the repository at this point in the history
Issue #9261

- K32W platform ran out of BSS last week
- Current usage is default max 100 events in queue. PR #17959 already reduced
  the size of each event and saved approx 600 bytes. This PR should reduce it
  further.

- This PR moves max queue size from 100 to 75, which is 25% reduction
  and much larger than some platforms which use 25 (which seems low to me)

- Other Thread platforms like QPG and EFR32 run on 25, so not worried here

Testing done:
- Sanity check on K32W via @doru91
  • Loading branch information
tcarmelveilleux authored and pull[bot] committed Jan 30, 2024
1 parent 12d4727 commit c788d40
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/platform/nxp/k32w/k32w0/CHIPDevicePlatformConfig.h
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,9 @@
#define CHIP_DEVICE_CONFIG_THREAD_TASK_STACK_SIZE 3072
#endif // CHIP_DEVICE_CONFIG_THREAD_TASK_STACK_SIZE

// Max size of event queue
#define CHIP_DEVICE_CONFIG_MAX_EVENT_QUEUE_SIZE 75

#ifndef CHIP_DEVICE_CONFIG_BLE_APP_TASK_NAME
#define CHIP_DEVICE_CONFIG_BLE_APP_TASK_NAME "BLE App Task"
#endif // CHIP_DEVICE_CONFIG_BLE_APP_TASK_NAME
Expand Down

0 comments on commit c788d40

Please sign in to comment.