[PATCH 1/4] MAINTAINERS: drop audio maintainership

Gerd Hoffmann posted 4 patches 6 months, 1 week ago
There is a newer version of this series
[PATCH 1/4] MAINTAINERS: drop audio maintainership
Posted by Gerd Hoffmann 6 months, 1 week ago
Remove myself from audio (both devices and backend) entries.
Flip status to "Orphan" for entries which have nobody else listed.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 MAINTAINERS | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 1b79767d6196..7f52e2912fc3 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1913,8 +1913,7 @@ F: include/hw/xtensa/mx_pic.h
 Devices
 -------
 Overall Audio frontends
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: hw/audio/
 F: include/hw/audio/
 F: tests/qtest/ac97-test.c
@@ -2388,7 +2387,6 @@ F: hw/virtio/virtio-mem-pci.c
 F: include/hw/virtio/virtio-mem.h
 
 virtio-snd
-M: Gerd Hoffmann <kraxel@redhat.com>
 R: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
 S: Supported
 F: hw/audio/virtio-snd.c
@@ -2767,7 +2765,6 @@ F: include/hw/hyperv/hv-balloon.h
 Subsystems
 ----------
 Overall Audio backends
-M: Gerd Hoffmann <kraxel@redhat.com>
 M: Marc-André Lureau <marcandre.lureau@redhat.com>
 S: Odd Fixes
 F: audio/
@@ -2783,13 +2780,11 @@ X: audio/spiceaudio.c
 F: qapi/audio.json
 
 ALSA Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 R: Christian Schoenebeck <qemu_oss@crudebyte.com>
 S: Odd Fixes
 F: audio/alsaaudio.c
 
 Core Audio framework backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 M: Philippe Mathieu-Daudé <philmd@linaro.org>
 R: Christian Schoenebeck <qemu_oss@crudebyte.com>
 R: Akihiko Odaki <akihiko.odaki@daynix.com>
@@ -2797,34 +2792,28 @@ S: Odd Fixes
 F: audio/coreaudio.c
 
 DSound Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: audio/dsound*
 
 JACK Audio Connection Kit backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 R: Christian Schoenebeck <qemu_oss@crudebyte.com>
 S: Odd Fixes
 F: audio/jackaudio.c
 
 Open Sound System (OSS) Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: audio/ossaudio.c
 
 PulseAudio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: audio/paaudio.c
 
 SDL Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 R: Thomas Huth <huth@tuxfamily.org>
 S: Odd Fixes
 F: audio/sdlaudio.c
 
 Sndio Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 R: Alexandre Ratchov <alex@caoua.org>
 S: Odd Fixes
 F: audio/sndioaudio.c
-- 
2.45.0


Re: [PATCH 1/4] MAINTAINERS: drop audio maintainership
Posted by Matias Ezequiel Vara Larsen 6 months ago
On Thu, May 16, 2024 at 02:03:38PM +0200, Gerd Hoffmann wrote:
> Remove myself from audio (both devices and backend) entries.
> Flip status to "Orphan" for entries which have nobody else listed.
> 
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  MAINTAINERS | 19 ++++---------------
>  1 file changed, 4 insertions(+), 15 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1b79767d6196..7f52e2912fc3 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1913,8 +1913,7 @@ F: include/hw/xtensa/mx_pic.h
>  Devices
>  -------
>  Overall Audio frontends
> -M: Gerd Hoffmann <kraxel@redhat.com>
> -S: Odd Fixes
> +S: Orphan
>  F: hw/audio/
>  F: include/hw/audio/
>  F: tests/qtest/ac97-test.c
> @@ -2388,7 +2387,6 @@ F: hw/virtio/virtio-mem-pci.c
>  F: include/hw/virtio/virtio-mem.h
>  
>  virtio-snd
> -M: Gerd Hoffmann <kraxel@redhat.com>
>  R: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
>  S: Supported
>  F: hw/audio/virtio-snd.c
> @@ -2767,7 +2765,6 @@ F: include/hw/hyperv/hv-balloon.h

For virtio-snd, I can join as Reviewer if that helps.

Matias
Re: [PATCH 1/4] MAINTAINERS: drop audio maintainership
Posted by Thomas Huth 6 months ago
On 16/05/2024 14.03, Gerd Hoffmann wrote:
> Remove myself from audio (both devices and backend) entries.
> Flip status to "Orphan" for entries which have nobody else listed.
> 
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>   MAINTAINERS | 19 ++++---------------
>   1 file changed, 4 insertions(+), 15 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1b79767d6196..7f52e2912fc3 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
...
> @@ -2388,7 +2387,6 @@ F: hw/virtio/virtio-mem-pci.c
>   F: include/hw/virtio/virtio-mem.h
>   
>   virtio-snd
> -M: Gerd Hoffmann <kraxel@redhat.com>
>   R: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
>   S: Supported

