Skip to content

Conversation

@josuah
Copy link
Contributor

@josuah josuah commented May 20, 2024

"Video subsystem for Zephyr" is actually only defined as a driver and the examples include <zephyr/drivers/video.h>.

This moves the sample/subsys/video to the sample/drivers/video directory to reflect that.

@josuah
Copy link
Contributor Author

josuah commented May 20, 2024

carlescufi
carlescufi previously approved these changes Jun 6, 2024
@josuah josuah force-pushed the pr-samples-video-to-drivers branch from bd312a2 to dc0c864 Compare June 6, 2024 12:03
@josuah
Copy link
Contributor Author

josuah commented Jun 6, 2024

I rebased for solving a merge conflict.

@josuah
Copy link
Contributor Author

josuah commented Jun 6, 2024

#66994 already picked this new directory tree, but no conflict regardless of the order they are added.
Probably still interesting to merge #66994 first as it is waiting for review since 6 months.

Copy link
Contributor

@kartben kartben left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you please add the appropriate redirect entries in doc/_scripts/redirects.py?
This allows to properly redirect people browsing e.g. https://docs.zephyrproject.org/3.6.0/samples/subsys/video/capture/README.html and clicking on "see the most recent version... " link

"Video subsystem for Zephyr" is actually only defined as a driver
and the examples actually include <zephyr/drivers/video.h>.

Signed-off-by: Josuah Demangeon <[email protected]>
@josuah josuah force-pushed the pr-samples-video-to-drivers branch from dc0c864 to 6ac69d5 Compare July 27, 2024 21:29
@zephyrbot zephyrbot requested a review from gmarull July 27, 2024 21:30
@josuah josuah requested a review from kartben July 28, 2024 00:56
@josuah josuah added the area: Video Video subsystem label Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants