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

Some standard west commands should Just Work for native_posix board #36706

Closed
palchak-google opened this issue Jul 2, 2021 · 3 comments · Fixed by #68835
Closed

Some standard west commands should Just Work for native_posix board #36706

palchak-google opened this issue Jul 2, 2021 · 3 comments · Fixed by #68835
Labels
area: native port Host native arch port (native_posix) area: West West utility Enhancement Changes/Updates/Additions to existing features

Comments

@palchak-google
Copy link
Contributor

Is your enhancement proposal related to a problem? Please describe.
Currently when building for the native_posix board it is not possible to use a west command to execute the resulting binary. This is inconsistent with the workflow used for other boards. While not all west commands make sense for the native_posix board, it should still be possible to at least invoke the native binary somehow.

Describe the solution you'd like
If the target board is native_posix then:

  • Running west flash should execute ${BUILD_DIR}/zephyr/zephyr.elf with no arguments
  • Running west attach should execute ${BUILD_DIR}/zephyr/zephyr.elf with the argument -attach_uart

Describe alternatives you've considered
Alternatively, there should be a west execute command that is only enabled for the native_posix board.

Additional context

[ This space intentionally left blank ]
@palchak-google palchak-google added the Enhancement Changes/Updates/Additions to existing features label Jul 2, 2021
@nashif
Copy link
Member

nashif commented Jul 3, 2021

this actually works with the arc simulator already, so I agree this could be made consistent...

i9:zephyr(main): west flash
-- west flash: rebuilding
ninja: no work to do.
-- west flash: using runner arc-nsim
Console now belongs to UART, hit CRTL-] to return to simulator.
*** Booting Zephyr OS build zephyr-v2.6.0-694-g3221a1e8e703  ***
Hello World! nsim

@palchak-google all of this can be done in cmake/west extensions in the tree, so patches welcome...

@nashif nashif added the area: West West utility label Jul 3, 2021
@kartben
Copy link
Collaborator

kartben commented Dec 14, 2023

@aescolar thoughts on this old issue? I wonder if it is still relevant in the light of native_sim

@kartben kartben added the area: native port Host native arch port (native_posix) label Dec 14, 2023
@aescolar
Copy link
Member

@aescolar thoughts on this old issue? I wonder if it is still relevant in the light of native_sim

I guess the same request applies to native_sim. But as Anas said, patches welcome.

nashif added a commit to nashif/zephyr that referenced this issue Feb 10, 2024
@nashif
west: add native_sim flash command
5159b18 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
nashif added a commit to nashif/zephyr that referenced this issue Feb 11, 2024
@nashif
west: add native_sim flash command
8b51b12 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
nashif added a commit to nashif/zephyr that referenced this issue Feb 12, 2024
@nashif
west: add native_sim flash command
a8b9a32 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
nashif added a commit to nashif/zephyr that referenced this issue Feb 12, 2024
@nashif
west: add native_sim flash command
8362620 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
nashif added a commit to nashif/zephyr that referenced this issue Feb 13, 2024
@nashif
west: add native_sim flash, debugserver command
7fce526 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Also add debugserver command.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
nashif added a commit to nashif/zephyr that referenced this issue Mar 11, 2024
@nashif
west: add native_sim flash, debugserver command
d186573 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Also add debugserver command.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
nashif added a commit to nashif/zephyr that referenced this issue Mar 12, 2024
@nashif
west: add native_sim flash, debugserver command
d6edf48 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Also add debugserver command.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
@nashif nashif mentioned this issue Mar 12, 2024
Merged
nashif added a commit that referenced this issue Mar 12, 2024
@nashif
west: add native_sim flash, debugserver command
be60134 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Also add debugserver command.

Fixes #36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
ashiroji pushed a commit to ashiroji/zephyr that referenced this issue Mar 19, 2024
@nashif @ashiroji
west: add native_sim flash, debugserver command
b081352 
Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Also add debugserver command.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: native port Host native arch port (native_posix) area: West West utility Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

support 'west flash' with native_sim, add debugserver support nashif/zephyr
4 participants
@kartben @nashif @palchak-google @aescolar