Re: [Qemu-devel] [PATCH] MAINTAINERS: list myself as maintainer for various Arm boards

[Alistair Francis]

On Thu, Nov 8, 2018 at 5:41 AM Peter Maydell <address@hidden> wrote:
>
> In practice for most of the more-or-less orphan Arm board models,
> I will review patches and put them in via the target-arm tree.
> So list myself as an "Odd Fixes" status maintainer for them.
>
> This commit downgrades these boards to "Odd Fixes":
>  * Allwinner-A10
>  * Exynos
>  * Calxeda Highbank
>  * Canon DIGIC
>  * Musicpal
>  * nSeries
>  * Palm
>  * PXA2xx
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> Thomas Huth sent a patch which only addressed collie, but
> let's be consistent.
>
> The set of downgrades to odd-fixes is based mostly on my
> subjective impression of whether anybody has made any changes
> to the board models over the last year or so, so it's quite
> possible I've forgotten that one of them is in better shape
> maintainer-activity-wise. If the maintainers for those boards
> disagree (cc'd) I'm happy to re-upgrade them to Maintained.
>
> Arguably it would be nice to have a letter prefix for "this
> is the person who will deal with getting the patch into the
> tree, but somebody else will be the primary maintainer for
> patch review" -- that's the situation for the maintained/
> active boards like ASpeed.
> ---
>  MAINTAINERS | 36 +++++++++++++++++++++++++++---------
>  1 file changed, 27 insertions(+), 9 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 0d68e4bc5eb..4443105686d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -436,8 +436,9 @@ ARM Machines
>  ------------
>  Allwinner-a10
>  M: Beniamino Galvani <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Maintained
> +S: Odd Fixes
>  F: hw/*/allwinner*
>  F: include/hw/*/allwinner*
>  F: hw/arm/cubieboard.c
> @@ -496,27 +497,31 @@ F: tests/test-arm-mptimer.c
>
>  Exynos
>  M: Igor Mitsyanko <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Maintained
> +S: Odd Fixes
>  F: hw/*/exynos*
>  F: include/hw/arm/exynos4210.h
>
>  Calxeda Highbank
>  M: Rob Herring <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Maintained
> +S: Odd Fixes
>  F: hw/arm/highbank.c
>  F: hw/net/xgmac.c
>
>  Canon DIGIC
>  M: Antony Pavlov <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Maintained
> +S: Odd Fixes
>  F: include/hw/arm/digic.h
>  F: hw/*/digic*
>
>  Gumstix
>  M: Philippe Mathieu-Daudé <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
>  L: address@hidden
>  S: Odd Fixes
> @@ -524,8 +529,9 @@ F: hw/arm/gumstix.c
>
>  i.MX31
>  M: Peter Chubb <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Odd fixes
> +S: Odd Fixes
>  F: hw/*/imx*
>  F: include/hw/*/imx*
>  F: hw/arm/kzm.c
> @@ -555,20 +561,23 @@ F: include/hw/misc/iotkit-sysinfo.h
>
>  Musicpal
>  M: Jan Kiszka <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Maintained
> +S: Odd Fixes
>  F: hw/arm/musicpal.c
>
>  nSeries
>  M: Andrzej Zaborowski <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Maintained
> +S: Odd Fixes
>  F: hw/arm/nseries.c
>
>  Palm
>  M: Andrzej Zaborowski <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Maintained
> +S: Odd Fixes
>  F: hw/arm/palm.c
>
>  Real View
> @@ -582,8 +591,9 @@ F: include/hw/intc/realview_gic.h
>
>  PXA2XX
>  M: Andrzej Zaborowski <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
> -S: Maintained
> +S: Odd Fixes
>  F: hw/arm/mainstone.c
>  F: hw/arm/spitz.c
>  F: hw/arm/tosa.c
> @@ -608,6 +618,7 @@ F: hw/misc/arm_sysctl.c
>  Xilinx Zynq
>  M: Edgar E. Iglesias <address@hidden>
>  M: Alistair Francis <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
>  S: Maintained
>  F: hw/*/xilinx_*
> @@ -619,6 +630,7 @@ X: hw/ssi/xilinx_*
>  Xilinx ZynqMP
>  M: Alistair Francis <address@hidden>
>  M: Edgar E. Iglesias <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
>  S: Maintained
>  F: hw/*/xlnx*.c
> @@ -632,6 +644,7 @@ F: hw/arm/virt-acpi-build.c
>
>  STM32F205
>  M: Alistair Francis <address@hidden>
> +M: Peter Maydell <address@hidden>
>  S: Maintained
>  F: hw/arm/stm32f205_soc.c
>  F: hw/misc/stm32f2xx_syscfg.c
> @@ -643,11 +656,13 @@ F: include/hw/*/stm32*.h
>
>  Netduino 2
>  M: Alistair Francis <address@hidden>
> +M: Peter Maydell <address@hidden>
>  S: Maintained
>  F: hw/arm/netduino2.c

These two and the Xilinx boards seem a little out of place in this
patch. I agree they probably aren't maintained as well as they should
be, but the patch talks about orphaned boards and these four all have
active QEMU maintainers listed.

Alistair

>
>  SmartFusion2
>  M: Subbaraya Sundeep <address@hidden>
> +M: Peter Maydell <address@hidden>
>  S: Maintained
>  F: hw/arm/msf2-soc.c
>  F: hw/misc/msf2-sysreg.c
> @@ -660,11 +675,13 @@ F: include/hw/ssi/mss-spi.h
>
>  Emcraft M2S-FG484
>  M: Subbaraya Sundeep <address@hidden>
> +M: Peter Maydell <address@hidden>
>  S: Maintained
>  F: hw/arm/msf2-som.c
>
>  ASPEED BMCs
>  M: Cédric Le Goater <address@hidden>
> +M: Peter Maydell <address@hidden>
>  R: Andrew Jeffery <address@hidden>
>  R: Joel Stanley <address@hidden>
>  L: address@hidden
> @@ -676,6 +693,7 @@ F: include/hw/net/ftgmac100.h
>
>  NRF51
>  M: Joel Stanley <address@hidden>
> +M: Peter Maydell <address@hidden>
>  L: address@hidden
>  S: Maintained
>  F: hw/arm/nrf51_soc.c
> --
> 2.19.1
>
>