mbox series

[v7,00/11] media: Report camera sensor properties

Message ID 20200318205034.949531-1-jacopo@jmondi.org
Headers show
Series media: Report camera sensor properties | expand

Message

Jacopo Mondi March 18, 2020, 8:50 p.m. UTC
This series has been floating around for quite a long time already.

This v7 is rebased on latest media master and the only change is actually
the rotation property description.

While working on libcamera we have gone through a -long- discussion and several
attempts to provide a precise definition for the camera rotation.  We came up
with a pretty lengthy description which I have replicated exactly here in the
DTS property definition. The description is meant to be read by integrators
which need a reference in order to properly calculate the rotation of their
camera modules. The property is thus expanded to be defined as the rotation
difference between to reference systems, precisely defined to reduce the
risk of having different interpretations between different camera module
integrators.

In the V4L2_CID_CAMERA_SENSOR_ROTATION control documentation I tried to provide
a description on how to use the control from a user point of view. The
description is meant to be read by application developers, which needs to
know how to rotate images to compensate the camera rotation. The value of the
control is exactly the same as the property one, but described from a consumer
point of view.

The rest of the series is unchanged.

Thanks
   j

Jacopo Mondi (11):
  dt-bindings: video-interfaces: Document 'location' property
  media: v4l2-ctrl: Document V4L2_CID_CAMERA_SENSOR_LOCATION
  dt-bindings: video-interface: Replace 'rotation' description
  media: v4l2-ctrl: Document V4L2_CID_CAMERA_SENSOR_ROTATION
  media: v4l2-ctrls: Add camera location and rotation
  media: v4l2-fwnode: Add helper to parse device properties
  include: v4l2-ctrl: Sort forward declarations
  media: v4l2-ctrls: Sort includes alphabetically
  media: v4l2-ctrls: Add helper to register properties
  media: i2c: ov5670: Parse and register properties
  media: i2c: ov13858: Parse and register properties

 .../bindings/media/video-interfaces.txt       | 372 +++++++++++++++++-
 .../media/uapi/v4l/ext-ctrls-camera.rst       | 153 +++++++
 drivers/media/i2c/ov13858.c                   |  11 +
 drivers/media/i2c/ov5670.c                    |  12 +
 drivers/media/v4l2-core/v4l2-ctrls.c          |  52 ++-
 drivers/media/v4l2-core/v4l2-fwnode.c         |  42 ++
 include/media/v4l2-ctrls.h                    |  34 +-
 include/media/v4l2-fwnode.h                   |  47 +++
 include/uapi/linux/v4l2-controls.h            |   7 +
 9 files changed, 718 insertions(+), 12 deletions(-)

--
2.25.1