Re: [git:media_tree/master] [media] UVC: Add support for ds4 depth camera

[Hans Verkuil <hverkuil@xs4all.nl>]

Hi Mauro, Aviv,

On 12/03/15 14:37, Mauro Carvalho Chehab wrote:
> This is an automatic generated email to let you know that the following patch were queued at the 
> http://git.linuxtv.org/cgit.cgi/media_tree.git tree:
> 
> Subject: [media] UVC: Add support for ds4 depth camera
> Author:  Aviv Greenberg <avivgr@gmail.com>
> Date:    Fri Oct 16 08:48:51 2015 -0300
> 
> Add support for Intel DS4 depth camera in uvc driver.
> This includes adding new uvc GUIDs for the new pixel formats,
> adding new V4L pixel format definition to user api headers,
> and updating the uvc driver GUID-to-4cc tables with the new formats.
> 
> Change-Id: If240d95a7d4edc8dcc3e02d58cd8267a6bbf6fcb
> 
> Tested-by: Greenberg, Aviv D <aviv.d.greenberg@intel.com>
> Signed-off-by: Aviv Greenberg <aviv.d.greenberg@intel.com>
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> 
>  drivers/media/usb/uvc/uvc_driver.c | 20 ++++++++++++++++++++
>  drivers/media/usb/uvc/uvcvideo.h   | 12 ++++++++++++
>  include/uapi/linux/videodev2.h     |  3 +++
>  3 files changed, 35 insertions(+)
> 
> ---
> 
> http://git.linuxtv.org/cgit.cgi/media_tree.git/commit/?id=120c41d3477a23c6941059401db63677736f1935

<snip>

> diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h
> index bd2dc9431ac1..0014529606e2 100644
> --- a/include/uapi/linux/videodev2.h
> +++ b/include/uapi/linux/videodev2.h
> @@ -621,6 +621,9 @@ struct v4l2_pix_format {
>  #define V4L2_PIX_FMT_JPGL	v4l2_fourcc('J', 'P', 'G', 'L') /* JPEG-Lite */
>  #define V4L2_PIX_FMT_SE401      v4l2_fourcc('S', '4', '0', '1') /* se401 janggu compressed rgb */
>  #define V4L2_PIX_FMT_S5C_UYVY_JPG v4l2_fourcc('S', '5', 'C', 'I') /* S5C73M3 interleaved UYVY/JPEG */
> +#define V4L2_PIX_FMT_Y8I      v4l2_fourcc('Y', '8', 'I', ' ') /* Greyscale 8-bit L/R interleaved */
> +#define V4L2_PIX_FMT_Y12I     v4l2_fourcc('Y', '1', '2', 'I') /* Greyscale 12-bit L/R interleaved */
> +#define V4L2_PIX_FMT_Z16      v4l2_fourcc('Z', '1', '6', ' ') /* Depth data 16-bit */
>  
>  /* SDR formats - used only for Software Defined Radio devices */
>  #define V4L2_SDR_FMT_CU8          v4l2_fourcc('C', 'U', '0', '8') /* IQ u8 */

I'm a bit surprised that this got accepted since there is no documentation for these new formats.
Building the DocBook should fail because of that.

Aviv, can you make a patch adding documentation for these new formats? If people don't know
what the format looks like, then it will be really hard to use :-)

Regards,

	Hans