[FFmpeg-devel] [PATCH 3/3] Map 444 pix fmts to new VdpYCbCr types defined in VDPAU.

Manoj Bonda mbonda at nvidia.com
Fri May 3 22:38:08 EEST 2019


Sorry for the delayed reply. 
Yes #if is needed in this patch too, to avoid build break incase system is having 
 vdpau.h earlier to libvdpau-1.2 
thanks for pointing it out.

Thanks,
ManojGupta.
> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of James
> Almer
> Sent: Friday, April 26, 2019 7:13 PM
> To: ffmpeg-devel at ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH 3/3] Map 444 pix fmts to new VdpYCbCr
> types defined in VDPAU.
> 
> On 4/26/2019 1:13 AM, ManojGuptaBonda wrote:
> > New VdpYCbCr Formats VDP_YCBCR_FORMAT_Y_U_V_444 and,
> > VDP_YCBCR_FORMAT_Y_UV_444 have been added in VDPAU to be used in
> > get/putbits for YUV 4:4:4 surfaces. Earlier mapping of
> > AV_PIX_FMT_YUV444P to VDP_YCBCR_FORMAT_YV12 is not valid.
> >
> > Hence this Change maps AV_PIX_FMT_YUV444P to
> > VDP_YCBCR_FORMAT_Y_U_V_444 to access the YUV 4:4:4 surface via read-
> back API's of VDPAU.
> > ---
> >  libavutil/hwcontext_vdpau.c | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/libavutil/hwcontext_vdpau.c b/libavutil/hwcontext_vdpau.c
> > index c11c3cfdab..786a51f802 100644
> > --- a/libavutil/hwcontext_vdpau.c
> > +++ b/libavutil/hwcontext_vdpau.c
> > @@ -73,8 +73,8 @@ static const VDPAUPixFmtMap pix_fmts_422[] = {  };
> >
> >  static const VDPAUPixFmtMap pix_fmts_444[] = {
> > -    { VDP_YCBCR_FORMAT_YV12, AV_PIX_FMT_YUV444P },
> > -    { 0,                     AV_PIX_FMT_NONE,   },
> > +    { VDP_YCBCR_FORMAT_Y_U_V_444, AV_PIX_FMT_YUV444P },
> 
> No need for an #ifdef here and below, like in the previous patch?
> 
> > +    { 0,                          AV_PIX_FMT_NONE,   },
> >  };
> >
> >  static const struct {
> > @@ -349,7 +349,7 @@ static int
> vdpau_transfer_data_from(AVHWFramesContext *ctx, AVFrame *dst,
> >          return AVERROR(EINVAL);
> >      }
> >
> > -    if (vdpau_format == VDP_YCBCR_FORMAT_YV12)
> > +    if ((vdpau_format == VDP_YCBCR_FORMAT_YV12) || (vdpau_format ==
> > + VDP_YCBCR_FORMAT_Y_U_V_444))
> >          FFSWAP(void*, data[1], data[2]);
> >
> >      err = priv->get_data(surf, vdpau_format, data, linesize); @@
> > -400,7 +400,7 @@ static int vdpau_transfer_data_to(AVHWFramesContext
> *ctx, AVFrame *dst,
> >          return AVERROR(EINVAL);
> >      }
> >
> > -    if (vdpau_format == VDP_YCBCR_FORMAT_YV12)
> > +    if ((vdpau_format == VDP_YCBCR_FORMAT_YV12) || (vdpau_format ==
> > + VDP_YCBCR_FORMAT_Y_U_V_444))
> >          FFSWAP(const void*, data[1], data[2]);
> >
> >      err = priv->put_data(surf, vdpau_format, data, linesize);
> >
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email ffmpeg-devel-request at ffmpeg.org
> with subject "unsubscribe".

-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------


More information about the ffmpeg-devel mailing list