[libcamera-devel] [PATCH v7 00/11] media: Report camera sensor properties

Jacopo Mondi jacopo at jmondi.org
Wed Mar 18 21:50:23 CET 2020


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



More information about the libcamera-devel mailing list