Re: [FFmpeg-devel] [PATCH v1] avcodec/evc_decoder: Fix pixel format handling in export_stream_params function

[Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of
> Andreas Rheinhardt
> Sent: wtorek, 25 marca 2025 07:46
> 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.kozin...@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");
> >
> >
> 
> Patch is still broken. Apart from that: Does the BE in
XEVD_CS_YCBCR444_10BE
> etc. mean big endian?
> 
> - Andreas

Fixed

Dawid
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://protect2.fireeye.com/v1/url?k=a97927d0-c804cd53-a978ac9f-
> 74fe48600158-e92750f94350d790&q=1&e=b1134fec-7a63-44be-a722-
> 98aa431c9a91&u=https%3A%2F%2Fffmpeg.org%2Fmailman%2Flistinfo%2Fffmp
> eg-devel
> 
> To unsubscribe, visit link above, or email ffmpeg-devel-requ...@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-requ...@ffmpeg.org with subject "unsubscribe".