I think the status should be downgraded to Orphan or at least Odd-fixes, 
unless Manos wants to upgrade from "R:" to "M:" ?

>   ALSA Audio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
>   R: Christian Schoenebeck <qemu_oss@crudebyte.com>
>   S: Odd Fixes
>   F: audio/alsaaudio.c

I'd also suggest that Christian either upgrade from R: to M: or that we 
change the status to Orphan

>   JACK Audio Connection Kit backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
>   R: Christian Schoenebeck <qemu_oss@crudebyte.com>
>   S: Odd Fixes
>   F: audio/jackaudio.c

dito

>   SDL Audio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
>   R: Thomas Huth <huth@tuxfamily.org>

I'm fine if you update my entry from R: to M: here.

>   S: Odd Fixes
>   F: audio/sdlaudio.c
>   
>   Sndio Audio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
>   R: Alexandre Ratchov <alex@caoua.org>
>   S: Odd Fixes
>   F: audio/sndioaudio.c

Same again, I'd suggest to either set to Orphan or upgrade the R: entry?

  Thomas
Re: [PATCH 1/4] MAINTAINERS: drop audio maintainership
Posted by Christian Schoenebeck 6 months ago
On Wednesday, May 22, 2024 2:54:51 PM CEST Thomas Huth wrote:
> On 16/05/2024 14.03, Gerd Hoffmann wrote:
[...]
> >   ALSA Audio backend
> > -M: Gerd Hoffmann <kraxel@redhat.com>
> >   R: Christian Schoenebeck <qemu_oss@crudebyte.com>
> >   S: Odd Fixes
> >   F: audio/alsaaudio.c
> 
> I'd also suggest that Christian either upgrade from R: to M: or that we 
> change the status to Orphan
> 
> >   JACK Audio Connection Kit backend
> > -M: Gerd Hoffmann <kraxel@redhat.com>
> >   R: Christian Schoenebeck <qemu_oss@crudebyte.com>
> >   S: Odd Fixes
> >   F: audio/jackaudio.c
> 
> dito

Well, best I can offer is "I try", but no guarantee for latency or anything.

On doubt put them on orphan.

Best regards,
Christian Schoenebeck
Re: [PATCH 1/4] MAINTAINERS: drop audio maintainership
Posted by Manos Pitsidianakis 6 months ago
On Wed, 22 May 2024 at 15:54, Thomas Huth <thuth@redhat.com> wrote:
>
> On 16/05/2024 14.03, Gerd Hoffmann wrote:
> > Remove myself from audio (both devices and backend) entries.
> > Flip status to "Orphan" for entries which have nobody else listed.
> >
> > Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> > ---
> >   MAINTAINERS | 19 ++++---------------
> >   1 file changed, 4 insertions(+), 15 deletions(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 1b79767d6196..7f52e2912fc3 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> ...
> > @@ -2388,7 +2387,6 @@ F: hw/virtio/virtio-mem-pci.c
> >   F: include/hw/virtio/virtio-mem.h
> >
> >   virtio-snd
> > -M: Gerd Hoffmann <kraxel@redhat.com>
> >   R: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> >   S: Supported
>
> I think the status should be downgraded to Orphan or at least Odd-fixes,
> unless Manos wants to upgrade from "R:" to "M:" ?

That's fine with me.

> >   ALSA Audio backend
> > -M: Gerd Hoffmann <kraxel@redhat.com>
> >   R: Christian Schoenebeck <qemu_oss@crudebyte.com>
> >   S: Odd Fixes
> >   F: audio/alsaaudio.c
>
> I'd also suggest that Christian either upgrade from R: to M: or that we
> change the status to Orphan

If Christian is not available I volunteer to be a Reviewer (but not
M:) since I have some familiarity with alsaaudio.c
This way even if Orphan it will have more reviewers available.

>
> >   JACK Audio Connection Kit backend
> > -M: Gerd Hoffmann <kraxel@redhat.com>
> >   R: Christian Schoenebeck <qemu_oss@crudebyte.com>
> >   S: Odd Fixes
> >   F: audio/jackaudio.c
>
> dito
>
> >   SDL Audio backend
> > -M: Gerd Hoffmann <kraxel@redhat.com>
> >   R: Thomas Huth <huth@tuxfamily.org>
>
> I'm fine if you update my entry from R: to M: here.
>
> >   S: Odd Fixes
> >   F: audio/sdlaudio.c
> >
> >   Sndio Audio backend
> > -M: Gerd Hoffmann <kraxel@redhat.com>
> >   R: Alexandre Ratchov <alex@caoua.org>
> >   S: Odd Fixes
> >   F: audio/sndioaudio.c
>
> Same again, I'd suggest to either set to Orphan or upgrade the R: entry?
>
>   Thomas
>