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 ESMTPS id 920634C638
	for <ffmpegdev@gitmailbox.com>; Mon,  7 Apr 2025 11:21:29 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 6DCC2687D3F;
	Mon,  7 Apr 2025 14:21:24 +0300 (EEST)
Received: from mailout2.w1.samsung.com (mailout2.w1.samsung.com
 [210.118.77.12])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id A5194687C49
 for <ffmpeg-devel@ffmpeg.org>; Mon,  7 Apr 2025 14:21:17 +0300 (EEST)
Received: from eucas1p1.samsung.com (unknown [182.198.249.206])
 by mailout2.w1.samsung.com (KnoxPortal) with ESMTP id
 20250407112115euoutp023891a891acdb77ca628160bfa4f5a6d7~0BWJJZaYC2282722827euoutp02J
 for <ffmpeg-devel@ffmpeg.org>; Mon,  7 Apr 2025 11:21:15 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.w1.samsung.com
 20250407112115euoutp023891a891acdb77ca628160bfa4f5a6d7~0BWJJZaYC2282722827euoutp02J
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com;
 s=mail20170921; t=1744024875;
 bh=OOttJYEZPJlBZQLH//A9YBYkLQGRxUjdJdA1pbVhCz8=;
 h=From:To:In-Reply-To:Subject:Date:References:From;
 b=XkffTInwd5hqvdcMZPZPxhWY0NQgSojys56znLA8wmOzvaKkdfimjy+6CHKWUE9Hx
 b9UhtKXscR56m/BOvnukZi3VVcVYZ51Czmgjd6ULwBlq5z7iIFkpZ9M2NcmQxQEjXo
 ZaOmg98SanYII3KFB9fRobbMOrS/tOTrGtrOD+j8=
Received: from eusmges1new.samsung.com (unknown [203.254.199.242]) by
 eucas1p2.samsung.com (KnoxPortal) with ESMTP id
 20250407112115eucas1p23fef93370edd67a010f7fc82c39c4935~0BWJAHgIf2160121601eucas1p2K
 for <ffmpeg-devel@ffmpeg.org>; Mon,  7 Apr 2025 11:21:15 +0000 (GMT)
Received: from eucas1p1.samsung.com ( [182.198.249.206]) by
 eusmges1new.samsung.com (EUCPMTA) with SMTP id 4C.24.20821.B25B3F76; Mon,  7
 Apr 2025 12:21:15 +0100 (BST)
Received: from eusmtrp1.samsung.com (unknown [182.198.249.138]) by
 eucas1p1.samsung.com (KnoxPortal) with ESMTPA id
 20250407112114eucas1p1ac0ab40db5193815c9ae66dfacb47f49~0BWIj2QMQ2122421224eucas1p1H
 for <ffmpeg-devel@ffmpeg.org>; Mon,  7 Apr 2025 11:21:14 +0000 (GMT)
Received: from eusmgms1.samsung.com (unknown [182.198.249.179]) by
 eusmtrp1.samsung.com (KnoxPortal) with ESMTP id
 20250407112114eusmtrp1fa29838734da4db9b2259c60a069b235~0BWIjbwON1289412894eusmtrp1L
 for <ffmpeg-devel@ffmpeg.org>; Mon,  7 Apr 2025 11:21:14 +0000 (GMT)
X-AuditID: cbfec7f2-b09c370000005155-5b-67f3b52bda9d
Received: from eusmtip2.samsung.com ( [203.254.199.222]) by
 eusmgms1.samsung.com (EUCPMTA) with SMTP id A7.21.19920.A25B3F76; Mon,  7
 Apr 2025 12:21:14 +0100 (BST)
Received: from AMDN5164 (unknown [106.210.132.171]) by eusmtip2.samsung.com
 (KnoxPortal) with ESMTPA id
 20250407112114eusmtip24cd5c582f8d37d8894ccd060d9d75a60~0BWIVXopX1980719807eusmtip2Y
 for <ffmpeg-devel@ffmpeg.org>; Mon,  7 Apr 2025 11:21:14 +0000 (GMT)
From: "Dawid Kozinski/Multimedia \(PLT\) /SRPOL/Staff Engineer/Samsung
 Electronics" <d.kozinski@samsung.com>
