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

drivers: video: sensors: add a sensor_skeleton.c to speed-up contribution of simple sensors #73867

Closed
josuah opened this issue Jun 6, 2024 · 2 comments · Fixed by #79482
Closed
Assignees
Labels
area: Video Video subsystem Enhancement Changes/Updates/Additions to existing features priority: low Low impact/importance bug

Comments

@josuah
Copy link
Collaborator

josuah commented Jun 6, 2024

Is your enhancement proposal related to a problem? Please describe.
There are more sensor contributions coming to Zephyr, from various location, and not yet guidelines for how to contribute them

Describe the solution you'd like
Having a sensor_skeleton.c would help maintainer communicate how they plan to integrate sensors, and help image sensor driver contributors to get going more quickly by skipping all the boilerplate.

Describe alternatives you've considered

  • Have a video-wide skeleton instead
  • Let things like they are now: no big priority.

Additional context
Other similar "template/skeleton/dummy" drivers:
https://github.com/zephyrproject-rtos/zephyr/blob/main/drivers/usb/udc/udc_skeleton.c
https://github.com/zephyrproject-rtos/zephyr/blob/main/drivers/display/display_dummy.c

@josuah josuah added the Enhancement Changes/Updates/Additions to existing features label Jun 6, 2024
@josuah
Copy link
Collaborator Author

josuah commented Jun 13, 2024

@josuah
Copy link
Collaborator Author

josuah commented Nov 29, 2024

Closing in favor of #79482 which permits to actually run the skeleton files, making sure the stub works.

@josuah josuah closed this as completed Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Video Video subsystem Enhancement Changes/Updates/Additions to existing features priority: low Low impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants