Skip to content

UNOQ: implement proper "wait for app" mode#369

Open
facchinm wants to merge 2 commits intomainfrom
wait_for_app_mode
Open

UNOQ: implement proper "wait for app" mode#369
facchinm wants to merge 2 commits intomainfrom
wait_for_app_mode

Conversation

@facchinm
Copy link
Member

The current implementation of "Flash Mode=RAM" was meant to syncronize the startup of the microcontroller with an application launched by AppLab.
However, it has a bunch of limitiations which make it basically useless for bigger projects.

Replace it with another step of (gpio-based) syncronization, so AppLab can safely decide when to unlock the boot.

  • Depends on the corresponding PR in arduino-app-cli
  • Requires a rebuild+repackage of zephyr-sketch-tool

@lucarin91 @pillo79

Flash mode (RAM) has always been a hack; get rid of it with a better syncronization with arduino-router and arduino-appl-cli, gpio based
@github-actions
Copy link

github-actions bot commented Mar 2, 2026

Built 0.53.2-0.dev+165f441

CI run PASSED 🟢

ArtifactBoardCoreTestsRAMSketchesWarningsErrors
✅* zephyr_contrib ek_ra8d1 📗 ✅*

11.9%

22-
frdm_mcxn947 3 🏷️ ✅*

57.9%

22-
frdm_rw612 1 🏷️ ✅*

83.0%

22-
✔️* zephyr_main giga 4 🏷️ ✅*

54.5%

4414-
nano33ble 1 🏷️ ✅*

78.7%

2210-
nano_matter 📗 ✔️*

⚠️ 85.7%

2010(2*)
niclasense 2 🏷️ ✅*

⚠️ 87.1%

2010-
opta 4 🏷️ ✔️*

46.6%

5426(2*)
portentac33 3 🏷️ ✔️*

⚠️ 95.1%

5628(8*)
portentah7 3 🏷️ ✔️*

47.3%

5828(2*)
✅* zephyr_unoq unoq 📗 ✅*

26.3%

5210-
Legend

BoardTestStatus description
🔥 🔥 Test run failed to complete.
🔴 Test completed with unexpected errors.
✔️* 🚫 Test completed with errors, but all are known/expected.
✅* 🟡 Test completed with some warnings; no errors detected.
🟢 Test passed successfully, with no warnings or errors.
🌑 🌑 Test was skipped.

// wait for another pin toggle
while (gpio_pin_get_dt(&spec) != 0) {
k_sleep(K_MSEC(10));
}

Choose a reason for hiding this comment

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

Maybe we should make this more robust by setting a max length of the LOW state, e.g., 200ms?

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UNO Q - Current sources - running a sketch, then running APP, power off and on - Faults

4 participants