[PATCH v2 0/5] Reduce load on ppc target maintainers

[PATCH v2 0/5] Reduce load on ppc target maintainers

[David Gibson]

Greg Kurz and myself have been co-maintainers for the ppc and ppc64
targets for some time now.  However, both our day job responsibilities
and interests are leading us towards other areas, so we have less time
to devote to this any more.

Therefore, here's a bunch of updates to MAINTAINERS, intended to
reduce the load on us.  Mostly this is marking fairly obscure
platforms as orphaned (if someone wants to take over maintainership,
let me know ASAP).  Bigger changes may be coming, but we haven't
decided exactly what that's going to look like yet.

Changes since v1:
 * Reworked how OpenPIC is listed

David Gibson (5):
  MAINTAINERS: Orphan obscure ppc platforms
  MAINTAINERS: Remove David & Greg as reviewers for a number of boards
  MAINTAINERS: Remove David & Greg as reviewers/co-maintainers of
    powernv
  MAINTAINERS: Add information for OpenPIC
  MAINTAINERS: Demote sPAPR from "Supported" to "Maintained"

 MAINTAINERS | 42 ++++++++++++++----------------------------
 1 file changed, 14 insertions(+), 28 deletions(-)

-- 
2.31.1

[PATCH v2 1/5] MAINTAINERS: Orphan obscure ppc platforms

[David Gibson]

There are a nunber of old embedded ppc machine types which have been little
changed and in "Odd Fixes" state for a long time.  With both myself and
Greg Kurz moving toward other areas, we no longer have the capacity to
keep reviewing and maintaining even the rare patches that come in for those
platforms.

Therefore, remove our names as reviewers and mark these platforms as
orphaned.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Greg Kurz <groug@kaod.org>
---
 MAINTAINERS | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index d7915ec128..3408618f2a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1236,24 +1236,18 @@ F: hw/openrisc/openrisc_sim.c
 PowerPC Machines
 ----------------
 405
-M: David Gibson <david@gibson.dropbear.id.au>
-M: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
-S: Odd Fixes
+S: Orphan
 F: hw/ppc/ppc405_boards.c
 
 Bamboo
-M: David Gibson <david@gibson.dropbear.id.au>
-M: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
-S: Odd Fixes
+S: Orphan
 F: hw/ppc/ppc440_bamboo.c
 
 e500
-M: David Gibson <david@gibson.dropbear.id.au>
-M: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
-S: Odd Fixes
+S: Orphan
 F: hw/ppc/e500*
 F: hw/gpio/mpc8xxx.c
 F: hw/i2c/mpc_i2c.c
@@ -1264,10 +1258,8 @@ F: include/hw/pci-host/ppce500.h
 F: pc-bios/u-boot.e500
 
 mpc8544ds
-M: David Gibson <david@gibson.dropbear.id.au>
-M: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
-S: Odd Fixes
+S: Orphan
 F: hw/ppc/mpc8544ds.c
 F: hw/ppc/mpc8544_guts.c
 
@@ -1783,9 +1775,8 @@ F: include/hw/acpi/ghes.h
 F: docs/specs/acpi_hest_ghes.rst
 
 ppc4xx
-M: David Gibson <david@gibson.dropbear.id.au>
 L: qemu-ppc@nongnu.org
-S: Odd Fixes
+S: Orphan
 F: hw/ppc/ppc4*.c
 F: hw/i2c/ppc4xx_i2c.c
 F: include/hw/ppc/ppc4xx.h
-- 
2.31.1

Re: [PATCH v2 1/5] MAINTAINERS: Orphan obscure ppc platforms

[Cédric Le Goater]

On 9/24/21 11:55, David Gibson wrote:
> There are a nunber of old embedded ppc machine types which have been little
> changed and in "Odd Fixes" state for a long time.  With both myself and
> Greg Kurz moving toward other areas, we no longer have the capacity to
> keep reviewing and maintaining even the rare patches that come in for those
> platforms.
> 
> Therefore, remove our names as reviewers and mark these platforms as
> orphaned.
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> Reviewed-by: Greg Kurz <groug@kaod.org>


