On 07/19/2017 12:54 PM, Hans Verkuil wrote: > On 17/07/17 12:33, Todor Tomov wrote: >> This patchset adds basic support for the Qualcomm Camera Subsystem found >> on Qualcomm MSM8916 and APQ8016 processors. >> >> The driver implements V4L2, Media controller and V4L2 subdev interfaces. >> Camera sensor using V4L2 subdev interface in the kernel is supported. >> >> The driver is implemented using as a reference the Qualcomm Camera >> Subsystem driver for Android as found in Code Aurora [1]. >> >> The driver is tested on Dragonboard 410C (APQ8016) with one and two >> OV5645 camera sensors. media-ctl [2] and yavta [3] applications were >> used for testing. Also Gstreamer 1.10.4 with v4l2src plugin is supported. >> >> More information is present in the document added by the third patch. > > OK, so this looks pretty good. I have one comment for patch 12/23, and the > dt-bindings need to be acked. > > I suggest you make a v3.1 for patch 12/23 and then I'll wait for the binding > ack. Once that's in (and there are no other comments) I will merge this. Since Sakari had a bunch of comments I'll mark this series as 'Changes Requested' and wait for a v4. Regards, Hans
