mcuboot icon indicating copy to clipboard operation
mcuboot copied to clipboard

zephyr: Use k_busy_wait if multithreading is not activated

Open kvaellning opened this issue 3 years ago • 3 comments

If CONFIG_MULTITHREADING is not set, k_sleep is not defined by Zephyr.

Signed-off-by: Robert Schulze [email protected]

kvaellning avatar Jan 26 '22 12:01 kvaellning

@nvlsianpu Then maybe just activate multi-thread mode in the prj.conf per default and generating a better error message during compile time?

kvaellning avatar Jun 28 '22 09:06 kvaellning

@nvlsianpu Then maybe just activate multi-thread mode in the prj.conf per default

CONFIG_MULTITHREADING=y is by default for all with excuse to some of configuration of the nRF devices.

... and generating a better error message during compile time?

Compilation will miss for instance k_thread_create() as well. Do you think that something like error on CONFIG_MULTITHREADING=n should be added?

nvlsianpu avatar Jul 01 '22 13:07 nvlsianpu

Compilation will miss for instance k_thread_create() as well. Do you think that something like error on CONFIG_MULTITHREADING=n should be added?

That might be the best solution. I'll update the pull request in the next time.

kvaellning avatar Jul 04 '22 08:07 kvaellning

This pull request has been marked as stale because it has been open (more than) 60 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 14 days. Note, that you can always re-open a closed pull request at any time.

github-actions[bot] avatar Jan 01 '23 02:01 github-actions[bot]