[SL-UP] 917ncp init failure fix #133
Conversation
|
Why does the efx32_ncp_host update cause errors? Did we root cause the the seen errors? |
The error is seen during SPI transfer, sl_si91x_host_spi_transfer() - This API is throwing SPI_BUSY error. I've tried to debug, but couldn't root cause it. |
Let me know when we have a better idea with what is happening with the latest efx32 ncp host file and the sl_net APIs. It might be worth while to open a Wiseconnect issue for this to get them involved. |
mkardous-silabs
added
the
sl-up
bhmanda-silabs
force-pushed
the
bugfix/init_fix_for_917ncp
branch
from
3bf866b
to
2792275
Compare
Wiseconnect ticket - https://jira.silabs.com/browse/WIFISDK-1109 |
| #define LDMA_MAX_TRANSFER_LENGTH 4096 | ||
| #define LDMA_DESCRIPTOR_ARRAY_LENGTH (LDMA_MAX_TRANSFER_LENGTH / 2048) | ||
|
|
||
| // use SPI handle for EXP header (configured in project settings) | ||
| extern SPIDRV_Handle_t sl_spidrv_exp_handle; | ||
| #define SPI_HANDLE sl_spidrv_exp_handle | ||
| static uint8_t dummy_buffer[MAX_DATA_PACKET_SIZE] = { 0 }; | ||
| static uint8_t dummy_buffer[1800] = { 0 }; |
There was a problem hiding this comment.
| static uint8_t dummy_buffer[1800] = { 0 }; | |
| static uint8_t dummy_buffer[MAX_DATA_PACKET_SIZE] = { 0 }; | |
| #include "sl_spidrv_exp_config.h" | ||
| #include "sl_spidrv_instances.h" | ||
| #include "spidrv.h" | ||
|
|
||
| #define MAX_DATA_PACKET_SIZE 1800 |
There was a problem hiding this comment.
can we keep this define for clarify?
| #include "sl_si91x_ncp_utility.h" | ||
| #include <platform/silabs/wifi/wf200/platform/spi_multiplex.h> |
There was a problem hiding this comment.
These don't need to be seperate - can we keep them with the rest?
This PR fixes the init failure in 917NCP boards.
After integrating the matter_sdk code with the sl_net APIs, the init is failing because LWIP init is called twice for 917NCP.
To resolve this, we have removed the LWIP init from PlatformManager and modified the init sequence to avoid errors while booting. But, with the recent SPI related changes, we are facing SPI_BUSY error. So, reverted the recent changes made in efx32_ncp_host.c file. With these changes, the init for 917NCP is passing.