>-----Original Message-----
>From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of
>Andrew Sayers
>Sent: Wednesday, May 22, 2024 12:32 AM
>To: FFmpeg development discussions and patches <ffmpeg-
>de...@ffmpeg.org>
>Subject: Re: [FFmpeg-devel] [PATCH v9 11/13] avutil/hwcontext_d3d12va: add
>Flags for resource creation
>
>(Only reviewing documentation, not code)
>
>On Mon, May 20, 2024 at 10:52:20PM +0800, tong1.wu-at-
>intel....@ffmpeg.org wrote:
>> From: Tong Wu <tong1...@intel.com>
>>
>> Flags field is added to support diffferent resource creation.
>>
>> Signed-off-by: Tong Wu <tong1...@intel.com>
>> ---
>> doc/APIchanges | 3 +++
>> libavutil/hwcontext_d3d12va.c | 2 +-
>> libavutil/hwcontext_d3d12va.h | 8 ++++++++
>> libavutil/version.h | 2 +-
>> 4 files changed, 13 insertions(+), 2 deletions(-)
>>
>> diff --git a/doc/APIchanges b/doc/APIchanges
>> index 269fd36559..808ba02f2d 100644
>> --- a/doc/APIchanges
>> +++ b/doc/APIchanges
>> @@ -2,6 +2,9 @@ The last version increases of all libraries were on 2024-03-
>07
>>
>> API changes, most recent first:
>>
>> +2024-01-xx - xxxxxxxxxx - lavu 59.20.100 - hwcontext_d3d12va.h
>> + Add AVD3D12VAFramesContext.flags
>> +
>> 2024-05-xx - xxxxxxxxxx - lavu 59.19.100 - hwcontext_qsv.h
>> Add AVQSVFramesContext.info
>>
>> diff --git a/libavutil/hwcontext_d3d12va.c b/libavutil/hwcontext_d3d12va.c
>> index cfc016315d..6507cf69c1 100644
>> --- a/libavutil/hwcontext_d3d12va.c
>> +++ b/libavutil/hwcontext_d3d12va.c
>> @@ -247,7 +247,7 @@ static AVBufferRef *d3d12va_pool_alloc(void
>*opaque, size_t size)
>> .Format = hwctx->format,
>> .SampleDesc = {.Count = 1, .Quality = 0 },
>> .Layout = D3D12_TEXTURE_LAYOUT_UNKNOWN,
>> - .Flags = D3D12_RESOURCE_FLAG_NONE,
>> + .Flags = hwctx->flags,
>> };
>>
>> frame = av_mallocz(sizeof(AVD3D12VAFrame));
>> diff --git a/libavutil/hwcontext_d3d12va.h b/libavutil/hwcontext_d3d12va.h
>> index ff06e6f2ef..608dbac97f 100644
>> --- a/libavutil/hwcontext_d3d12va.h
>> +++ b/libavutil/hwcontext_d3d12va.h
>> @@ -129,6 +129,14 @@ typedef struct AVD3D12VAFramesContext {
>> * If unset, will be automatically set.
>> */
>> DXGI_FORMAT format;
>> +
>> + /**
>> + * This field is used to specify options for working with resources.
>> + * If unset, this will be D3D12_RESOURCE_FLAG_NONE.
>> + *
>> + * @see: https://learn.microsoft.com/en-
>us/windows/win32/api/d3d12/ne-d3d12-d3d12_resource_flags.
>> + */
>> + D3D12_RESOURCE_FLAGS flags;
>
>Some nitpicks:
>
>* "This field is used to specify" is redundant, you can save the reader
> a few seconds by starting the sentence with just "Options..."
>* "@see" starts a paragraph, so the rendered documentation will look better
> without the ":"
>* the full stop after the URL makes it harder to copy/paste the text -
> remove the full stop or use a [markdown link](...)
Sounds good. I've updated it with a new version.
-Tong
_______________________________________________
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".
