Hi Philipp,

Thank you for the patch.

On Wednesday 03 December 2014 11:28:20 Philipp Zabel wrote:
> Commit 9e74d2926a28 ("staging: imx-drm: add LVDS666 support for parallel
> display") describes a 24-bit bus format where three 6-bit components each
> take the lower part of 8 bits with the two high bits zero padded. Add a
> component-wise padded media bus format RGB666_1X24_CPADHI to support this
> connection.
> 
> Cc: Emil Renner Berthing <[email protected]>
> Signed-off-by: Philipp Zabel <[email protected]>
> ---
>  Documentation/DocBook/media/v4l/subdev-formats.xml | 30 +++++++++++++++++++
>  include/uapi/linux/media-bus-format.h              |  1 +
>  2 files changed, 31 insertions(+)
> 
> diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml
> b/Documentation/DocBook/media/v4l/subdev-formats.xml index 9afb846..c259b9e
> 100644
> --- a/Documentation/DocBook/media/v4l/subdev-formats.xml
> +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
> @@ -469,6 +469,36 @@
>             <entry>b<subscript>1</subscript></entry>
>             <entry>b<subscript>0</subscript></entry>
>           </row>
> +         <row id="MEDIA-BUS-FMT-RGB666-1X24_CPADHI">
> +           <entry>MEDIA_BUS_FMT_RGB666_1X24_CPADHI</entry>
> +           <entry>0x1015</entry>
> +           <entry></entry>
> +           &dash-ent-8;
> +           <entry>0</entry>
> +           <entry>0</entry>
> +           <entry>r<subscript>5</subscript></entry>
> +           <entry>r<subscript>4</subscript></entry>
> +           <entry>r<subscript>3</subscript></entry>
> +           <entry>r<subscript>2</subscript></entry>
> +           <entry>r<subscript>1</subscript></entry>
> +           <entry>r<subscript>0</subscript></entry>
> +           <entry>0</entry>
> +           <entry>0</entry>
> +           <entry>g<subscript>5</subscript></entry>
> +           <entry>g<subscript>4</subscript></entry>
> +           <entry>g<subscript>3</subscript></entry>
> +           <entry>g<subscript>2</subscript></entry>
> +           <entry>g<subscript>1</subscript></entry>
> +           <entry>g<subscript>0</subscript></entry>
> +           <entry>0</entry>
> +           <entry>0</entry>
> +           <entry>b<subscript>5</subscript></entry>
> +           <entry>b<subscript>4</subscript></entry>
> +           <entry>b<subscript>3</subscript></entry>
> +           <entry>b<subscript>2</subscript></entry>
> +           <entry>b<subscript>1</subscript></entry>
> +           <entry>b<subscript>0</subscript></entry>
> +         </row>
>           <row id="MEDIA-BUS-FMT-RGB888-1X24">
>             <entry>MEDIA_BUS_FMT_RGB888_1X24</entry>
>             <entry>0x100a</entry>
> diff --git a/include/uapi/linux/media-bus-format.h
> b/include/uapi/linux/media-bus-format.h index 977316e..ec80fb8 100644
> --- a/include/uapi/linux/media-bus-format.h
> +++ b/include/uapi/linux/media-bus-format.h
> @@ -45,6 +45,7 @@

Could you please update the next value comment ?

>  #define MEDIA_BUS_FMT_RGB565_2X8_BE          0x1007
>  #define MEDIA_BUS_FMT_RGB565_2X8_LE          0x1008
>  #define MEDIA_BUS_FMT_RGB666_1X18            0x1009
> +#define MEDIA_BUS_FMT_RGB666_1X24_CPADHI     0x1015
>  #define MEDIA_BUS_FMT_RGB666_LVDS_SPWG               0x1010
>  #define MEDIA_BUS_FMT_RGB888_1X24            0x100a
>  #define MEDIA_BUS_FMT_BGR888_1X24            0x1013

-- 
Regards,

Laurent Pinchart

--
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