Ticket #11699 (closed defect: fixed)

Opened 14 months ago

Last modified 12 months ago

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:
Action Needed: add to build Verified: no
Deployments affected: Blocked By:
Blocking:

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

midscan.png (91.0 kB) - added by greenfeld 14 months ago.
Record preview photo (base64 decoded from preview file) showing brightness change midframe

Change History

Changed 14 months ago by greenfeld

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

Changed 14 months 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.

Changed 14 months ago by saadia

  • cc greenfield@… added
  • next_action changed from diagnose to test in build

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/

Changed 14 months ago by greenfeld

  • next_action 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.

Changed 14 months ago by greenfeld

  • cc greenfield@… removed

Changed 12 months ago by martin.langhoff

  • status changed from new to closed
  • resolution set to fixed
  • milestone changed from 1.75-software to 11.3.1

This has been fixed for a while.

Note: See TracTickets for help on using tickets.