Re: [Qemu-devel] [PATCH] MAINTAINERS: update maintainer for target-arm and ARM devboards

[Paul Brook <paul@codesourcery.com>]

> > Add myself as co-maintainer alongside Paul Brook for the TCG ARM
> > guest implementation (target-arm) and the ARM dev boards (integratorcp,
> > realview, stellaris, versatilepb).
> > 
> > Signed-off-by: Peter Maydell<peter.maydell@linaro.org>
> 
> Acked-by: Anthony Liguori <aliguori@us.ibm.com>
> 
> Paul, speak now or forever hold your peace ;-)

I approve in general principle, and haven't being paying enough attention to 
make a specific technical judgement.

For the record I consider having multiple maintainers to be a good thing.

IMO the role of a maintainer should be to make sure sub-par code is brought up 
up to standard before it is comitted.  Approving patches is easy, rejecting 
the bad patches[1] is where you add value.  Maintainership is more an 
indication that you're competent to review those bits of code than a grant of 
control.

Paul

[1] With constructive criticism on how to fix the problem.