Skip to content

Conversation

@tomchy
Copy link
Contributor

@tomchy tomchy commented Dec 1, 2025

Disable BT stack as well as MPSL before resetting the SoC.

Ref: NCSDK-36564

NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Dec 1, 2025
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#3570

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
@de-nordic
Copy link
Contributor

I wonder whether we should have some transport universal way (callbacks?) that can be used here, to address the issue across all transport requiring such operations

NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Dec 4, 2025
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#3570

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
Disable BT stack as well as MPSL before resetting the SoC.

Ref: NCSDK-36564

Signed-off-by: Tomasz Chyrowicz <tomasz.chyrowicz@nordicsemi.no>
@nvlsianpu nvlsianpu force-pushed the test/reset/NCSDK-36564_Analyze_device_reset branch from 90be114 to bc55cc7 Compare December 4, 2025 11:56
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Dec 4, 2025
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#3570

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
Copy link
Contributor

@nordicjm nordicjm left a comment

Choose a reason for hiding this comment

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

use a reset hook, keep it in ncs without no-up, or alternatively copy file to ncs and add code there with alternative Kconfig to enable the ncs version instead

@tomchy
Copy link
Contributor Author

tomchy commented Dec 8, 2025

Replaced with hooks in: nrfconnect/sdk-nrf#26118

@tomchy tomchy closed this Dec 8, 2025
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.

3 participants