Reviewed-by: Cédric Le Goater <clg@kaod.org>

> ---
>   MAINTAINERS | 19 +++++--------------
>   1 file changed, 5 insertions(+), 14 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index d7915ec128..3408618f2a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1236,24 +1236,18 @@ F: hw/openrisc/openrisc_sim.c
>   PowerPC Machines
>   ----------------
>   405
> -M: David Gibson <david@gibson.dropbear.id.au>
> -M: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
> -S: Odd Fixes
> +S: Orphan
>   F: hw/ppc/ppc405_boards.c
>   
>   Bamboo
> -M: David Gibson <david@gibson.dropbear.id.au>
> -M: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
> -S: Odd Fixes
> +S: Orphan
>   F: hw/ppc/ppc440_bamboo.c
>   
>   e500
> -M: David Gibson <david@gibson.dropbear.id.au>
> -M: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
> -S: Odd Fixes
> +S: Orphan
>   F: hw/ppc/e500*
>   F: hw/gpio/mpc8xxx.c
>   F: hw/i2c/mpc_i2c.c
> @@ -1264,10 +1258,8 @@ F: include/hw/pci-host/ppce500.h
>   F: pc-bios/u-boot.e500
>   
>   mpc8544ds
> -M: David Gibson <david@gibson.dropbear.id.au>
> -M: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
> -S: Odd Fixes
> +S: Orphan
>   F: hw/ppc/mpc8544ds.c
>   F: hw/ppc/mpc8544_guts.c
>   
> @@ -1783,9 +1775,8 @@ F: include/hw/acpi/ghes.h
>   F: docs/specs/acpi_hest_ghes.rst
>   
>   ppc4xx
> -M: David Gibson <david@gibson.dropbear.id.au>
>   L: qemu-ppc@nongnu.org
> -S: Odd Fixes
> +S: Orphan
>   F: hw/ppc/ppc4*.c
>   F: hw/i2c/ppc4xx_i2c.c
>   F: include/hw/ppc/ppc4xx.h
> 

[PATCH v2 2/5] MAINTAINERS: Remove David & Greg as reviewers for a number of boards

[David Gibson]

Greg and I are moving towards other areas and no longer have capacity to
act as regular reviewers for several of the secondary ppc machine types.
So, remove ourselves as reviewers for Macintosh, PReP, sam460ex and
pegasos2 in MAINTAINERS.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Greg Kurz <groug@kaod.org>
---
 MAINTAINERS | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 3408618f2a..20436be86f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1265,8 +1265,6 @@ F: hw/ppc/mpc8544_guts.c
 
 New World (mac99)
 M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
-R: David Gibson <david@gibson.dropbear.id.au>
-R: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
 S: Odd Fixes
 F: hw/ppc/mac_newworld.c
@@ -1285,8 +1283,6 @@ F: pc-bios/qemu_vga.ndrv
 
 Old World (g3beige)
 M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
-R: David Gibson <david@gibson.dropbear.id.au>
-R: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
 S: Odd Fixes
 F: hw/ppc/mac_oldworld.c
@@ -1300,8 +1296,6 @@ F: pc-bios/qemu_vga.ndrv
 
 PReP
 M: Hervé Poussineau <hpoussin@reactos.org>
-R: David Gibson <david@gibson.dropbear.id.au>
-R: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
 S: Maintained
 F: hw/ppc/prep.c
@@ -1356,8 +1350,6 @@ F: hw/ppc/virtex_ml507.c
 
 sam460ex
 M: BALATON Zoltan <balaton@eik.bme.hu>
-R: David Gibson <david@gibson.dropbear.id.au>
-R: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
 S: Maintained
 F: hw/ppc/sam460ex.c
