Re: [PATCH 07/10] media: mtk-vcodec: venc: remove redundant code

2020-06-22 Thread Tiffany Lin
On Mon, 2020-06-22 at 22:10 +0900, Alexandre Courbot wrote: > On Fri, Jun 19, 2020 at 3:59 PM Tiffany Lin wrote: > > > > On Wed, 2020-05-20 at 17:27 +0900, Alexandre Courbot wrote: > > > vidioc_try_fmt() does clamp height and width when called on the OUTPUT > > > queue, so clamping them prior to

Re: [PATCH 07/10] media: mtk-vcodec: venc: remove redundant code

2020-06-22 Thread Alexandre Courbot
On Fri, Jun 19, 2020 at 3:59 PM Tiffany Lin wrote: > > On Wed, 2020-05-20 at 17:27 +0900, Alexandre Courbot wrote: > > 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

Re: [PATCH 07/10] media: mtk-vcodec: venc: remove redundant code

2020-06-19 Thread Tiffany Lin
On Wed, 2020-05-20 at 17:27 +0900, Alexandre Courbot wrote: > 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

[PATCH 07/10] media: mtk-vcodec: venc: remove redundant code

2020-05-20 Thread Alexandre Courbot
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 ---