Hi Jerry, On Wed, May 13, 2020 at 11:45:37PM +0200, Tomasz Figa wrote: > Hi Jerry, > > On Fri, May 8, 2020 at 4:03 AM Jerry-ch Chen <Jerry-ch.Chen@xxxxxxxxxxxx> wrote: > > > > Hi Laurent, Tomasz, Matthias, > > > > gentle ping for this patch set, > > If no new comments, I would like to send a newer version. > > > > Sorry, I still haven't had a chance to look at the series, so feel > free to send a new version and I will take a look at the new one. > Finally found some time to review the series. Again sorry for the delay and thanks for your patience. Some general comments: 1) The metadata format FourCC should be added in a separate patch, together with documentation for it. 2) Control IDs, structs used by the userspace, etc. should be defined in a header under include/uapi/linux. Please also check my replies to particular patches for further comments. Best regards, Tomasz
