Hi Jacopo, Thank you for the patch.
On Tue, Aug 27, 2019 at 11:23:29AM +0200, Jacopo Mondi wrote: > Add documentation for the V4L2_CID_CAMERA_SENSOR_ROTATION camera > control. The newly added read-only control reports the camera device > mounting rotation. > > Signed-off-by: Jacopo Mondi <jac...@jmondi.org> > --- > Documentation/media/uapi/v4l/ext-ctrls-camera.rst | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/Documentation/media/uapi/v4l/ext-ctrls-camera.rst > b/Documentation/media/uapi/v4l/ext-ctrls-camera.rst > index ecf151f3f0f4..03d1c23d18f7 100644 > --- a/Documentation/media/uapi/v4l/ext-ctrls-camera.rst > +++ b/Documentation/media/uapi/v4l/ext-ctrls-camera.rst > @@ -544,6 +544,15 @@ enum v4l2_scene_mode - > > > > +``V4L2_CID_CAMERA_SENSOR_ROTATION (integer)`` > + This read-only control describes the camera sensor orientation by > + reporting its mounting rotation in respect to the device intended usage > + orientation, expressed in counter clockwise degrees. The control value is Counter clockwise relative to what ? :-) > + constant and not modifiable by software and its value is retrieved from > the > + firmware interface by parsing the 'rotation' property. Typical values > are 0 > + degrees for upright mounted sensors and 180 degrees for sensors mounted > + upside down. Same comment as for 02/10 regarding how the value is known to the driver. > + > .. [#f1] > This control may be changed to a menu control in the future, if more > options are required. -- Regards, Laurent Pinchart
