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

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

 



Hi Bhupesh,

On Thursday 17 January 2013 16:23:48 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.

Do you think we could integrate those uvc-gadget changes into the public code 
base ?

> Changes since V2:
> - Addresses the review comments received on V1 of this patchset from Laurent
> and Alan
> 
> - The changes suggested by Laurent and Alan to remove WARN_ON messages
>   from the UDC controller drivers in case the gadget tries to dequeue a USB
>   request which was never queued to the UDC controller, will be floated
>   as a seperate patch.
> 
> Bhupesh Sharma (5):
>   usb: gadget/uvc: Fix VS_INPUT_HEADER.bEndpointAddress and Video
>     Streaming.bEndpointAddress values
>   usb: gadget/uvc: Make video streaming buffer size compliant to USB3.0
>     SS specs
>   usb: gadget/uvc: Port UVC webcam gadget to use videobuf2 framework
>   usb: gadget/uvc: Add support for 'USB_GADGET_DELAYED_STATUS' response
>     for a set_intf(alt-set 1) command
>   usb: gadget/uvc: Add support for 'get_unmapped_area' for MMUless
>     architectures
> 
>  drivers/usb/gadget/Kconfig     |    1 +
>  drivers/usb/gadget/f_uvc.c     |   45 +++--
>  drivers/usb/gadget/uvc.h       |    1 +
>  drivers/usb/gadget/uvc_queue.c |  543 ++++++++++++-------------------------
>  drivers/usb/gadget/uvc_queue.h |   30 ++--
>  drivers/usb/gadget/uvc_v4l2.c  |   56 +++--
>  drivers/usb/gadget/uvc_video.c |   27 ++-
>  7 files changed, 271 insertions(+), 432 deletions(-)
-- 
Regards,

Laurent Pinchart

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