Skip to content
Snippets Groups Projects

s5k3l6xx: Integration time control

All threads resolved!

Merge request reports

Pipeline #67517 passed

Pipeline passed for 791a4536 on sebastian.krzyszkowiak:exposure

Approval is optional

Merged by Dorota CzaplejewiczDorota Czaplejewicz 3 years ago (May 11, 2021 12:47pm UTC)

Merge details

  • Changes merged into pureos/byzantium with 791a4536.
  • Deleted the source branch.

Pipeline #67524 passed

Pipeline passed for 791a4536 on pureos/byzantium

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Exposure should be expressed in units of 1s/10000, so I will accept this only very reluctantly.

    The exposure time is limited by the frame interval. Drivers should interpret the values as 100 µs units, where the value 1 stands for 1/10000th of a second, 10000 for 1 second and 100000 for 10 seconds.

    https://www.kernel.org/doc/html/v4.10/media/uapi/v4l/extended-controls.html#camera-control-ids

    I guess if we want to be all proper, then we won't have the control at all for some time, but it would be nice to see some affordances for changing the unit later.

  • mentioned in merge request !382 (merged)

  • added 26 commits

    Compare with previous version

  • Dorota Czaplejewicz resolved all threads

    resolved all threads

  • It does get reset after taking a picture. I wonder when exactly.

  • Please register or sign in to reply
    Loading