Re: [PATCH 0/2] uvcvideo: Refactor code to ease metadata implementation
Hi Guennadi, On Tuesday, 12 December 2017 09:45:11 EET Guennadi Liakhovetski wrote: > Hi Laurent, > > Thanks for the patches. Please feel free to add either or both of > > Reviewed-by: Guennadi Liakhovetski > Tested-by: Guennadi Liakhovetski > > to both of the patches. Whereas in fact strictly speaking your current > tree has updated improved versions of the patches, at least of the first > of them - it now correctly handles the struct video_device::vfl_dir field, > even thoough I'd still find merging that "if" with the following "switch" > prettier ;-) So, strictly speaking you'd have to post those updated > versions, in any case my approval tags refer to versions in your tree with > commit IDs > > 53464c9f76da054ac3c291d27f348170d2a346c6 > and > b6c5f10563c4ee8437cd9131bc3d389514456519 Thank you. You're absolutely right, I've reposted the patches in a v2 with your tags included. -- Regards, Laurent Pinchart
Re: [PATCH 0/2] uvcvideo: Refactor code to ease metadata implementation
Hi Laurent, Thanks for the patches. Please feel free to add either or both of Reviewed-by: Guennadi Liakhovetski Tested-by: Guennadi Liakhovetski to both of the patches. Whereas in fact strictly speaking your current tree has updated improved versions of the patches, at least of the first of them - it now correctly handles the struct video_device::vfl_dir field, even thoough I'd still find merging that "if" with the following "switch" prettier ;-) So, strictly speaking you'd have to post those updated versions, in any case my approval tags refer to versions in your tree with commit IDs 53464c9f76da054ac3c291d27f348170d2a346c6 and b6c5f10563c4ee8437cd9131bc3d389514456519 Thanks Guennadi On Tue, 5 Dec 2017, Laurent Pinchart wrote: > Hi Guennadi, > > This small patch series refactors the uvc_video_register() function to extract > the code that you need into a new uvc_video_register_device() function. Please > let me know if it can help. > > Laurent Pinchart (2): > uvcvideo: Factor out video device registration to a function > uvcvideo: Report V4L2 device caps through the video_device structure > > drivers/media/usb/uvc/uvc_driver.c | 77 > +- > drivers/media/usb/uvc/uvc_v4l2.c | 4 -- > drivers/media/usb/uvc/uvcvideo.h | 8 > 3 files changed, 60 insertions(+), 29 deletions(-) > > -- > Regards, > > Laurent Pinchart >
[PATCH 0/2] uvcvideo: Refactor code to ease metadata implementation
Hi Guennadi, This small patch series refactors the uvc_video_register() function to extract the code that you need into a new uvc_video_register_device() function. Please let me know if it can help. Laurent Pinchart (2): uvcvideo: Factor out video device registration to a function uvcvideo: Report V4L2 device caps through the video_device structure drivers/media/usb/uvc/uvc_driver.c | 77 +- drivers/media/usb/uvc/uvc_v4l2.c | 4 -- drivers/media/usb/uvc/uvcvideo.h | 8 3 files changed, 60 insertions(+), 29 deletions(-) -- Regards, Laurent Pinchart