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

Add camera library. #41

Merged
merged 4 commits into from
Jan 29, 2025
Merged

Add camera library. #41

merged 4 commits into from
Jan 29, 2025

Conversation

iabdalkader
Copy link

No description provided.

@facchinm
Copy link
Member

Wonderful! I'd drop 861c4aa (see comment)

@iabdalkader
Copy link
Author

Note that this will not build until we get my recent fixes upstream.

@iabdalkader
Copy link
Author

Updated. This will not build or work until all upstream fixes are megred.
Also a25ca3f
broke this library. @facchinm

@facchinm
Copy link
Member

Yup, I'm going to add the patches for core zephyr to our branch in the meantime, and fixup the breakage introduced by a25ca3f

- Enable video subsystem, DCMI and sensors.
- Switch overlay to GC2145 sensor.
- Enable shared multi-heap.

Signed-off-by: iabdalkader <[email protected]>
@iabdalkader
Copy link
Author

Yup, I'm going to add the patches for core zephyr to our branch in the meantime, and fixup the breakage introduced by a25ca3f

They're not all merged yet, so I would wait for now. If you can fix the pwm issue, I can retest it and move on to something else.

The granularity of the camera clock was too low otherwise and the camera failed to start
@facchinm
Copy link
Member

Pushed 0ddba96 to tackle the regression introduced by a25ca3f , tested and got my first frames 🙂
There's one thing though, I keep also getting spammed on the debug console with [00:01:50.679,000] <dbg> video_stm32_dcmi: HAL_DCMI_FrameEventCallback: Failed to get buffer from fifo
Any hint about what could be happening?
2025-01-27-124438_1920x1080_scrot

@facchinm
Copy link
Member

As per zephyr base, just run a west update while in the venv to pull your commits (I merged them in our fork here https://github.com/arduino/zephyr/tree/arduino_core_merge_4.0.99 )

@iabdalkader
Copy link
Author

HAL_DCMI_FrameEventCallback: Failed to get buffer from fifo

No issue, it's just the FIFO is full, won't stop the streaming. Try more framebuffers.

@facchinm facchinm merged commit b28f7a1 into arduino:arduino Jan 29, 2025
3 of 4 checks passed
@iabdalkader iabdalkader deleted the camera_library branch January 29, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants