Skip to content
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

[SL-UP] 917ncp init failure fix #133

Conversation

arun-silabs
Copy link
Contributor

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.

@arun-silabs arun-silabs requested a review from a team as a code owner November 26, 2024 16:36
@arun-silabs arun-silabs changed the title 917ncp init failure fix [SL-UP] 917ncp init failure fix Nov 26, 2024
@mkardous-silabs
Copy link
Contributor

Why does the efx32_ncp_host update cause errors? Did we root cause the the seen errors?

@arun-silabs
Copy link
Contributor Author

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.
The SPI issue with the latest efx32_ncp_host.c file from wifi_sdk is coming up after integrating with sl_net layer. Need to debug the exact rootcause.

@mkardous-silabs
Copy link
Contributor

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. The SPI issue with the latest efx32_ncp_host.c file from wifi_sdk is coming up after integrating with sl_net layer. Need to debug the exact rootcause.

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 mkardous-silabs added the sl-up This TAG indicates that this commit needs to be upstreamed to CSA before its next release. label Nov 27, 2024
@bhmanda-silabs bhmanda-silabs force-pushed the bugfix/init_fix_for_917ncp branch from 3bf866b to 2792275 Compare November 29, 2024 16:26
@bhmanda-silabs
Copy link
Contributor

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. The SPI issue with the latest efx32_ncp_host.c file from wifi_sdk is coming up after integrating with sl_net layer. Need to debug the exact rootcause.

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.

Wiseconnect ticket - https://jira.silabs.com/browse/WIFISDK-1109

@mkardous-silabs mkardous-silabs merged commit 00775a3 into SiliconLabsSoftware:release_2.5-1.4 Dec 2, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sl-up This TAG indicates that this commit needs to be upstreamed to CSA before its next release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants