diff mbox

V4L2: Add documentation for SDI timings and related flags

Message ID 1469113476-1645-1-git-send-email-charles-antoine.couret@nexvision.fr (mailing list archive)
State New, archived
Headers show

Commit Message

Charles-Antoine Couret July 21, 2016, 3:04 p.m. UTC
Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@nexvision.fr>
---
 Documentation/media/uapi/v4l/vidioc-enuminput.rst  | 31 +++++++++++++++++-----
 .../media/uapi/v4l/vidioc-g-dv-timings.rst         | 15 +++++++++++
 2 files changed, 39 insertions(+), 7 deletions(-)

Comments

Hans Verkuil Aug. 4, 2016, 10:11 a.m. UTC | #1
On 07/21/2016 05:04 PM, Charles-Antoine Couret wrote:
> Signed-off-by: Charles-Antoine Couret <charles-antoine.couret@nexvision.fr>
> ---
>  Documentation/media/uapi/v4l/vidioc-enuminput.rst  | 31 +++++++++++++++++-----
>  .../media/uapi/v4l/vidioc-g-dv-timings.rst         | 15 +++++++++++
>  2 files changed, 39 insertions(+), 7 deletions(-)
> 

<snip>

> diff --git a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
> index f7bf21f..9acfa19 100644
> --- a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
> +++ b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
> @@ -339,6 +339,13 @@ EBUSY
>  
>         -  The timings follow the VESA Generalized Timings Formula standard
>  
> +    -  .. row 7
> +
> +       -  ``V4L2_DV_BT_STD_SDI``
> +
> +       -  The timings follow the SDI Timings standard.
> +	  There are no horizontal syncs/porches at all in this format.
> +	  Total blanking timings must be set in hsync or vsync fields only.

Didn't you mention on irc that there are actually two blanking timings for
vertical blanking? Something frontporch like? I can't remember the details,
but if I remember correctly, then you should specify what goes where.

Regards,

	Hans

>  
>  
>  .. _dv-bt-flags:
> @@ -415,3 +422,11 @@ EBUSY
>  	  R'G'B' values use limited range (i.e. 16-235) as opposed to full
>  	  range (i.e. 0-255). All formats defined in CEA-861 except for the
>  	  640x480p59.94 format are CE formats.
> +
> +    -  .. row 8
> +
> +       -  ``V4L2_DV_FL_FIRST_FIELD_EXTRA_LINE``
> +
> +       -  Some formats like SMPTE-125M have an interlaced signal with a odd
> +	  total height. For these formats, if this flag is set, the first
> +	  field has the extra line. Else, it is the second field.
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Charles-Antoine Couret Aug. 4, 2016, 3:21 p.m. UTC | #2
Le 04/08/2016 à 12:11, Hans Verkuil a écrit :

>> --- a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
>> +++ b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
>> @@ -339,6 +339,13 @@ EBUSY
>>  
>>         -  The timings follow the VESA Generalized Timings Formula standard
>>  
>> +    -  .. row 7
>> +
>> +       -  ``V4L2_DV_BT_STD_SDI``
>> +
>> +       -  The timings follow the SDI Timings standard.
>> +	  There are no horizontal syncs/porches at all in this format.
>> +	  Total blanking timings must be set in hsync or vsync fields only.
> 
> Didn't you mention on irc that there are actually two blanking timings for
> vertical blanking? Something frontporch like? I can't remember the details,
> but if I remember correctly, then you should specify what goes where.


Yes, your're right about this.
In SMPTE 125M (I like this standard :D), some "frontporchs" or similar values are available.
So, I can precise that in the documentation and fix SMPTE 125M definition to take this into account.

Thank you very much.
Regards,
Charles-Antoine Couret
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/media/uapi/v4l/vidioc-enuminput.rst b/Documentation/media/uapi/v4l/vidioc-enuminput.rst
index 5060f54..18331b9 100644
--- a/Documentation/media/uapi/v4l/vidioc-enuminput.rst
+++ b/Documentation/media/uapi/v4l/vidioc-enuminput.rst
@@ -260,17 +260,34 @@  at index zero, incrementing by one until the driver returns ``EINVAL``.
 
     -  .. row 11
 
-       -  :cspan:`2` Digital Video
+       -  ``V4L2_IN_ST_NO_V_LOCK``
+
+       -  0x00000400
+
+       -  No vertical sync lock.
 
     -  .. row 12
 
+       -  ``V4L2_IN_ST_NO_STD_LOCK``
+
+       -  0x00000800
+
+       -  No standard format lock in case of auto-detection format
+	  by the component.
+
+    -  .. row 13
+
+       -  :cspan:`2` Digital Video
+
+    -  .. row 14
+
        -  ``V4L2_IN_ST_NO_SYNC``
 
        -  0x00010000
 
        -  No synchronization lock.
 
-    -  .. row 13
+    -  .. row 15
 
        -  ``V4L2_IN_ST_NO_EQU``
 
@@ -278,7 +295,7 @@  at index zero, incrementing by one until the driver returns ``EINVAL``.
 
        -  No equalizer lock.
 
-    -  .. row 14
+    -  .. row 16
 
        -  ``V4L2_IN_ST_NO_CARRIER``
 
@@ -286,11 +303,11 @@  at index zero, incrementing by one until the driver returns ``EINVAL``.
 
        -  Carrier recovery failed.
 
-    -  .. row 15
+    -  .. row 17
 
        -  :cspan:`2` VCR and Set-Top Box
 
-    -  .. row 16
+    -  .. row 18
 
        -  ``V4L2_IN_ST_MACROVISION``
 
@@ -300,7 +317,7 @@  at index zero, incrementing by one until the driver returns ``EINVAL``.
 	  signal to confuse video recorders. When this flag is set
 	  Macrovision has been detected.
 
-    -  .. row 17
+    -  .. row 19
 
        -  ``V4L2_IN_ST_NO_ACCESS``
 
@@ -308,7 +325,7 @@  at index zero, incrementing by one until the driver returns ``EINVAL``.
 
        -  Conditional access denied.
 
-    -  .. row 18
+    -  .. row 20
 
        -  ``V4L2_IN_ST_VTR``
 
diff --git a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
index f7bf21f..9acfa19 100644
--- a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
+++ b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
@@ -339,6 +339,13 @@  EBUSY
 
        -  The timings follow the VESA Generalized Timings Formula standard
 
+    -  .. row 7
+
+       -  ``V4L2_DV_BT_STD_SDI``
+
+       -  The timings follow the SDI Timings standard.
+	  There are no horizontal syncs/porches at all in this format.
+	  Total blanking timings must be set in hsync or vsync fields only.
 
 
 .. _dv-bt-flags:
@@ -415,3 +422,11 @@  EBUSY
 	  R'G'B' values use limited range (i.e. 16-235) as opposed to full
 	  range (i.e. 0-255). All formats defined in CEA-861 except for the
 	  640x480p59.94 format are CE formats.
+
+    -  .. row 8
+
+       -  ``V4L2_DV_FL_FIRST_FIELD_EXTRA_LINE``
+
+       -  Some formats like SMPTE-125M have an interlaced signal with a odd
+	  total height. For these formats, if this flag is set, the first
+	  field has the extra line. Else, it is the second field.