diff mbox

[v3,07/11] Documentation: v4l: Documentation for HEVC v4l2 definition

Message ID 1490951200-32070-8-git-send-email-smitha.t@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Smitha T Murthy March 31, 2017, 9:06 a.m. UTC
Add V4L2 definition for HEVC compressed format

Signed-off-by: Smitha T Murthy <smitha.t@samsung.com>
Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
---
 Documentation/media/uapi/v4l/pixfmt-013.rst | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Hans Verkuil April 3, 2017, 8:10 a.m. UTC | #1
On 03/31/2017 11:06 AM, Smitha T Murthy wrote:
> Add V4L2 definition for HEVC compressed format
> 
> Signed-off-by: Smitha T Murthy <smitha.t@samsung.com>
> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
> ---
>  Documentation/media/uapi/v4l/pixfmt-013.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/media/uapi/v4l/pixfmt-013.rst b/Documentation/media/uapi/v4l/pixfmt-013.rst
> index 728d7ed..ff4cac2 100644
> --- a/Documentation/media/uapi/v4l/pixfmt-013.rst
> +++ b/Documentation/media/uapi/v4l/pixfmt-013.rst
> @@ -90,3 +90,8 @@ Compressed Formats
>        - ``V4L2_PIX_FMT_VP9``
>        - 'VP90'
>        - VP9 video elementary stream.
> +    * .. _V4L2-PIX-FMT-HEVC:
> +
> +      - ``V4L2_PIX_FMT_HEVC``
> +      - 'HEVC'
> +      - HEVC video elementary stream.
> 

You should mention here that HEVC == H.265.

Regards,

	Hans
Hans Verkuil April 3, 2017, 12:12 p.m. UTC | #2
On 04/03/2017 02:12 PM, Smitha T Murthy wrote:
> On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote:
>> On 03/31/2017 11:06 AM, Smitha T Murthy wrote:
>>> Add V4L2 definition for HEVC compressed format
>>>
>>> Signed-off-by: Smitha T Murthy <smitha.t@samsung.com>
>>> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
>>> ---
>>>  Documentation/media/uapi/v4l/pixfmt-013.rst | 5 +++++
>>>  1 file changed, 5 insertions(+)
>>>
>>> diff --git a/Documentation/media/uapi/v4l/pixfmt-013.rst b/Documentation/media/uapi/v4l/pixfmt-013.rst
>>> index 728d7ed..ff4cac2 100644
>>> --- a/Documentation/media/uapi/v4l/pixfmt-013.rst
>>> +++ b/Documentation/media/uapi/v4l/pixfmt-013.rst
>>> @@ -90,3 +90,8 @@ Compressed Formats
>>>        - ``V4L2_PIX_FMT_VP9``
>>>        - 'VP90'
>>>        - VP9 video elementary stream.
>>> +    * .. _V4L2-PIX-FMT-HEVC:
>>> +
>>> +      - ``V4L2_PIX_FMT_HEVC``
>>> +      - 'HEVC'
>>> +      - HEVC video elementary stream.
>>>
>>
>> You should mention here that HEVC == H.265.
>>
>> Regards,
>>
>> 	Hans
>>
> Do you mean change to "HEVC/H.265 video elementary stream" ?

Yes, that should do it.

Regards,

	Hans

> 
> Thank you for the review.
> Regards,
> Smitha
>>
> 
>
Smitha T Murthy April 3, 2017, 12:12 p.m. UTC | #3
On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote:
> On 03/31/2017 11:06 AM, Smitha T Murthy wrote:
> > Add V4L2 definition for HEVC compressed format
> > 
> > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com>
> > Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
> > ---
> >  Documentation/media/uapi/v4l/pixfmt-013.rst | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/Documentation/media/uapi/v4l/pixfmt-013.rst b/Documentation/media/uapi/v4l/pixfmt-013.rst
> > index 728d7ed..ff4cac2 100644
> > --- a/Documentation/media/uapi/v4l/pixfmt-013.rst
> > +++ b/Documentation/media/uapi/v4l/pixfmt-013.rst
> > @@ -90,3 +90,8 @@ Compressed Formats
> >        - ``V4L2_PIX_FMT_VP9``
> >        - 'VP90'
> >        - VP9 video elementary stream.
> > +    * .. _V4L2-PIX-FMT-HEVC:
> > +
> > +      - ``V4L2_PIX_FMT_HEVC``
> > +      - 'HEVC'
> > +      - HEVC video elementary stream.
> > 
> 
> You should mention here that HEVC == H.265.
> 
> Regards,
> 
> 	Hans
> 
Do you mean change to "HEVC/H.265 video elementary stream" ?

Thank you for the review.
Regards,
Smitha
>
Smitha T Murthy April 3, 2017, 12:20 p.m. UTC | #4
On Mon, 2017-04-03 at 14:12 +0200, Hans Verkuil wrote:
> On 04/03/2017 02:12 PM, Smitha T Murthy wrote:
> > On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote:
> >> On 03/31/2017 11:06 AM, Smitha T Murthy wrote:
> >>> Add V4L2 definition for HEVC compressed format
> >>>
> >>> Signed-off-by: Smitha T Murthy <smitha.t@samsung.com>
> >>> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
> >>> ---
> >>>  Documentation/media/uapi/v4l/pixfmt-013.rst | 5 +++++
> >>>  1 file changed, 5 insertions(+)
> >>>
> >>> diff --git a/Documentation/media/uapi/v4l/pixfmt-013.rst b/Documentation/media/uapi/v4l/pixfmt-013.rst
> >>> index 728d7ed..ff4cac2 100644
> >>> --- a/Documentation/media/uapi/v4l/pixfmt-013.rst
> >>> +++ b/Documentation/media/uapi/v4l/pixfmt-013.rst
> >>> @@ -90,3 +90,8 @@ Compressed Formats
> >>>        - ``V4L2_PIX_FMT_VP9``
> >>>        - 'VP90'
> >>>        - VP9 video elementary stream.
> >>> +    * .. _V4L2-PIX-FMT-HEVC:
> >>> +
> >>> +      - ``V4L2_PIX_FMT_HEVC``
> >>> +      - 'HEVC'
> >>> +      - HEVC video elementary stream.
> >>>
> >>
> >> You should mention here that HEVC == H.265.
> >>
> >> Regards,
> >>
> >> 	Hans
> >>
> > Do you mean change to "HEVC/H.265 video elementary stream" ?
> 
> Yes, that should do it.
> 
> Regards,
> 
> 	Hans
> 
> > 
> > Thank you for the review.
> > Regards,
> > Smitha
> >>
> > 
> > 
> 
Ok I will change it accordingly.

Thank you,
Smitha
> 
>
diff mbox

Patch

diff --git a/Documentation/media/uapi/v4l/pixfmt-013.rst b/Documentation/media/uapi/v4l/pixfmt-013.rst
index 728d7ed..ff4cac2 100644
--- a/Documentation/media/uapi/v4l/pixfmt-013.rst
+++ b/Documentation/media/uapi/v4l/pixfmt-013.rst
@@ -90,3 +90,8 @@  Compressed Formats
       - ``V4L2_PIX_FMT_VP9``
       - 'VP90'
       - VP9 video elementary stream.
+    * .. _V4L2-PIX-FMT-HEVC:
+
+      - ``V4L2_PIX_FMT_HEVC``
+      - 'HEVC'
+      - HEVC video elementary stream.