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 0071D4C14E for ; Tue, 29 Jul 2025 23:40:48 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id AE05068B760; Wed, 30 Jul 2025 02:40:44 +0300 (EEST) Received: from relay16.mail.gandi.net (relay16.mail.gandi.net [217.70.178.236]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 95A23687DE4 for ; Wed, 30 Jul 2025 02:40:38 +0300 (EEST) Received: by mail.gandi.net (Postfix) with ESMTPSA id 9B022438D0 for ; Tue, 29 Jul 2025 23:40:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=niedermayer.cc; s=gm1; t=1753832437; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=MdGZaYCJ84Nu10h9FGUdvNus55Wo7zTIi/laL+AHHH8=; b=g0heMBFdajJJ3L2oP73C2efYBt02tx/cnFB0RQ0xKpdhKuSpAdoLalXc6mIMKvoT7TzPXH pFO1zth8dSFlyupEZSQ+gk+P6J7PJRTSOkt0FgQ8/vxcJbAOvfpQ185NpTbtSoD8im/xpS 1JKdZEyvMb7h3tEtwhVHegTnVuIti2aEJDD2/HYnl3+akfZfWUSapMKGGfYFSNE/hunPkD Ua2vyTr26ZSotczVOdp4LKTALCfmPOaCY7WGD4DKj39MaTeaLGRzPcFXLBbQKC+KUZSmMX wNzpX6620pJAszqe6AXZYFixuwwhJEwhRj5Vfc+tsnUQmI76CvyDLyTI6QQmNA== Date: Wed, 30 Jul 2025 01:40:36 +0200 From: Michael Niedermayer To: FFmpeg development discussions and patches Message-ID: <20250729234036.GO29660@pb2> References: <20250723111349.1032083-1-d.frankiewic@samsung.com> <20250728151559.GA29660@pb2> <000601dc0068$8b689d00$a239d700$@samsung.com> MIME-Version: 1.0 In-Reply-To: <000601dc0068$8b689d00$a239d700$@samsung.com> X-GND-State: clean X-GND-Score: -90 X-GND-Cause: gggruggvucftvghtrhhoucdtuddrgeeffedrtdefgdelieeflecutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfitefpfffkpdcuggftfghnshhusghstghrihgsvgenuceurghilhhouhhtmecufedtudenucesvcftvggtihhpihgvnhhtshculddquddttddmnegfrhhlucfvnfffucdluddtmdenucfjughrpeffhffvuffkfhggtggujgesghdtreertddtvdenucfhrhhomhepofhitghhrggvlhcupfhivgguvghrmhgrhigvrhcuoehmihgthhgrvghlsehnihgvuggvrhhmrgihvghrrdgttgeqnecuggftrfgrthhtvghrnhepieegkedtjeduffejhfetgeejtdegteetgfegtdfhjefgvefhteegkeejtddvhfevnecukfhppeeguddrieeirdeihedrudejieenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepihhnvghtpeeguddrieeirdeihedrudejiedphhgvlhhopehlohgtrghlhhhoshhtpdhmrghilhhfrhhomhepmhhitghhrggvlhesnhhivgguvghrmhgrhigvrhdrtggtpdhnsggprhgtphhtthhopedupdhrtghpthhtohepfhhfmhhpvghgqdguvghvvghlsehffhhmphgvghdrohhrgh Subject: Re: [FFmpeg-devel] [PATCH] avcodec/apv: align APV color format support with latest liboapv version 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: , Reply-To: FFmpeg development discussions and patches Content-Type: multipart/mixed; boundary="===============0206743212286750230==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --===============0206743212286750230== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="letbn8V9wVsPeVOI" Content-Disposition: inline --letbn8V9wVsPeVOI Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 29, 2025 at 11:09:52AM +0200, d.frankiewic@samsung.com wrote: > -----Original Message----- > From: ffmpeg-devel On Behalf Of Michael > Niedermayer > Sent: Monday, July 28, 2025 5:16 PM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] avcodec/apv: align APV color format > support with latest liboapv version >=20 > >On Wed, Jul 23, 2025 at 01:13:49PM +0200, Dariusz Frankiewicz wrote: > >> Signed-off-by: Dariusz Frankiewicz > >> --- > >> configure | 2 +- > >> libavcodec/liboapvenc.c | 24 ++++++++++++++++++++++++ > >> 2 files changed, 25 insertions(+), 1 deletion(-) > >>=20 > >> diff --git a/configure b/configure > >> index ac143f2fea..5527c20578 100755 > >> --- a/configure > >> +++ b/configure > >> @@ -6983,7 +6983,7 @@ enabled jni && { [ $target_os =3D > "android" ] && check_headers jni.h > >> enabled ladspa && require_headers "ladspa.h dlfcn.h" > >> enabled lcms2 && require_pkg_config lcms2 "lcms2 >=3D 2.1= 3" > lcms2.h cmsCreateContext > >> enabled libaom && require_pkg_config libaom "aom >=3D 2.0.= 0" > aom/aom_codec.h aom_codec_version > >> -enabled liboapv && require_pkg_config liboapv "oapv >=3D > 0.1.13.1" "oapv/oapv.h" oapve_encode > >> +enabled liboapv && require_pkg_config liboapv "oapv >=3D > 0.2.0.0" "oapv/oapv.h" oapve_encode > >> enabled libaribb24 && { check_pkg_config libaribb24 "aribb24 > > 1.0.3" "aribb24/aribb24.h" arib_instance_new || > >> { enabled gpl && require_pkg_config > libaribb24 aribb24 "aribb24/aribb24.h" arib_instance_new; } || > >> die "ERROR: libaribb24 requires=20 > >> version higher than 1.0.3 or --enable-gpl."; } diff --git=20 > >> a/libavcodec/liboapvenc.c b/libavcodec/liboapvenc.c index=20 > >> e62ef57d21..7f9fbf6929 100644 > >> --- a/libavcodec/liboapvenc.c > >> +++ b/libavcodec/liboapvenc.c > >> @@ -102,9 +102,27 @@ static inline int get_color_format(enum > AVPixelFormat pix_fmt) > >> int cf =3D OAPV_CF_UNKNOWN; > >> =20 > >> switch (pix_fmt) { > >> + case AV_PIX_FMT_GRAY10: > >> + cf =3D OAPV_CF_YCBCR400; > >> + break; > >> case AV_PIX_FMT_YUV422P10: > >> cf =3D OAPV_CF_YCBCR422; > >> break; > >> + case AV_PIX_FMT_YUV422P12: > >> + cf =3D OAPV_CF_YCBCR422; > >> + break; > >> + case AV_PIX_FMT_YUV444P10: > >> + cf =3D OAPV_CF_YCBCR444; > >> + break; > >> + case AV_PIX_FMT_YUV444P12: > >> + cf =3D OAPV_CF_YCBCR444; > >> + break; > >> + case AV_PIX_FMT_YUVA444P10: > >> + cf =3D OAPV_CF_YCBCR4444; > >> + break; > >> + case AV_PIX_FMT_YUVA444P12: > >> + cf =3D OAPV_CF_YCBCR4444; > >> + break; > >> default: > >> av_assert0(cf !=3D OAPV_CF_UNKNOWN); > >> } > > > >does it make sense to continue supporting older versions ? > >or is there no distribution that uses a version prior 0.2.0.0 ? > > > >thx >=20 > Those colorspaces are supported from 0.2.0.0 version of OpenAPV so bumping > version is required to make them work with library. > There are no official distributions that are using OpenAPV old versions so > it shouldn't be a problem. > I think that we can fully switch to 0.2.0.0 as dependency. ok thx [...] --=20 Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I have never wished to cater to the crowd; for what I know they do not approve, and what they approve I do not know. -- Epicurus --letbn8V9wVsPeVOI Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABEKAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCaIlb7QAKCRBhHseHBAsP q5QJAJ9WAwk+iRoAy1/KfF2N3RKWQR+pMgCgiIZBxtMfyZvMUfEvaMKjZ2PTWT4= =ULaQ -----END PGP SIGNATURE----- --letbn8V9wVsPeVOI-- --===============0206743212286750230== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ 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". --===============0206743212286750230==--