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

hal_nordic: nrf_glue.h change mapped assert function #22541

Closed
koffes opened this issue Feb 6, 2020 · 0 comments · Fixed by #23125
Closed

hal_nordic: nrf_glue.h change mapped assert function #22541

koffes opened this issue Feb 6, 2020 · 0 comments · Fixed by #23125
Assignees
Labels
Enhancement Changes/Updates/Additions to existing features platform: nRF Nordic nRFx

Comments

@koffes
Copy link
Collaborator

koffes commented Feb 6, 2020

Today the mapping from NRFX_ASSERT is done like this:

#define NRFX_ASSERT(expression) assert(expression)

When we are using a different version of libc, the assert is mapped to libc and we cannot reach the zephyr assert handler and consequently, we cannot reach
assert_post_action to handle the fault as we desire.

Suggest we instead map this to __ASSERT_NO_MSG

@anangl

@koffes koffes added the Enhancement Changes/Updates/Additions to existing features label Feb 6, 2020
@koffes koffes changed the title hal_nordic: nrf_glue.h change assert call hal_nordic: nrf_glue.h change mapped assert function Feb 6, 2020
@carlescufi carlescufi added the platform: nRF Nordic nRFx label Feb 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Changes/Updates/Additions to existing features platform: nRF Nordic nRFx
Projects
None yet
3 participants