From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id C177C4BBBA for ; Sun, 24 Aug 2025 15:42:01 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 4100C68015A; Sun, 24 Aug 2025 18:41:57 +0300 (EEST) Received: from mail.overt.org (mail.overt.org [72.14.183.176]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 1928C68015A for ; Sun, 24 Aug 2025 18:41:50 +0300 (EEST) Received: from authenticated-user (mail.overt.org [72.14.183.176]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mail.overt.org (Postfix) with ESMTPSA id 509F960680; Sun, 24 Aug 2025 10:41:48 -0500 (CDT) Date: Sun, 24 Aug 2025 08:41:43 -0700 To: Diego de Souza via ffmpeg-devel Message-ID: <20250824084143.3e56f088@fido7> In-Reply-To: References: <20250707094536.22157-1-ddesouza@nvidia.com> MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH v2 3/3] avfilter/scale_cuda: Add support for 4:2:2 chroma subsampling X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Philip Langdale via ffmpeg-devel Reply-To: FFmpeg development discussions and patches Cc: Philip Langdale , Diego de Souza , Andreas Rheinhardt Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Tue, 8 Jul 2025 11:19:45 +0000 Diego de Souza via ffmpeg-devel wrote: > Hi Andreas, > > Do you mean something like this? > > struct format_entry { > enum AVPixelFormat format; > char name[16]; > }; > > static const struct format_entry supported_formats[] = { > {AV_PIX_FMT_YUV420P, "planar8"}, > {AV_PIX_FMT_YUV422P, "planar8"}, > {AV_PIX_FMT_YUV444P, "planar8"}, > {AV_PIX_FMT_YUV420P10,"planar10"}, > {AV_PIX_FMT_YUV422P10,"planar10"}, > {AV_PIX_FMT_YUV444P10,"planar10"}, > {AV_PIX_FMT_YUV444P16,"planar16"}, > {AV_PIX_FMT_NV12, "semiplanar8"}, > {AV_PIX_FMT_NV16, "semiplanar8"}, > {AV_PIX_FMT_P010, "semiplanar10"}, > {AV_PIX_FMT_P210, "semiplanar10"}, > {AV_PIX_FMT_P016, "semiplanar16"}, > {AV_PIX_FMT_P216, "semiplanar16"}, > {AV_PIX_FMT_0RGB32, "bgr0"}, > {AV_PIX_FMT_0BGR32, "rgb0"}, > {AV_PIX_FMT_RGB32, "bgra"}, > {AV_PIX_FMT_BGR32, "rgba"}, > }; > > Best regards, > > Diego de Souza > Sr. Video Coding DevTech > NVIDIA > On 07.07.25, 12:24, "ffmpeg-devel" > wrote: > > External email: Use caution opening links or attachments > > > Diego Felix de Souza via ffmpeg-devel: > > +struct format_entry { > > + enum AVPixelFormat format; > > + const char *name; > > +}; > > + > > +static const struct format_entry supported_formats[] = { > > + {AV_PIX_FMT_YUV420P, "planar8"}, > > + {AV_PIX_FMT_YUV422P, "planar8"}, > > + {AV_PIX_FMT_YUV444P, "planar8"}, > > + {AV_PIX_FMT_YUV420P10,"planar10"}, > > + {AV_PIX_FMT_YUV422P10,"planar10"}, > > + {AV_PIX_FMT_YUV444P10,"planar10"}, > > + {AV_PIX_FMT_YUV444P16,"planar16"}, > > + {AV_PIX_FMT_NV12, "semiplanar8"}, > > + {AV_PIX_FMT_NV16, "semiplanar8"}, > > + {AV_PIX_FMT_P010, "semiplanar10"}, > > + {AV_PIX_FMT_P210, "semiplanar10"}, > > + {AV_PIX_FMT_P016, "semiplanar16"}, > > + {AV_PIX_FMT_P216, "semiplanar16"}, > > + {AV_PIX_FMT_0RGB32, "bgr0"}, > > + {AV_PIX_FMT_0BGR32, "rgb0"}, > > + {AV_PIX_FMT_RGB32, "bgra"}, > > + {AV_PIX_FMT_BGR32, "rgba"}, > > }; > > Please avoid reloactions here. Hi Andreas, I'm trying to go through these patches, and I don't understand what you were asking for Diego to change here. The original format list is replaced, but that's because it's turned into a list of structs from a list of constants. Or were you referring to something else. I also can't see what is different about Diego's proposed alternative. I want to make sure whatever change you're asking for is eventually included. 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".