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

*native_posix* driver name renaming #86480

Open
aescolar opened this issue Feb 28, 2025 · 5 comments
Open

*native_posix* driver name renaming #86480

aescolar opened this issue Feb 28, 2025 · 5 comments
Assignees
Labels
area: native port Host native arch port (native_sim) RFC Request For Comments: want input from the community

Comments

@aescolar
Copy link
Member

aescolar commented Feb 28, 2025

We have quite a few drivers with native_posix in their name, DTS and/or kconfig options and file names.
Now that native_posix is being removed we have two options: We let them be, or we rename them
following appropriate deprecation.
Letting them be as they are could be a bit confusing for users, but minimizes churn, and viceversa.
We also don't necessarily need to rename all.

Possible renaming:

  • native_sim board specific drivers:
    • native_posix_timer => native_sim_timer
      • CONFIG_NATIVE_POSIX_TIMER => CONFIG_NATIVE_SIM_TIMER
    • counter_native_posix => counter_native_sim
  • native simulator specific (can be used in all current POSIX arch targets):
    • fake_entropy_native_posix => fake_entropy_native_sim
      • zephyr_native_posix_rng => zephyr_native_sim_rng
      • CONFIG_FAKE_ENTROPY_NATIVE_POSIX => CONFIG_FAKE_ENTROPY_NATIVE_SIM
    • CONFIG_UART_NATIVE_POSIX => CONFIG_UART_NATIVE_PTTY
    • CONFIG_ETH_NATIVE_POSIX => CONFIG_ETH_NATIVE_TUN_TAP ?
  • For any posix arch target:
    • log_backend_native_posix => log_backend_native ?
    • usb_native_posix ?
  • DTS zephyr,native-posix-cpu => zephyr,native-sim-cpu ?
@aescolar aescolar added the RFC Request For Comments: want input from the community label Feb 28, 2025
@aescolar
Copy link
Member Author

@tmon-nordic
Copy link
Contributor

USB_NATIVE_POSIX will be depreated alongside the old USB stack. The implementation is completely wrong. The new stack has better USBIP support which does not share the USB_NATIVE_POSIX problems (including, but not limited to confusing name).

@aescolar aescolar added the area: native port Host native arch port (native_sim) label Feb 28, 2025
@aescolar aescolar self-assigned this Feb 28, 2025
@jukkar
Copy link
Member

jukkar commented Feb 28, 2025

No objection for changing native_posix -> native_sim in Ethernet driver. The driver has not seen much changes lately so I am not expecting merge issues in the future. Renaming Kconfig options is of course a bigger change that might affect users but I suppose that is unavoidable.

@ceolin
Copy link
Member

ceolin commented Feb 28, 2025

@aescolar No objections on my side for the entropy as well :)

@dcpleung
Copy link
Member

CONFIG_UART_NATIVE_PTTY is better since the source file is named uart_native_ptty.c already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: native port Host native arch port (native_sim) RFC Request For Comments: want input from the community
Projects
Status: No status
Development

No branches or pull requests

5 participants