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.

Sign up for GitHub

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

Jump to bottom

shell on Arduino Due prints "shell>" before the delayed boot banner #1280

Closed
inakypg opened this issue Aug 28, 2017 · 2 comments
Closed

shell on Arduino Due prints "shell>" before the delayed boot banner #1280

inakypg opened this issue Aug 28, 2017 · 2 comments
Assignees
@inakypg
Labels
bug The issue is a bug, or the PR is fixing a bug
Milestone
v1.10.0

Comments

@inakypg
Copy link
Contributor

inakypg commented Aug 28, 2017

When running the samples/subsys/shell/ app on an Arduino Due (knowing it will fail because there is no IRQ serial support):

shell> ***** delaying boot 1000ms (per build configuration) *****
***** BOOTING ZEPHYR OS v1.9.0-rc2 (delayed boot 1000ms) *****

thus, the delayed boot banner is being printed AFTER the app is running (that prints shell).

@nashif suggests moving the banner printing / delay on top of _sys_device_do_config_level(_SYS_INIT_LEVEL_APPLICATION);
@nashif
Copy link
Member

nashif commented Aug 28, 2017

this is not related to Arduino Due, it is a generic bug that can be reproduced with qemu

@inakypg
Copy link
Contributor Author

inakypg commented Aug 28, 2017

Correct.

@nashif nashif added the bug The issue is a bug, or the PR is fixing a bug label Aug 28, 2017
@nashif nashif added this to the v1.10 milestone Aug 28, 2017
@inakypg inakypg mentioned this issue Sep 7, 2017
Merged
nashif pushed a commit that referenced this issue Sep 7, 2017
@inaky-intc
boot: move boot banner and delay before SYS_INIT_LEVEL_APPLICATION
1abd064 
Fixes #1280, but
also many other failures, where output was garbled due to this. Other
similarly affected issues are missing first benchmark (context) in
latency benchmark and some net tests.

Signed-off-by: Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>
This was referenced Sep 18, 2017
Open
Open
@zephyrbot zephyrbot mentioned this issue Sep 23, 2017
Closed
@nashif nashif modified the milestones: v1.10, v1.10.0 Oct 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@inakypg inakypg

Labels
bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Milestone
v1.10.0
Development

No branches or pull requests
2 participants
@nashif @inakypg