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 <guennadi.liakhovetski@xxxxxxxxx> > Tested-by: Guennadi Liakhovetski <guennadi.liakhovetski@xxxxxxxxx> > > 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