On Fri, May 21, 2021 at 3:02 PM Irui Wang <irui.wang@xxxxxxxxxxxx> wrote: > > From: Alexandre Courbot <acourbot@xxxxxxxxxxxx> > > vidioc_try_fmt() does clamp height and width when called on the OUTPUT > queue, so clamping them prior to calling this function is redundant. Set > the queue's parameters after calling vidioc_try_fmt() so we can use the > values it computed. > > Signed-off-by: Alexandre Courbot <acourbot@xxxxxxxxxxxx> >From the guideline[1]: Notably, the last Signed-off-by: must always be that of the developer submitting the patch You should provide your signed-off. [1]: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by