Hello, On Thursday 03 December 2015 16:18:42 Hans Verkuil wrote: > 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@xxxxxxxxx> > > 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@xxxxxxxxx> > > Signed-off-by: Aviv Greenberg <aviv.d.greenberg@xxxxxxxxx> > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx> > > > > 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=120c41d3477a23c6 > > 941059401db63677736f1935 > <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. So am I. And also because I haven't been CC'ed on the patch. > 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 :-) As the new formats are related to 3D I expect discussions to follow, which might result in a different implementation. I thus don't want to commit to the ABI this patch introduces. -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html