To: "'FFmpeg development discussions and patches'" <ffmpeg-devel@ffmpeg.org>
In-Reply-To: <DUZSPRMB0005E9D2805D5522C26A4C7B8FA72@DUZSPRMB0005.EURP250.PROD.OUTLOOK.COM>
Date: Mon, 7 Apr 2025 13:21:14 +0200
Message-ID: <000001dba7af$2c8487c0$858d9740$@samsung.com>
MIME-Version: 1.0
X-Mailer: Microsoft Outlook 16.0
Thread-Index: AQHF2oAu15rrUi+CwSmODPULzFghIQGhlNt9AYdc83qzqmBKoA==
Content-Language: pl
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrOIsWRmVeSWpSXmKPExsWy7djPc7raWz+nG8y/yGPx7dMZZgdGjz+L
 NrMEMEZx2aSk5mSWpRbp2yVwZaxpEys4Klpxa61qA+NMwS5GTg4JAROJx0t3MncxcnEICaxg
 lFiz9iYjhDOJSaKzvRnKmcgksWL5TGaYlot/LrFDJJYzSrQ/PwVV1cYk0djfxAJSxSaQJ/H4
 81qwDhEBH4nu9etZQWxOgSiJG603weLCAhUSa6avB6tnEVCRuP7uClicV8BSYubKQ2wQtqDE
 yZlPwGqYBeQltr+dA3WFgsTPp8tYIeY7SbxcvZsRokZE4sajFrCDJAR6OSTmdE+CanCRuDn1
 OiuELSzx6vgWdghbRuL05B6gBRxAdrHEoX4HCLNG4tCPdIgKa4m3jccZQcLMApoS63fpQ4Qd
 JeY9aGODqOaTuPFWEOIAPolJ26YzQ4R5JTrahCBMFYm+TjGIRimJp8vmME9gVJqF5MNZSD6c
 heSTWQhrFzCyrGIUTy0tzk1PLTbMSy3XK07MLS7NS9dLzs/dxAhMC6f/Hf+0g3Huq496hxiZ
 OBgPMUpwMCuJ8Fqe+pQuxJuSWFmVWpQfX1Sak1p8iFGag0VJnHfR/tZ0IYH0xJLU7NTUgtQi
 mCwTB6dUA5PCzkiezVas3Y/viszW47r6X35n2dR3Dht5RfMFp7UXtL70umFsPsf1wGzvtzcl
 Vdrl3IVPel/klll65alLxlX9hqK4Qo3vVbrdS/c9WOH2fTXD/OKPaxgdJBl8m415Gbq1v7zZ
 svrdhzlv+s8fuHRpP3/Br4bvffv1F7hk5i9dqnaO0eBb7us7BU2PmvvWM4V9Zjg9n0V7967n
 3ctmSOawVXOxqKbulZqjdvLEmUnHORVPJYoIqvSKLCqdE1lykVH+eYXYffmvU/a5ux6Sk32Z
 JKHH892Jz0M6R+KgMEeh61SbG0vcV9cdmr7Nhuvir/NzF/x+3nbqWqDjjrIDp6IFFRy7nLdF
 5nF3RL/SjFBiKc5INNRiLipOBAACqvL9egMAAA==
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrDLMWRmVeSWpSXmKPExsVy+t/xe7paWz+nGyxfr23x7dMZZgdGjz+L
 NrMEMEbp2RTll5akKmTkF5fYKkUbWhjpGVpa6BmZWOoZGpvHWhmZKunb2aSk5mSWpRbp2yXo
 ZaxpEys4Klpxa61qA+NMwS5GTg4JAROJi38usXcxcnEICSxllLhzfys7REJKYunSRYwQtrDE
 n2tdbBBFLUwSnXMvghWxCeRIrJ09kQnEFhHwkehev54Vouguo0TD5glgCU6BKIkbrTeZQWxh
 gTKJp/O62UBsFgEVievvroDFeQUsJWauPMQGYQtKnJz5hKWLkYODWUBPom0j2BHMAvIS29/O
 YYY4SEHi59NlrBB7nSRert4NVSMiceNRC+MERqFZSCbNQpg0C8mkWUg6FjCyrGIUSS0tzk3P
 LTbUK07MLS7NS9dLzs/dxAiMhm3Hfm7ewTjv1Ue9Q4xMHIyHGCU4mJVEeC1PfUoX4k1JrKxK
 LcqPLyrNSS0+xGgK9NlEZinR5HxgPOaVxBuaGZgamphZGphamhkrifO6XT6fJiSQnliSmp2a
 WpBaBNPHxMEp1cDEXDfvh/Y8LZ+8/FlCerUWmw5n8L51kTJ8bW/KEuosMv9A3cfiWS4vpmjv
 +br+QqNXQ6djf0W+OLP3agktmT5ZgfNiX5TVomKCv3u/4v9jWKD5Zk3kL7k5rtNDNGuubm1O
 THzTaqrWs6/X8cvk/ff2PLiema2yliluQdkEjQVeiTkhhy74Rl5k+K5yYmtsmT3n1V55r6Jl
 hew763yUbkt5Med+cfrmMmdG81LXDy+Dn2+3fl2rzf3hTGvvsb8bTpy/bpC+eYZfr17b5Kdy
 Ad8q2CK6f3BJteydKBsZc0QrNSVM+/m/mTHqFXz6qyVOd0xc7auhf/tAZMLBMHt1rbfli862
 LD6mOjH4rWJKqxJLcUaioRZzUXEiAIqHf0YPAwAA
