Opened 3 years ago

Closed 3 years ago

#11699 closed defect (fixed)

Camera still thumbnail is captured before camera is ready in Record Audio Capture

Reported by: greenfeld Owned by: saadia
Priority: normal Milestone: 11.3.1
Component: kernel Version: Development build as of this date
Keywords: Cc:
Blocked By: Blocking:
Deployments affected: Action Needed: add to build
Verified: no

Description

When testing the Audio-only (+ preview frame) recording functionality in Record, it looks the Camera may capture a single frame mid-frame and not wait for vsync sometimes.

The preview video gives no hint this may occur as it does not seem to flicker.

An example frame capture showing this problem from 11.3.1 os30 + a 1e953b13f -ramp kernel is attached.

Attachments (1)

midscan.png (91.0 KB) - added by greenfeld 3 years ago.
Record preview photo (base64 decoded from preview file) showing brightness change midframe

Download all attachments as: .zip

Change History (6)

Changed 3 years ago by greenfeld

Record preview photo (base64 decoded from preview file) showing brightness change midframe

comment:1 Changed 3 years ago by saadia

  • Owner set to saadia
  • Summary changed from Camera may capture a single frame mid-frame and not wait for vsync to Camera still thumbnail is captured before camera is ready in Record Audio Capture

Changed the name of this bug, as a variety of dim images are a result of this bug. When audio capture is initiated in Record, the camera is powered down, then up, then down and up again when audio capture is stopped. The still captured frame is grabbed before the camera is ready, hence the dim frame.

comment:2 Changed 3 years ago by saadia

  • Action Needed changed from diagnose to test in build
  • Cc greenfield@… added

I've put in a fix for this problem in arm-3.0-wip
http://dev.laptop.org/git/olpc-kernel/commit/?h=arm-3.0-wip&id=5de2547434c225f7dec0fe18718cb4e5758c5605
Please test and let me know if it should be pushed to ramp/

comment:3 Changed 3 years ago by greenfeld

  • Action Needed changed from test in build to add to build

Tested the fix with a monolithic -wip kernel at 5de2547.

The fix seems to work so it should be added to the -ramp kernel, and then added to the next os build.

comment:4 Changed 3 years ago by greenfeld

  • Cc greenfield@… removed

comment:5 Changed 3 years ago by martin.langhoff

  • Milestone changed from 1.75-software to 11.3.1
  • Resolution set to fixed
  • Status changed from new to closed

This has been fixed for a while.

Note: See TracTickets for help on using tickets.