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

Resolved init failure for 917NCP #242

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arun-silabs
Copy link
Collaborator

Problem

  • Init failure is seen in 917NCP in sl_net_wifi_client_init API

Change overview

During init, the 917ncp is going into infinite loop inside tcpip_init() call. This tcpip_init() is called in InitChipStack PlatformManagerImpl::_InitChipStack(void)
So removing this for 917ncp.

Testing

How was this tested?

  • Tested the build locally, init passed with this change.

Copy link
Collaborator

@mkardous-silabs mkardous-silabs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should be able to use the sl_net init process as is.
The file is not in sdk support to modify the init sequence - just to fix the build issues.
The right fix, is removing the tcpip_init() call from our side.

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.

2 participants