X-CMS-MailID: 20250407112114eucas1p1ac0ab40db5193815c9ae66dfacb47f49
X-Msg-Generator: CA
X-RootMTR: 20250325075648eucas1p191d4e82df18831d08b9e7f09dbca5a20
X-EPHeader: CA
CMS-TYPE: 201P
X-CMS-RootMailID: 20250325075648eucas1p191d4e82df18831d08b9e7f09dbca5a20
References: <CGME20250325075648eucas1p191d4e82df18831d08b9e7f09dbca5a20@eucas1p1.samsung.com>
 <20250325075643.1273491-1-d.kozinski@samsung.com>
 <DUZSPRMB0005E9D2805D5522C26A4C7B8FA72@DUZSPRMB0005.EURP250.PROD.OUTLOOK.COM>
Subject: Re: [FFmpeg-devel] [PATCH v1] avcodec/evc_decoder: Fix pixel format
 handling in export_stream_params function
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/000001dba7af$2c8487c0$858d9740$@samsung.com/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>

Andreas, what about my patch? I submitted a fix some time ago
(https://patchwork.ffmpeg.org/project/ffmpeg/patch/20250325085030.1306501-1-
d.kozinski@samsung.com/_ .
 Is there still something wrong? 
I'm waiting for some feedback.

We would like to close the topic now.

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> Andreas Rheinhardt
> Sent: wtorek, 25 marca 2025 09:18
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v1] avcodec/evc_decoder: Fix pixel
format
> handling in export_stream_params function
> 
> Dawid Kozinski:
> > Signed-off-by: Dawid Kozinski <d.kozinski@samsung.com>
> > ---
> >  libavcodec/libxevd.c | 12 ++++++++----
> >  1 file changed, 8 insertions(+), 4 deletions(-)
> >
> > diff --git a/libavcodec/libxevd.c b/libavcodec/libxevd.c index
> > 520fdab7d8..483700c81e 100644
> > --- a/libavcodec/libxevd.c
> > +++ b/libavcodec/libxevd.c
> > @@ -152,16 +152,20 @@ static int export_stream_params(const XevdContext
> *xectx, AVCodecContext *avctx)
> >      }
> >      switch(color_space) {
> >      case XEVD_CS_YCBCR400_10LE:
> > -        avctx->pix_fmt = AV_PIX_FMT_GRAY10LE;
> > +    case XEVD_CS_SET(XEVD_CF_YCBCR400, 10, 1): //
> XEVD_CS_YCBCR400_10BE
> > +        avctx->pix_fmt = AV_PIX_FMT_GRAY10;
> >          break;
> >      case XEVD_CS_YCBCR420_10LE:
> > -        avctx->pix_fmt = AV_PIX_FMT_YUV420P10LE;
> > +    case XEVD_CS_SET(XEVD_CF_YCBCR420, 10, 1): //
> XEVD_CS_YCBCR420_10BE
> > +        avctx->pix_fmt = AV_PIX_FMT_YUV420P10;
> >          break;
> >      case XEVD_CS_YCBCR422_10LE:
> > -        avctx->pix_fmt = AV_PIX_FMT_YUV422P10LE;
> > +    case XEVD_CS_SET(XEVD_CF_YCBCR422, 10, 1): //
> XEVD_CS_YCBCR422_10BE
> > +        avctx->pix_fmt = AV_PIX_FMT_YUV422P10;
> >          break;
> >      case XEVD_CS_YCBCR444_10LE:
> > -        avctx->pix_fmt = AV_PIX_FMT_YUV444P10LE;
> > +    case XEVD_CS_SET(XEVD_CF_YCBCR444, 10, 1): //
> XEVD_CS_YCBCR444_10BE
> > +        avctx->pix_fmt = AV_PIX_FMT_YUV444P10;
> >          break;
> >      default:
> >          av_log(avctx, AV_LOG_ERROR, "Unknown color space\n");
> 
> This patch maps both BE and LE formats to FFmpeg's corresponding native
> endian formats. This is wrong.
> 
> - Andreas
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://protect2.fireeye.com/v1/url?k=f6892c97-970239ad-f688a7d8-
> 74fe4860008a-8439094af49407cb&q=1&e=cc888bcc-836f-4cab-acb4-
> a2034e7875c6&u=https%3A%2F%2Fffmpeg.org%2Fmailman%2Flistinfo%2Fffmp
> eg-devel
> 
> To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org
> with subject "unsubscribe".

_______________________________________________
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".