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

samples: cellular: MSS Onomondo softsim support #20233

Draft
wants to merge 1 commit into
base: v2.6-branch
Choose a base branch
from

Conversation

plskeggs
Copy link
Contributor

@plskeggs plskeggs commented Feb 6, 2025

Add an overlay for Onomondo.

Disable APP FOTA and MCUboot to conserve flash for softsim use.

Switch to newlib libc to be compatible with prebuilt Onomondo library.

Add checking for softsim profile to cloud_connection.c. If none present, receive it over serial then reboot.

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Feb 6, 2025
Add an overlay for Onomondo.

Disable APP FOTA and MCUboot to conserve flash for softsim use.

Switch to newlib libc to be compatible with prebuilt Onomondo
library.

Add checking for softsim profile to cloud_connection.c. If
none present, receive it over serial then reboot.

Signed-off-by: Pete Skeggs <[email protected]>
@plskeggs plskeggs force-pushed the feature-mss-softsim-integration branch from 64d677d to 6831a6a Compare February 6, 2025 21:40
@lemrey lemrey requested a review from MirkoCovizzi February 17, 2025 08:18
@MirkoCovizzi
Copy link
Contributor

MirkoCovizzi commented Feb 17, 2025

I believe the configurations should sit in the Onomondo's repository at https://github.com/onomondo/nrf-softsim when possible, and the patches should be in a fork of sdk-nrf managed by Onomondo.

Is there any particular reason for wanting to add it here?

@plskeggs
Copy link
Contributor Author

@MirkoCovizzi I leave Nordic (unwillingly) at the end of this month.

I opened this so I could share it with a customer in support of a DevZone ticket. I had no intention of going further with it.

Perhaps it should be in the Onomondo repo, I don't know.

@pascal-nordic leaving this to you to follow up on.

@plskeggs plskeggs requested a review from a team February 18, 2025 19:05
@MirkoCovizzi
Copy link
Contributor

@MirkoCovizzi I leave Nordic (unwillingly) at the end of this month.

I opened this so I could share it with a customer in support of a DevZone ticket. I had no intention of going further with it.

Perhaps it should be in the Onomondo repo, I don't know.

@pascal-nordic leaving this to you to follow up on.

Sorry to hear that! I will keep in touch with @pascal-nordic then, thanks!

Copy link

This pull request has been marked as stale because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 7 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Mar 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants