Re: [FFmpeg-devel] [PATCH 3/3] lavu/hwcontext_qsv: Make sure hardware vendor is Intel for qsv on d3d11va

From: "Xiang, Haihao" <haihao.xiang-at-intel.com@ffmpeg.org>
To: "anton@khirnov.net" <anton@khirnov.net>,
	"ffmpeg-devel@ffmpeg.org" <ffmpeg-devel@ffmpeg.org>
Cc: "Galin, Artem" <artem.galin@intel.com>
Subject: Re: [FFmpeg-devel] [PATCH 3/3] lavu/hwcontext_qsv: Make sure hardware vendor is Intel for qsv on d3d11va
Date: Mon, 27 Nov 2023 02:33:20 +0000
Message-ID: <eaf2dffa9e1e21d939bafd62ce3f1fcc2e8857d3.camel@intel.com> (raw)
In-Reply-To: <170099589600.8914.8364179093434160453@lain.khirnov.net>

On So, 2023-11-26 at 11:51 +0100, Anton Khirnov wrote:
> Quoting Xiang, Haihao (2023-11-24 06:43:48)
> > From: Haihao Xiang <haihao.xiang@intel.com>
> > 
> > When multiple hardware are available, the default one might not be
> > Intel Hareware. We can use option vendor_id to choose the required
> > vendor.
> > 
> > Tested-by: Artem Galin <artem.galin@intel.com>
> > Signed-off-by: Haihao Xiang <haihao.xiang@intel.com>
> > ---
> >  libavutil/hwcontext_qsv.c | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c
> > index 1bfda9e69b..7a9cd0cc37 100644
> > --- a/libavutil/hwcontext_qsv.c
> > +++ b/libavutil/hwcontext_qsv.c
> > @@ -2200,6 +2200,12 @@ static int qsv_device_create(AVHWDeviceContext *ctx,
> > const char *device,
> >  #endif
> >  #if CONFIG_D3D11VA
> >      case AV_HWDEVICE_TYPE_D3D11VA:
> > +        {
> 
> Useless braces.

Yes, they are useless. Adding braces here is to keep the code style consistent
in this file. 

Thanks
Haihao

> 

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