RE: [PATCH V3 0/5] UVC webcam gadget related changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent,

> -----Original Message-----
> From: Laurent Pinchart [mailto:laurent.pinchart@xxxxxxxxxxxxxxxx]
> Sent: Monday, January 21, 2013 6:16 PM
> To: balbi@xxxxxx
> Cc: Bhupesh SHARMA; linux-usb@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH V3 0/5] UVC webcam gadget related changes
> 
> Hi Felipe,
> 
> On Friday 18 January 2013 20:46:59 Felipe Balbi wrote:
> > On Thu, Jan 17, 2013 at 04:23:48PM +0530, Bhupesh Sharma wrote:
> > > This patchset tries to enhance the UVC webcam gadget driver and is
> > > based on Laurent's git tree available here (head uvc-gadget):
> > > git://linuxtv.org/pinchartl/uvcvideo.git
> > >
> > > Note that to ease review and integration of these patches, I have
> > > rebased them on Laurent's repo and all the relevant patches after
> > > review can be pushed in Felipe's repo in one go.
> > >
> > > The patches 3/5 and 4/5 in this patchset try to handle all the
> > > review comments received on the following UVC gadget related patches:
> > >
> > > [PATCH V2 1/2] usb: gadget/uvc: Port UVC webcam gadget to use
> > > videobuf2 framework [PATCH V2 2/2] usb: gadget/uvc: Add support for
> > > 'USB_GADGET_DELAYED_STATUS' response for a set_intf(alt-set 1)
> > > command
> > >
> > > which can be viewed here:
> > > [1] http://www.spinics.net/lists/linux-usb/msg68297.html
> > > [2] http://www.spinics.net/lists/linux-usb/msg68298.html
> > >
> > > I have tested this patchset on a super-speed compliant USB device
> > > controller (DWC3), with the VIVI capture device acting as a dummy
> > > source of video data and I also have modified the 'uvc-gadget'
> > > application written by Laurent (original application available here:
> > > http://git.ideasonboard.org/uvc-gadget.git) for testing the complete
> > > flow from V4L2 to UVC domain and vice versa.
> >
> > Laurent, do you wanna queue this yourself or should I take it ?
> 
> I was away last week, please give me a couple of days to review the patches.
> I'll take them in my tree with the other UVC gadget patches I have and I'll
> send you a pull request.
> 

Did you get any time to review this patchset?

I would like to send other patches which eliminate the need for doing any copy from CPU to copy the video frame into the USB request's buffer,
in-case the underlying UDC controller supports Scatter-Gather DMA. I plan to queue them up after you merge all the UVC related patches to
your repo and send a pull request to Felipe, so that I can rebase the new patches on top of the merged repo.

Regards,
Bhupesh

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux