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

Port to Nuttx #41

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tito97sp
Copy link

@tito97sp tito97sp commented Dec 8, 2022

RATIONALE:
This PR makes a first porting of memfault-sdk for the Nuttx platform.

It implements the necessary platform specific functions to compiling the following memfault components:

  • Core
  • Utils
  • Panics
  • Metrics

KNOWN LIMITATIONS
On this version, the functions under memfault_platform_unimplemented.c file are stubbed, and its functionality is reduced, the affected functions are:

  • memfault_platform_halt_if_debugging
  • memfault_platform_sanitize_address_range
  • memfault_platform_reboot

TESTING
This porting is tested for the latest version of Nuttx (commit hash: 1ecaa4e6) running into a Nucleo-H743ZI board.

Signed-off-by: Andrés Sánchez Pascual [email protected]

Signed-off-by: Andrés Sánchez Pascual <[email protected]>
@tito97sp
Copy link
Author

tito97sp commented Dec 12, 2022

Hi @noahp, @chrisc11. Please review if this porting of memfault-sdk to Nuttx is helpful for you.

Signed-off-by: Andrés Sánchez Pascual <[email protected]>
Signed-off-by: Andrés Sánchez Pascual <[email protected]>
@noahp
Copy link
Contributor

noahp commented Apr 27, 2023

Hi @tito97sp , thanks so much for this pull request! We'll give it a try and get back to you.

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