@@ -1371,7 +1363,6 @@ F: roms/u-boot-sam460ex
 
 pegasos2
 M: BALATON Zoltan <balaton@eik.bme.hu>
-R: David Gibson <david@gibson.dropbear.id.au>
 L: qemu-ppc@nongnu.org
 S: Maintained
 F: hw/ppc/pegasos2.c
-- 
2.31.1

Re: [PATCH v2 2/5] MAINTAINERS: Remove David & Greg as reviewers for a number of boards

[BALATON Zoltan]

[-- Attachment #1: Type: text/plain, Size: 1889 bytes --]

On Fri, 24 Sep 2021, David Gibson wrote:
> Greg and I are moving towards other areas and no longer have capacity to
> act as regular reviewers for several of the secondary ppc machine types.
> So, remove ourselves as reviewers for Macintosh, PReP, sam460ex and
> pegasos2 in MAINTAINERS.
>
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> Reviewed-by: Greg Kurz <groug@kaod.org>

Acked-by: BALATON Zoltan <balaton@eik.bme.hu>

> ---
> MAINTAINERS | 9 ---------
> 1 file changed, 9 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3408618f2a..20436be86f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1265,8 +1265,6 @@ F: hw/ppc/mpc8544_guts.c
>
> New World (mac99)
> M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
> L: qemu-ppc@nongnu.org
> S: Odd Fixes
> F: hw/ppc/mac_newworld.c
> @@ -1285,8 +1283,6 @@ F: pc-bios/qemu_vga.ndrv
>
> Old World (g3beige)
> M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
> L: qemu-ppc@nongnu.org
> S: Odd Fixes
> F: hw/ppc/mac_oldworld.c
> @@ -1300,8 +1296,6 @@ F: pc-bios/qemu_vga.ndrv
>
> PReP
> M: Hervé Poussineau <hpoussin@reactos.org>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
> L: qemu-ppc@nongnu.org
> S: Maintained
> F: hw/ppc/prep.c
> @@ -1356,8 +1350,6 @@ F: hw/ppc/virtex_ml507.c
>
> sam460ex
> M: BALATON Zoltan <balaton@eik.bme.hu>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
> L: qemu-ppc@nongnu.org
> S: Maintained
> F: hw/ppc/sam460ex.c
> @@ -1371,7 +1363,6 @@ F: roms/u-boot-sam460ex
>
> pegasos2
> M: BALATON Zoltan <balaton@eik.bme.hu>
> -R: David Gibson <david@gibson.dropbear.id.au>
> L: qemu-ppc@nongnu.org
> S: Maintained
> F: hw/ppc/pegasos2.c
>

Re: [PATCH v2 2/5] MAINTAINERS: Remove David & Greg as reviewers for a number of boards

[Cédric Le Goater]

On 9/24/21 11:55, David Gibson wrote:
> Greg and I are moving towards other areas and no longer have capacity to
> act as regular reviewers for several of the secondary ppc machine types.
> So, remove ourselves as reviewers for Macintosh, PReP, sam460ex and
> pegasos2 in MAINTAINERS.
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> Reviewed-by: Greg Kurz <groug@kaod.org>

Reviewed-by: Cédric Le Goater <clg@kaod.org>

> ---
>   MAINTAINERS | 9 ---------
>   1 file changed, 9 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3408618f2a..20436be86f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1265,8 +1265,6 @@ F: hw/ppc/mpc8544_guts.c
>   
>   New World (mac99)
>   M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
>   S: Odd Fixes
>   F: hw/ppc/mac_newworld.c
> @@ -1285,8 +1283,6 @@ F: pc-bios/qemu_vga.ndrv
>   
>   Old World (g3beige)
>   M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
>   S: Odd Fixes
>   F: hw/ppc/mac_oldworld.c
> @@ -1300,8 +1296,6 @@ F: pc-bios/qemu_vga.ndrv
>   
>   PReP
>   M: Hervé Poussineau <hpoussin@reactos.org>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
>   S: Maintained
>   F: hw/ppc/prep.c
> @@ -1356,8 +1350,6 @@ F: hw/ppc/virtex_ml507.c
>   
>   sam460ex
>   M: BALATON Zoltan <balaton@eik.bme.hu>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
>   S: Maintained
>   F: hw/ppc/sam460ex.c
> @@ -1371,7 +1363,6 @@ F: roms/u-boot-sam460ex
>   
>   pegasos2
>   M: BALATON Zoltan <balaton@eik.bme.hu>
> -R: David Gibson <david@gibson.dropbear.id.au>
>   L: qemu-ppc@nongnu.org
>   S: Maintained
>   F: hw/ppc/pegasos2.c
> 

Re: [PATCH v2 2/5] MAINTAINERS: Remove David & Greg as reviewers for a number of boards

[Greg Kurz]

There's a typo in Hervé's address.

Fixing it from hpussin@reactos.org to hpoussin@reactos.org .

On Fri, 24 Sep 2021 19:55:33 +1000
David Gibson <david@gibson.dropbear.id.au> wrote:

> Greg and I are moving towards other areas and no longer have capacity to
> act as regular reviewers for several of the secondary ppc machine types.
> So, remove ourselves as reviewers for Macintosh, PReP, sam460ex and
> pegasos2 in MAINTAINERS.
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> Reviewed-by: Greg Kurz <groug@kaod.org>
> ---
>  MAINTAINERS | 9 ---------
>  1 file changed, 9 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3408618f2a..20436be86f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1265,8 +1265,6 @@ F: hw/ppc/mpc8544_guts.c
>  
>  New World (mac99)
>  M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>  L: qemu-ppc@nongnu.org
>  S: Odd Fixes
>  F: hw/ppc/mac_newworld.c
> @@ -1285,8 +1283,6 @@ F: pc-bios/qemu_vga.ndrv
>  
>  Old World (g3beige)
>  M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>  L: qemu-ppc@nongnu.org
>  S: Odd Fixes
>  F: hw/ppc/mac_oldworld.c
> @@ -1300,8 +1296,6 @@ F: pc-bios/qemu_vga.ndrv
>  
>  PReP
>  M: Hervé Poussineau <hpoussin@reactos.org>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>  L: qemu-ppc@nongnu.org
>  S: Maintained
>  F: hw/ppc/prep.c
> @@ -1356,8 +1350,6 @@ F: hw/ppc/virtex_ml507.c
>  
>  sam460ex
>  M: BALATON Zoltan <balaton@eik.bme.hu>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>  L: qemu-ppc@nongnu.org
>  S: Maintained
>  F: hw/ppc/sam460ex.c
> @@ -1371,7 +1363,6 @@ F: roms/u-boot-sam460ex
>  
>  pegasos2
>  M: BALATON Zoltan <balaton@eik.bme.hu>
> -R: David Gibson <david@gibson.dropbear.id.au>
>  L: qemu-ppc@nongnu.org
>  S: Maintained
>  F: hw/ppc/pegasos2.c

[PATCH v2 3/5] MAINTAINERS: Remove David & Greg as reviewers/co-maintainers of powernv

[David Gibson]

With our interests moving to other areas, Greg and myself no longer have
capacity to be regular reviewers of code for the powernv machine type, let
alone co-maintainers.  Additionally, not being IBM employees, we don't have
easy access to the hardware information we'd need for good review.

Therefore, remove our names as reviewers and/or co-maintainers of the
powernv machine type, and the related XIVE interrupt controller.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Greg Kurz <groug@kaod.org>
---
 MAINTAINERS | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 20436be86f..c82a634a70 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1329,8 +1329,6 @@ F: tests/qtest/libqos/rtas*
 
 PowerNV (Non-Virtualized)
 M: Cédric Le Goater <clg@kaod.org>
-M: David Gibson <david@gibson.dropbear.id.au>
-M: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
 S: Maintained
 F: hw/ppc/pnv*
@@ -2222,8 +2220,6 @@ T: git https://github.com/philmd/qemu.git fw_cfg-next
 
 XIVE
 M: Cédric Le Goater <clg@kaod.org>
-R: David Gibson <david@gibson.dropbear.id.au>
-R: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
 S: Supported
 F: hw/*/*xive*
-- 
2.31.1

Re: [PATCH v2 3/5] MAINTAINERS: Remove David & Greg as reviewers/co-maintainers of powernv

[Cédric Le Goater]

On 9/24/21 11:55, David Gibson wrote:
> With our interests moving to other areas, Greg and myself no longer have
> capacity to be regular reviewers of code for the powernv machine type, let
> alone co-maintainers.  Additionally, not being IBM employees, we don't have
> easy access to the hardware information we'd need for good review.
> 
> Therefore, remove our names as reviewers and/or co-maintainers of the
> powernv machine type, and the related XIVE interrupt controller.
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> Reviewed-by: Greg Kurz <groug@kaod.org>

Acked-by: Cédric Le Goater <clg@kaod.org>

> ---
>   MAINTAINERS | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 20436be86f..c82a634a70 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1329,8 +1329,6 @@ F: tests/qtest/libqos/rtas*
>   
>   PowerNV (Non-Virtualized)
>   M: Cédric Le Goater <clg@kaod.org>
> -M: David Gibson <david@gibson.dropbear.id.au>
> -M: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
>   S: Maintained
>   F: hw/ppc/pnv*
> @@ -2222,8 +2220,6 @@ T: git https://github.com/philmd/qemu.git fw_cfg-next
>   
>   XIVE
>   M: Cédric Le Goater <clg@kaod.org>
> -R: David Gibson <david@gibson.dropbear.id.au>
> -R: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
>   S: Supported
>   F: hw/*/*xive*
> 

[PATCH v2 4/5] MAINTAINERS: Add information for OpenPIC

[David Gibson]

The OpenPIC interrupt controller was once the de facto standard on ppc
machines.  In qemu it's now only used on some Macintosh and the
Freescale e500 machine.  It has no listed maintainer, and as far as I
know, no-one who's really familiar with it any more.

Since I'm moving away from the area, I no longer have capacity to do even
minimal maintenance of it under the auspices of the ppc targets in general.

Therefore, this patch lists the main part of openpic, and marks it as
"Odd Fixes" to be looked after by Mark Cave-Ayland who handles the
Macintosh targets.  The openpic_kvm variant is only used on e500, so
add it to the files for that machine type (itself already Orphaned).

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index c82a634a70..937c191c8d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1256,6 +1256,8 @@ F: hw/pci-host/ppce500.c
 F: include/hw/ppc/ppc_e500.h
 F: include/hw/pci-host/ppce500.h
 F: pc-bios/u-boot.e500
+F: hw/intc/openpic_kvm.h
+F: include/hw/ppc/openpic_kvm.h
 
 mpc8544ds
 L: qemu-ppc@nongnu.org
@@ -2255,6 +2257,12 @@ F: net/can/*
 F: hw/net/can/*
 F: include/net/can_*.h
 
+OpenPIC interrupt controller
+M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
+S: Odd Fixes
+F: hw/intc/openpic.c
+F: include/hw/ppc/openpic.h
+
 Subsystems
 ----------
 Overall Audio backends
-- 
2.31.1

Re: [PATCH v2 4/5] MAINTAINERS: Add information for OpenPIC

[Cédric Le Goater]

On 9/24/21 11:55, David Gibson wrote:
> The OpenPIC interrupt controller was once the de facto standard on ppc
> machines.  In qemu it's now only used on some Macintosh and the
> Freescale e500 machine.  It has no listed maintainer, and as far as I
> know, no-one who's really familiar with it any more.
> 
> Since I'm moving away from the area, I no longer have capacity to do even
> minimal maintenance of it under the auspices of the ppc targets in general.
> 
> Therefore, this patch lists the main part of openpic, and marks it as
> "Odd Fixes" to be looked after by Mark Cave-Ayland who handles the
> Macintosh targets.  The openpic_kvm variant is only used on e500, so
> add it to the files for that machine type (itself already Orphaned).
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>

Reviewed-by: Cédric Le Goater <clg@kaod.org>

> ---
>   MAINTAINERS | 8 ++++++++
>   1 file changed, 8 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c82a634a70..937c191c8d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1256,6 +1256,8 @@ F: hw/pci-host/ppce500.c
>   F: include/hw/ppc/ppc_e500.h
>   F: include/hw/pci-host/ppce500.h
>   F: pc-bios/u-boot.e500
> +F: hw/intc/openpic_kvm.h
> +F: include/hw/ppc/openpic_kvm.h
>   
>   mpc8544ds
>   L: qemu-ppc@nongnu.org
> @@ -2255,6 +2257,12 @@ F: net/can/*
>   F: hw/net/can/*
>   F: include/net/can_*.h
>   
> +OpenPIC interrupt controller
> +M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> +S: Odd Fixes
> +F: hw/intc/openpic.c
> +F: include/hw/ppc/openpic.h
> +
>   Subsystems
>   ----------
>   Overall Audio backends
> 

Re: [PATCH v2 4/5] MAINTAINERS: Add information for OpenPIC

[Greg Kurz]

On Fri, 24 Sep 2021 19:55:35 +1000
David Gibson <david@gibson.dropbear.id.au> wrote:

> The OpenPIC interrupt controller was once the de facto standard on ppc
> machines.  In qemu it's now only used on some Macintosh and the
> Freescale e500 machine.  It has no listed maintainer, and as far as I
> know, no-one who's really familiar with it any more.
> 
> Since I'm moving away from the area, I no longer have capacity to do even
> minimal maintenance of it under the auspices of the ppc targets in general.
> 
> Therefore, this patch lists the main part of openpic, and marks it as
> "Odd Fixes" to be looked after by Mark Cave-Ayland who handles the
> Macintosh targets.  The openpic_kvm variant is only used on e500, so
> add it to the files for that machine type (itself already Orphaned).
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> ---

Reviewed-by: Greg Kurz <groug@kaod.org>

>  MAINTAINERS | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c82a634a70..937c191c8d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1256,6 +1256,8 @@ F: hw/pci-host/ppce500.c
>  F: include/hw/ppc/ppc_e500.h
>  F: include/hw/pci-host/ppce500.h
>  F: pc-bios/u-boot.e500
> +F: hw/intc/openpic_kvm.h
> +F: include/hw/ppc/openpic_kvm.h
>  
>  mpc8544ds
>  L: qemu-ppc@nongnu.org
> @@ -2255,6 +2257,12 @@ F: net/can/*
>  F: hw/net/can/*
>  F: include/net/can_*.h
>  
> +OpenPIC interrupt controller
> +M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> +S: Odd Fixes
> +F: hw/intc/openpic.c
> +F: include/hw/ppc/openpic.h
> +
>  Subsystems
>  ----------
>  Overall Audio backends

[PATCH v2 5/5] MAINTAINERS: Demote sPAPR from "Supported" to "Maintained"

[David Gibson]

qemu/KVM on Power is no longer my primary job responsibility, nor Greg
Kurz'.  I still have some time for upstream maintenance, but it's no longer
accurate to say that I'm paid to do so.  Therefore, reduce sPAPR (the
"pseries" machine type) from Supported to Maintained.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Greg Kurz <groug@kaod.org>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 937c191c8d..bfae4a55e8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1316,7 +1316,7 @@ sPAPR
 M: David Gibson <david@gibson.dropbear.id.au>
 M: Greg Kurz <groug@kaod.org>
 L: qemu-ppc@nongnu.org
-S: Supported
+S: Maintained
 F: hw/*/spapr*
 F: include/hw/*/spapr*
 F: hw/*/xics*
-- 
2.31.1

Re: [PATCH v2 5/5] MAINTAINERS: Demote sPAPR from "Supported" to "Maintained"

[Cédric Le Goater]

On 9/24/21 11:55, David Gibson wrote:
> qemu/KVM on Power is no longer my primary job responsibility, nor Greg
> Kurz'.  I still have some time for upstream maintenance, but it's no longer
> accurate to say that I'm paid to do so.  Therefore, reduce sPAPR (the
> "pseries" machine type) from Supported to Maintained.
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> Reviewed-by: Greg Kurz <groug@kaod.org>

