Skip to content

Conversation

@CharlesDias
Copy link
Contributor

@CharlesDias CharlesDias commented Feb 8, 2025

Update the st,stm32-dcmi.yaml to use the common properties defined in video-interfaces.yaml.

@ngphibang @josuah @erwango, is it possible to add the capture-rate property into video-interfaces.yaml?

@uLipe, this PR updates the zephyr/boards/arduino/nicla_vision/arduino_nicla_vision_stm32h747xx_m7.dts.

Issue : #80514

Update to use the common properties defined in video-interfaces.yaml

Signed-off-by: Charles Dias <[email protected]>
@ngphibang
Copy link
Contributor

ngphibang commented Feb 8, 2025

"capture-rate" is actually the frame rate ? :-). I think you need to implement it in a "correct" way by using the set/get/enum_frival() APIs because the frame rate is not a property that relates only to the current device, here the DCMI (receiver), but it also depends on other devices along the pipeline (e.g., the camera sensor).

A reference implementation could be found the in the RT1170 camera pipeline : ov5640 --> mipi_csi2rx --> csi

@CharlesDias CharlesDias marked this pull request as draft February 8, 2025 19:15
@CharlesDias
Copy link
Contributor Author

Let's go back to the drawing board! ;-)

@github-actions
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Apr 10, 2025
@github-actions github-actions bot closed this Apr 24, 2025
@ngphibang
Copy link
Contributor

@CharlesDias Would you have time to complete this PR ? If framerate support is difficult, I think we can do the easy parts first and merge it first, e.g. moving to video interface binding and adding build_all/video's built-only test for the dcmi driver (this is important to have CI run on this driver each time we modify the driver).
Thanks

@CharlesDias
Copy link
Contributor Author

Hi, @ngphibang .

I'll have return to this around mid-May. 😄

@CharlesDias CharlesDias deleted the stm32_dcmi_video_interface branch October 25, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Shields Shields (add-on boards) area: Video Video subsystem platform: STM32 ST Micro STM32 Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants