diff mbox

Doc*/media/uapi: fix control name

Message ID 20170527081239.GA9484@amd (mailing list archive)
State Accepted, archived
Delegated to: Hans Verkuil
Headers

Commit Message

Pavel Machek May 27, 2017, 8:12 a.m. UTC 
  V4L2_CID_EXPOSURE_BIAS does not exist, fix documentation.

Signed-off-by: Pavel Machek <pavel@ucw.cz>

Comments

Sakari Ailus May 27, 2017, 7:30 p.m. UTC | #1 
On Sat, May 27, 2017 at 10:12:40AM +0200, Pavel Machek wrote:
> V4L2_CID_EXPOSURE_BIAS does not exist, fix documentation.
> 
> Signed-off-by: Pavel Machek <pavel@ucw.cz>
> 
> diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst
> index abb1057..76c5b1a 100644
> --- a/Documentation/media/uapi/v4l/extended-controls.rst
> +++ b/Documentation/media/uapi/v4l/extended-controls.rst
> @@ -2019,7 +2019,7 @@ enum v4l2_exposure_auto_type -
>      dynamically vary the frame rate. By default this feature is disabled
>      (0) and the frame rate must remain constant.
>  
> -``V4L2_CID_EXPOSURE_BIAS (integer menu)``
> +``V4L2_CID_AUTO_EXPOSURE_BIAS (integer menu)``
>      Determines the automatic exposure compensation, it is effective only
>      when ``V4L2_CID_EXPOSURE_AUTO`` control is set to ``AUTO``,
>      ``SHUTTER_PRIORITY`` or ``APERTURE_PRIORITY``. It is expressed in
> 

Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>

Generally linux-media is enough, for other lists such as LKML this is just
noise.
Pavel Machek May 27, 2017, 9:06 p.m. UTC | #2 
On Sat 2017-05-27 22:30:35, Sakari Ailus wrote:
> On Sat, May 27, 2017 at 10:12:40AM +0200, Pavel Machek wrote:
> > V4L2_CID_EXPOSURE_BIAS does not exist, fix documentation.
> > 
> > Signed-off-by: Pavel Machek <pavel@ucw.cz>
> > 
> > diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst
> > index abb1057..76c5b1a 100644
> > --- a/Documentation/media/uapi/v4l/extended-controls.rst
> > +++ b/Documentation/media/uapi/v4l/extended-controls.rst
> > @@ -2019,7 +2019,7 @@ enum v4l2_exposure_auto_type -
> >      dynamically vary the frame rate. By default this feature is disabled
> >      (0) and the frame rate must remain constant.
> >  
> > -``V4L2_CID_EXPOSURE_BIAS (integer menu)``
> > +``V4L2_CID_AUTO_EXPOSURE_BIAS (integer menu)``
> >      Determines the automatic exposure compensation, it is effective only
> >      when ``V4L2_CID_EXPOSURE_AUTO`` control is set to ``AUTO``,
> >      ``SHUTTER_PRIORITY`` or ``APERTURE_PRIORITY``. It is expressed in
> > 
> 
> Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> 
> Generally linux-media is enough, for other lists such as LKML this is just
> noise.

Well, this is what get_maintainer suggested. LKML is used to a lot of
traffic ;-) [I know, I am subscribed.]
									Pavel
diff mbox

Patch

diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst
index abb1057..76c5b1a 100644
--- a/Documentation/media/uapi/v4l/extended-controls.rst
+++ b/Documentation/media/uapi/v4l/extended-controls.rst
@@ -2019,7 +2019,7 @@  enum v4l2_exposure_auto_type -
     dynamically vary the frame rate. By default this feature is disabled
     (0) and the frame rate must remain constant.
 
-``V4L2_CID_EXPOSURE_BIAS (integer menu)``
+``V4L2_CID_AUTO_EXPOSURE_BIAS (integer menu)``
     Determines the automatic exposure compensation, it is effective only
     when ``V4L2_CID_EXPOSURE_AUTO`` control is set to ``AUTO``,
     ``SHUTTER_PRIORITY`` or ``APERTURE_PRIORITY``. It is expressed in