diff mbox series

[FFmpeg-devel,v3] MAINTAINERS: add myself as amfenc* maintainer

Message ID 20221104185756.2089-1-ovchinnikov.dmitrii@gmail.com
State Accepted
Commit 9b13078c6aa404bb8405651ee2d01d08f9b0925c
Headers show
Series [FFmpeg-devel,v3] MAINTAINERS: add myself as amfenc* maintainer | expand

Checks

Context Check Description
yinshiyou/make_fate_loongarch64 fail Make fate failed
yinshiyou/make_loongarch64 warning New warnings during build
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Dmitrii Ovchinnikov Nov. 4, 2022, 6:57 p.m. UTC 
Due to the lack of an active AMF maintainer at the moment, as well
as plans to add the av1 encoder and other improvements of AMF,
I added myself to the maintainers. Timely review and merging
patches targeting AMF integration should improve support
of AMD GPUs and APUs in FFmpeg.
For the last couple of years I have been working on AMF related
patches to ffmpeg and other open source projects.
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

Comments

Timo Rothenpieler Nov. 5, 2022, 12:44 a.m. UTC | #1 
On 04.11.2022 19:57, OvchinnikovDmitrii wrote:
> Due to the lack of an active AMF maintainer at the moment, as well
> as plans to add the av1 encoder and other improvements of AMF,
> I added myself to the maintainers. Timely review and merging
> patches targeting AMF integration should improve support
> of AMD GPUs and APUs in FFmpeg.
> For the last couple of years I have been working on AMF related
> patches to ffmpeg and other open source projects.

LGTM from me

One note: The usual format FFmpeg follows for the commit Author is 
"Firstname Lastname <mail@domain>".
You're close enough, so it hardly matters. But something to keep in mind 
for future patches.
Soft Works Nov. 5, 2022, 5:10 a.m. UTC | #2 
> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> OvchinnikovDmitrii
> Sent: Friday, November 4, 2022 7:58 PM
> To: ffmpeg-devel@ffmpeg.org
> Cc: OvchinnikovDmitrii <ovchinnikov.dmitrii@gmail.com>
> Subject: [FFmpeg-devel] [PATCH, v3] MAINTAINERS: add myself as
> amfenc* maintainer
> 
> Due to the lack of an active AMF maintainer at the moment, as well
> as plans to add the av1 encoder and other improvements of AMF,
> I added myself to the maintainers. Timely review and merging
> patches targeting AMF integration should improve support
> of AMD GPUs and APUs in FFmpeg.
> For the last couple of years I have been working on AMF related
> patches to ffmpeg and other open source projects.

Considering the lack of ambition in the past, I think there's
no point in waiting for AMD to come around, and having a maintainer
would surely be beneficial, hence
LGTM.

softworkz
Timo Rothenpieler Nov. 10, 2022, 3:29 p.m. UTC | #3 
applied
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index eebfa5cfb7..2bc6613a83 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -136,6 +136,7 @@  Codecs:
   adpcm.c                               Zane van Iperen
   alacenc.c                             Jaikrishnan Menon
   alsdec.c                              Thilo Borgmann, Umair Khan
+  amfenc*                               Dmitrii Ovchinnikov
   aptx.c                                Aurelien Jacobs
   ass*                                  Aurelien Jacobs
   asv*                                  Michael Niedermayer