From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ffmpeg-devel-bounces@ffmpeg.org>
Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100])
	by master.gitmailbox.com (Postfix) with ESMTP id 8EE42422A0
	for <ffmpegdev@gitmailbox.com>; Fri, 17 Dec 2021 03:30:42 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 612F968AE9A;
	Fri, 17 Dec 2021 05:30:40 +0200 (EET)
Received: from mail.overt.org (mail.overt.org [157.230.92.47])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1C9ED68AE7F
 for <ffmpeg-devel@ffmpeg.org>; Fri, 17 Dec 2021 05:30:33 +0200 (EET)
Received: from authenticated-user (mail.overt.org [157.230.92.47])
 (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits))
 (No client certificate requested)
 by mail.overt.org (Postfix) with ESMTPSA id 3499E401B9
 for <ffmpeg-devel@ffmpeg.org>; Thu, 16 Dec 2021 21:30:31 -0600 (CST)
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=overt.org; s=mail;
 t=1639711831; bh=tSTJqIex/jmte/CUPeXdi23uza7QzFJi5u1f9+lTGfI=;
 h=Date:From:To:Subject:In-Reply-To:References:From;
 b=kWnxCCf4FB+5gnDLNzV7BhPNMiF52AlkHAMIlXuI22AUyywiJcl5QOu+jz+ZCGmJZ
 m13NXUANE4voGuXL7vqTxf1ga5W3NJuTz4IdwTHvmd5/MQew5/mD9evScqP15fHQ3I
 RCwGs51ZLVH9yML/GiIPTJFWikw4IjCezHVly2HYLx7kj3pDHg/g7Q9PFa1L9ux5VQ
 B6f9rvFNYdwYTizVCJLR0omZZr3rmQrl2eliu3T28M5kcMwSZwFKHKxxFlGzz/8n7L
 NtArxLuAD3HMbBHNXctPfGOY2onnwdWrt9Ei+VpkpLSYgyZ2Lzn3sX6J7qoNHGp46C
 AniPtC5h5AFoA==
Date: Thu, 16 Dec 2021 19:30:28 -0800
From: Philip Langdale <philipl@overt.org>
To: ffmpeg-devel@ffmpeg.org
Message-ID: <20211216193028.556e09a4@fido6>
In-Reply-To: <20211216202858.77643-1-ffmpeg@tmm1.net>
References: <20211216202858.77643-1-ffmpeg@tmm1.net>
MIME-Version: 1.0
Subject: Re: [FFmpeg-devel] [PATCH v3 1/5] avfilter/vf_yadif_cuda: simplify
 filter definition
X-BeenThere: ffmpeg-devel@ffmpeg.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org>
List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe>
List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel>
List-Post: <mailto:ffmpeg-devel@ffmpeg.org>
List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help>
List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe>
Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: ffmpeg-devel-bounces@ffmpeg.org
Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org>
Archived-At: <https://master.gitmailbox.com/ffmpegdev/20211216193028.556e09a4@fido6/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>

On Thu, 16 Dec 2021 12:28:54 -0800
Aman Karmani <ffmpeg@tmm1.net> wrote:

> From: Aman Karmani <aman@tmm1.net>
> 
> Signed-off-by: Aman Karmani <aman@tmm1.net>
> ---
>  libavfilter/vf_yadif_cuda.c | 19 +------------------
>  1 file changed, 1 insertion(+), 18 deletions(-)
> 
> diff --git a/libavfilter/vf_yadif_cuda.c b/libavfilter/vf_yadif_cuda.c
> index da1ab5a8ff..685b8a2035 100644
> --- a/libavfilter/vf_yadif_cuda.c
> +++ b/libavfilter/vf_yadif_cuda.c
> @@ -212,23 +212,6 @@ static av_cold void
> deint_cuda_uninit(AVFilterContext *ctx) s->input_frames = NULL;
>  }
>  
> -static int deint_cuda_query_formats(AVFilterContext *ctx)
> -{
> -    enum AVPixelFormat pix_fmts[] = {
> -        AV_PIX_FMT_CUDA, AV_PIX_FMT_NONE,
> -    };
> -    int ret;
> -
> -    if ((ret = ff_formats_ref(ff_make_format_list(pix_fmts),
> -                              &ctx->inputs[0]->outcfg.formats)) < 0)
> -        return ret;
> -    if ((ret = ff_formats_ref(ff_make_format_list(pix_fmts),
> -                              &ctx->outputs[0]->incfg.formats)) < 0)
> -        return ret;
> -
> -    return 0;
> -}
> -
>  static int config_input(AVFilterLink *inlink)
>  {
>      AVFilterContext *ctx = inlink->dst;
> @@ -380,9 +363,9 @@ const AVFilter ff_vf_yadif_cuda = {
>      .priv_size      = sizeof(DeintCUDAContext),
>      .priv_class     = &yadif_cuda_class,
>      .uninit         = deint_cuda_uninit,
> +    FILTER_SINGLE_PIXFMT(AV_PIX_FMT_CUDA),
>      FILTER_INPUTS(deint_cuda_inputs),
>      FILTER_OUTPUTS(deint_cuda_outputs),
> -    FILTER_QUERY_FUNC(deint_cuda_query_formats),
>      .flags          = AVFILTER_FLAG_SUPPORT_TIMELINE_INTERNAL,
>      .flags_internal = FF_FILTER_FLAG_HWFRAME_AWARE,
>  };

LGTM. Thanks.

--phil
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".