Hi Jernej, On Mon 24 Oct 22, 22:15, Jernej Skrabec wrote: > cedrus_try_fmt_vid_out() is called two times inside > cedrus_s_fmt_vid_out(), but nothing changes between calls which would > influence output format. Remove first call, which was added later. Thanks for the cleanup! Acked-by: Paul Kocialkowski <paul.kocialkowski@xxxxxxxxxxx> Cheers, Paul > Signed-off-by: Jernej Skrabec <jernej.skrabec@xxxxxxxxx> > --- > drivers/staging/media/sunxi/cedrus/cedrus_video.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/staging/media/sunxi/cedrus/cedrus_video.c b/drivers/staging/media/sunxi/cedrus/cedrus_video.c > index 33726175d980..1c3c1d080d31 100644 > --- a/drivers/staging/media/sunxi/cedrus/cedrus_video.c > +++ b/drivers/staging/media/sunxi/cedrus/cedrus_video.c > @@ -309,10 +309,6 @@ static int cedrus_s_fmt_vid_out(struct file *file, void *priv, > struct vb2_queue *peer_vq; > int ret; > > - ret = cedrus_try_fmt_vid_out(file, priv, f); > - if (ret) > - return ret; > - > vq = v4l2_m2m_get_vq(ctx->fh.m2m_ctx, f->type); > /* > * In order to support dynamic resolution change, > -- > 2.38.1 > -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com
Attachment:
signature.asc
Description: PGP signature
