On 09/07/2016 11:26 AM, Charles-Antoine Couret wrote:
> Le 12/08/2016 à 15:17, Hans Verkuil a écrit :
>> On 08/04/2016 05:39 PM, Charles-Antoine Couret wrote:
>>
>> A commit log is missing here.
>
> Yeah I will fix that.
>
>>> diff --git a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
>>> b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
>>> index f7bf21f..0205bf6 100644
>>> --- a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
>>> +++ b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst
>>> @@ -339,6 +339,14 @@ 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 not always horizontal syncs/porches or similar in this
>>> format.
>>> + If it is not precised by standard, blanking timings must be set in
>>> + hsync or vsync fields by default.
>>
>> OK. This is confusing. The text was changed after my question about
>> something porch-like
>> in the SMPTE-125M standard. But I see nothing like that after re-reading it.
>>
>> So what sort of 'porch' timing were you thinking of?
>
> In SMPTE-125M for example, the time between the real horizontal blanking is
> precised (16 pixelclock).
> For me it looks like front porch timing.
Well, for some variants it is actually half-timings (21.5). In addition, it
doesn't
seem to be used at all, it is just to relate the analog hsync to the digital
sample.
>> I wonder if I shouldn't just use the text from your first patch:
>>
>> - ``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.
>
> I agree with that if you prefer, after all the front/backporch are probably
> irrelevant in this case.
> So, if you confirm this way, I would send you another patchset to fix that.
I think we should stick to this text.
Regards,
Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
