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

fix: #80, v4l2_drm dual-camera demo glitch, refactor with single-thread, fix AE unstable problem #101

Merged
merged 14 commits into from
Jun 7, 2022

Conversation

MrThanlon
Copy link
Member

@MrThanlon MrThanlon commented Jun 7, 2022

PR描述:

修复v4l2_drm的各种问题,包括水波纹和跳帧,改为单线程实现。

详细描述:

为了便于测试,n_buffer指定为14,有非常大的延迟,最终合并前需要改为5(已修改)

这个分支使用的单线程比较简单粗暴,就是在等待vsync的同时也检查v4l2是否能获取新的buffer,如果能则直接QBUF回去,但实际上我们是完全可以利用起来的,这种做法会导致帧率降低,但是能保证AE正常。#100 则更直观一些,但是AE不正常。

关联issue:

#80

MrThanlon added 12 commits May 30, 2022 15:21
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
@MrThanlon MrThanlon added the bug Something isn't working label Jun 7, 2022
@MrThanlon MrThanlon added this to the v1.7 milestone Jun 7, 2022
MrThanlon added 2 commits June 7, 2022 15:04
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
Signed-off-by: 黄子懿 <huangziyi@canaan-creative.com>
@MrThanlon MrThanlon changed the title fix: #80, v4l2_drm dual-camera demo glitch with single-thread and AE fix: #80, v4l2_drm dual-camera demo glitch, refactor with single-thread, fix AE unstable problem Jun 7, 2022
@MrThanlon MrThanlon requested a review from longyiluo June 7, 2022 09:58
@MrThanlon MrThanlon marked this pull request as ready for review June 7, 2022 10:00
@MrThanlon MrThanlon requested a review from zzxcanaan June 7, 2022 11:27
@zzxcanaan
Copy link
Collaborator

LGTM

@zhangxiaojingCAN zhangxiaojingCAN merged commit 625d2c5 into kendryte:dev Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants