On 07/21/2014 10:39 PM, Laurent Pinchart wrote:
> The RGB666 pixel format doesn't include an alpha channel. Document it as
> such.
> 
> Signed-off-by: Laurent Pinchart <[email protected]>
> ---
>  .../DocBook/media/v4l/pixfmt-packed-rgb.xml          | 20 
> ++++++--------------
>  1 file changed, 6 insertions(+), 14 deletions(-)
> 
> diff --git a/Documentation/DocBook/media/v4l/pixfmt-packed-rgb.xml 
> b/Documentation/DocBook/media/v4l/pixfmt-packed-rgb.xml
> index 32feac9..c47692a 100644
> --- a/Documentation/DocBook/media/v4l/pixfmt-packed-rgb.xml
> +++ b/Documentation/DocBook/media/v4l/pixfmt-packed-rgb.xml
> @@ -330,20 +330,12 @@ colorspace 
> <constant>V4L2_COLORSPACE_SRGB</constant>.</para>
>           <entry></entry>
>           <entry>r<subscript>1</subscript></entry>
>           <entry>r<subscript>0</subscript></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> -         <entry></entry>
> +         <entry>-</entry>
> +         <entry>-</entry>
> +         <entry>-</entry>
> +         <entry>-</entry>
> +         <entry>-</entry>
> +         <entry>-</entry>

Just to clarify: BGR666 is a three byte format, not a four byte format?

If so, then:

Acked-by: Hans Verkuil <[email protected]>

Regards,

        Hans

>         </row>
>         <row id="V4L2-PIX-FMT-BGR24">
>           <entry><constant>V4L2_PIX_FMT_BGR24</constant></entry>
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to