Re: [PATCH 0/7] g_webcam: Preparation for configfs

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

 



On Wed, Aug 20, 2014 at 07:02:29PM +0200, Laurent Pinchart wrote:
> Hi Felipe,
> 
> On Wednesday 20 August 2014 10:39:43 Felipe Balbi wrote:
> > On Mon, Aug 18, 2014 at 09:45:08PM +0200, Laurent Pinchart wrote:
> > > Hi Andrzej,
> > > 
> > > Thank you for the patches. The series looks good, I only had a few
> > > comments.
> > > 
> > > I have rebased the patches on top of my UVC gadget branch, addressed my
> > > comments (the modified patches are marked as such in the commit message)
> > > and pushed the result to
> > > 
> > > 	git://linuxtv.org/pinchartl/media.git uvc/gadget
> > > 
> > > Would you be able to test the result and hopefully ack it ? If everything
> > > is fine with you there's no need to submit a new version.
> > > 
> > > On Wednesday 16 July 2014 16:54:10 Andrzej Pietrasiewicz wrote:
> > > > This patch series prepares the webcam gadget for supporting configfs.
> > > > Compared to the RFC:
> > > > 
> > > > http://www.spinics.net/lists/linux-usb/msg103301.html
> > > > 
> > > > - the patch fixing potential memory leak has been removed,
> > > > because it is already in Laurent's tree.
> > > > 
> > > > - a patch changing function names to avoid conflicts has been added,
> > > > because now the said functions are not static.
> > > > 
> > > > - Laurent's comments have been addressed.
> > > > 
> > > > This series _does not_ contain a patch adding configfs support,
> > > > as Laurent suggests - first have the preparation series merged.
> > > > 
> > > > This series depends on the gadget directory cleanup series:
> > > > 
> > > > www.spinics.net/lists/linux-usb/msg110545.html
> > > > 
> > > > Andrzej Pietrasiewicz (7):
> > > >   usb: gadget: uvc: move module parameters from f_uvc
> > > >   usb: gadget: uvc: rename functions to avoid conflicts with host uvc
> > > >   usb: gadget: uvc: separately compile some components of f_uvc
> > > >   usb: gadget: f_uvc: convert f_uvc to new function interface
> > > >   usb: gadget: webcam: convert webcam to new interface of f_uvc
> > > >   usb: gadget: f_uvc: remove compatibility layer
> > > >   usb: gadget: f_uvc: use usb_gstrings_attach
> > 
> > all patches acked-by laurent are in my testing/next already.
> 
> I've modified a couple of those. If Andrzej is fine with the modifications 
> I'll prepare a branch and send you a pull request.

alright, I'll drop them

-- 
balbi

Attachment: signature.asc
Description: Digital signature


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

  Powered by Linux