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

eespressif: ParallelImageCapture: keep peripheral running #5457

Merged
merged 1 commit into from
Oct 12, 2021

Conversation

jepler
Copy link
Member

@jepler jepler commented Oct 11, 2021

Otherwise, a lot of frames were being dropped, leading to a low framerate in the "directio" demo for OV5640 cameras

Before:
image (3)

After:
image (4)

Note that as we're not doing anything to overlap the display write with the capture of the next frame, it's expected that 2 vsync pulses are needed for one display refresh.

Otherwise, a lot of frames were being dropped, leading to a low framerate
in the "directio" demo for OV5640 cameras
@ladyada
Copy link
Member

ladyada commented Oct 12, 2021

ok tested, it def cuts down on capture time, now 400ms instead of 600ms. still - why are we only able to get 4mhz pixclk? i feel like we should be zooming thru the data. and why does hqvga take the same amount of time as qvga??? so many ????!

@ladyada ladyada merged commit 1099716 into adafruit:main Oct 12, 2021
@jepler
Copy link
Member Author

jepler commented Oct 12, 2021

Yeah, the OV5640 datasheet says we can do up to 120fps at 320x240, with a pixel clock of "24/48MHz". Clock/mode setting would need to be an issue filed against Adafruit_CircuitPython_OV5460. Adding background capturing would be an issue to add in the core first, then in the driver.

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