Skip to content

drivers: video: stm32_dcmipp: move video/ header directory to drivers/ #92784

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

Conversation

josuah
Copy link
Collaborator

@josuah josuah commented Jul 7, 2025

Moving a header added by mistake on the wrong place... I missed it during #89407 review!

No Zephyr code relies on it currently, it is only there in the meantime that APIs get designed for hardware ISP features.

This was introduced recently so an avoid a breaking change in 4.3 if merged in 4.2. otherwise generic API for it can land in 4.3 <zephyr/drivers/video.h>.

$ west twister -i -s drivers.video.stm32_dcmipp.build
Renaming output directory to /home/tinyvision/zephyrproject/zephyr/twister-out.1
INFO    - Using Ninja..
INFO    - Zephyr version: v4.2.0-rc1-225-g42f54973b231
INFO    - Using 'zephyr' toolchain.
INFO    - Selecting default platforms per testsuite scenario
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/tinyvision/zephyrproject/zephyr/twister-out/testplan.json
INFO    - JOBS: 4
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    1/   1  100%  built (not run):    1, filtered:    0, failed:    0, error:    0
INFO    - 1 test scenarios (1 configurations) selected, 0 configurations filtered (0 by static filter, 0 at runtime).
INFO    - 0 of 1 executed test configurations passed (0.00%), 1 built (not run), 0 failed, 0 errored, with no warnings in 96.43 seconds.
INFO    - 0 of 0 executed test cases passed (0.00%) on 0 out of total 1115 platforms (0.00%).
INFO    - 1 selected test cases not executed: 1 not run (built only).
INFO    - 0 test configurations executed on platforms, 1 test configurations were only built.
INFO    - Saving reports...
INFO    - Writing JSON report /home/tinyvision/zephyrproject/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/tinyvision/zephyrproject/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/tinyvision/zephyrproject/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

During zephyrproject-rtos#89407 a driver-specific header stm32_dcmipp.h got introduced to
support new functionnalities not yet covered by video APIs.
Move this header into a new <include/zephyr/drivers/video/> include
direcctory like what other driver classes do.

Signed-off-by: Josuah Demangeon <me@josuah.net>
@josuah josuah added this to the v4.2.0 milestone Jul 7, 2025
@josuah josuah requested a review from avolmat-st July 7, 2025 13:39
@josuah josuah added platform: STM32 ST Micro STM32 area: Video Video subsystem labels Jul 7, 2025
@github-actions github-actions bot added the size: XS A PR changing only a single line of code label Jul 7, 2025
Copy link

sonarqubecloud bot commented Jul 7, 2025

@avolmat-st
Copy link
Collaborator

Thanks @josuah !

@danieldegrasse danieldegrasse merged commit e0b706b into zephyrproject-rtos:main Jul 9, 2025
30 checks passed
@josuah josuah deleted the pr_video_move_header branch July 9, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Video Video subsystem platform: STM32 ST Micro STM32 size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants