[GitHub] [mynewt-core] chrisc11 commented on pull request #2686: Memfault-SDK integration with MyNewt (Alternate)

[GitBox <gi...@apache.org>]

chrisc11 commented on pull request #2686:
URL: https://github.com/apache/mynewt-core/pull/2686#issuecomment-938193838


   > So just add remaining code from this PR to memfault repo and close this PR, we'll merge os_coredump_cb from the other PR.
   
   hi @andrzej-kaczmarek, catching up on this thread, thanks for the feedback! We can definitely go ahead and pull the `util/memfault_sdk` part into the memfault-firmware-sdk directly.
   
   However, I do see benefit in having a memfault porting layer directly in mynewt (similar to how tinyusb looks like it's integrated today). It would make it easier for an end user to get started, follow mynewt core API changes, and integrate directly into tools like mcumgr (there was an early prototype for that [here](https://github.com/apache/mynewt-core/pull/2286#issuecomment-636707524)). Let me know if you see a path forward for including the port directly to make things easier for end users in the future. thanks!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org