Reviewed-by: Cédric Le Goater <clg@kaod.org>


> ---
>   MAINTAINERS | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 937c191c8d..bfae4a55e8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1316,7 +1316,7 @@ sPAPR
>   M: David Gibson <david@gibson.dropbear.id.au>
>   M: Greg Kurz <groug@kaod.org>
>   L: qemu-ppc@nongnu.org
> -S: Supported
> +S: Maintained
>   F: hw/*/spapr*
>   F: include/hw/*/spapr*
>   F: hw/*/xics*
> 

Re: [PATCH v2 5/5] MAINTAINERS: Demote sPAPR from "Supported" to "Maintained"

[Philippe Mathieu-Daudé]

On 9/24/21 11:55, David Gibson wrote:
> qemu/KVM on Power is no longer my primary job responsibility, nor Greg
> Kurz'.  I still have some time for upstream maintenance, but it's no longer
> accurate to say that I'm paid to do so.  Therefore, reduce sPAPR (the
> "pseries" machine type) from Supported to Maintained.
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> Reviewed-by: Greg Kurz <groug@kaod.org>
> ---
>   MAINTAINERS | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Re: [PATCH v2 0/5] Reduce load on ppc target maintainers

[Philippe Mathieu-Daudé]

Hervé email is invalid, so forwarding only the cover to him,
the series is available here:
https://lore.kernel.org/qemu-devel/20210924095536.441849-1-david@gibson.dropbear.id.au/

On 9/24/21 11:55, David Gibson wrote:
> Greg Kurz and myself have been co-maintainers for the ppc and ppc64
> targets for some time now.  However, both our day job responsibilities
> and interests are leading us towards other areas, so we have less time
> to devote to this any more.
> 
> Therefore, here's a bunch of updates to MAINTAINERS, intended to
> reduce the load on us.  Mostly this is marking fairly obscure
> platforms as orphaned (if someone wants to take over maintainership,
> let me know ASAP).  Bigger changes may be coming, but we haven't
> decided exactly what that's going to look like yet.
> 
> Changes since v1:
>   * Reworked how OpenPIC is listed
> 
> David Gibson (5):
>    MAINTAINERS: Orphan obscure ppc platforms
>    MAINTAINERS: Remove David & Greg as reviewers for a number of boards
>    MAINTAINERS: Remove David & Greg as reviewers/co-maintainers of
>      powernv
>    MAINTAINERS: Add information for OpenPIC
>    MAINTAINERS: Demote sPAPR from "Supported" to "Maintained"
> 
>   MAINTAINERS | 42 ++++++++++++++----------------------------
>   1 file changed, 14 insertions(+), 28 deletions(-)
> 

Re: [PATCH v2 0/5] Reduce load on ppc target maintainers

[Philippe Mathieu-Daudé]

On 9/24/21 11:55, David Gibson wrote:
> Greg Kurz and myself have been co-maintainers for the ppc and ppc64
> targets for some time now.  However, both our day job responsibilities
> and interests are leading us towards other areas, so we have less time
> to devote to this any more.
> 
> Therefore, here's a bunch of updates to MAINTAINERS, intended to
> reduce the load on us.  Mostly this is marking fairly obscure
> platforms as orphaned (if someone wants to take over maintainership,
> let me know ASAP).  Bigger changes may be coming, but we haven't
> decided exactly what that's going to look like yet.
> 
> Changes since v1:
>   * Reworked how OpenPIC is listed
> 
> David Gibson (5):
>    MAINTAINERS: Orphan obscure ppc platforms
>    MAINTAINERS: Remove David & Greg as reviewers for a number of boards
>    MAINTAINERS: Remove David & Greg as reviewers/co-maintainers of
>      powernv
>    MAINTAINERS: Add information for OpenPIC
>    MAINTAINERS: Demote sPAPR from "Supported" to "Maintained"
> 
>   MAINTAINERS | 42 ++++++++++++++----------------------------
>   1 file changed, 14 insertions(+), 28 deletions(-)
> 

So after this series we still have:

PowerPC TCG CPUs
M: David Gibson <david@gibson.dropbear.id.au>
M: Greg Kurz <groug@kaod.org>
L: qemu-ppc@nongnu.org
S: Maintained
F: target/ppc/
F: hw/ppc/
F: include/hw/ppc/
F: disas/ppc.c
F: tests/acceptance/machine_ppc.py

You might want to drop the *hw/ppc* lines which should
be covered elsewhere now.

tests/acceptance/machine_ppc.py should be split in multiple
files to ease the tests maintainance.

Thanks for staying PPC/TCG maintainers :)

Regards,

Phil.

Re: [PATCH v2 0/5] Reduce load on ppc target maintainers

[David Gibson]

[-- Attachment #1: Type: text/plain, Size: 2200 bytes --]

On Fri, Sep 24, 2021 at 05:16:45PM +0200, Philippe Mathieu-Daudé wrote:
65;6402;1c> On 9/24/21 11:55, David Gibson wrote:
> > Greg Kurz and myself have been co-maintainers for the ppc and ppc64
> > targets for some time now.  However, both our day job responsibilities
> > and interests are leading us towards other areas, so we have less time
> > to devote to this any more.
> > 
> > Therefore, here's a bunch of updates to MAINTAINERS, intended to
> > reduce the load on us.  Mostly this is marking fairly obscure
> > platforms as orphaned (if someone wants to take over maintainership,
> > let me know ASAP).  Bigger changes may be coming, but we haven't
> > decided exactly what that's going to look like yet.
> > 
> > Changes since v1:
> >   * Reworked how OpenPIC is listed
> > 
> > David Gibson (5):
> >    MAINTAINERS: Orphan obscure ppc platforms
> >    MAINTAINERS: Remove David & Greg as reviewers for a number of boards
> >    MAINTAINERS: Remove David & Greg as reviewers/co-maintainers of
> >      powernv
> >    MAINTAINERS: Add information for OpenPIC
> >    MAINTAINERS: Demote sPAPR from "Supported" to "Maintained"
> > 
> >   MAINTAINERS | 42 ++++++++++++++----------------------------
> >   1 file changed, 14 insertions(+), 28 deletions(-)
> > 
> 
> So after this series we still have:
> 
> PowerPC TCG CPUs
> M: David Gibson <david@gibson.dropbear.id.au>
> M: Greg Kurz <groug@kaod.org>
> L: qemu-ppc@nongnu.org
> S: Maintained
> F: target/ppc/
> F: hw/ppc/
> F: include/hw/ppc/
> F: disas/ppc.c
> F: tests/acceptance/machine_ppc.py
> 
> You might want to drop the *hw/ppc* lines which should
> be covered elsewhere now.

Good idea, I'll revise that.

> tests/acceptance/machine_ppc.py should be split in multiple
> files to ease the tests maintainance.

Also a good idea.

> Thanks for staying PPC/TCG maintainers :)

Uhhh... we're not.  I just haven't tackled that side of things yet to
reduce the number of spinning plates.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]