Re: [PATCH 4/5] MAINTAINERS: Explicitly orphan OpenPIC

[Mark Cave-Ayland]

On 21/09/2021 06:26, Cédric Le Goater wrote:

> On 9/21/21 06:17, 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 a few old
> > embedded machine types.  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, explicitly list it in MAINTAINERS as orphaned.
>
> I guess Mark can take over since OpenPIC falls under the Mac world machines.
>
> C.

Well I can at least test patches on the Mac machines and make sure they don't break 
anything :)

I don't think I've ever really looked at hw/intc/openpic_kvm.c and it seems it is 
only used by the e500 machine which this patchset marks as Orphaned.

Perhaps then we could have hw/intc/openpic.c as Odd Fixes and also move 
hw/intc/openpic_kvm.c over to Orphaned at the same time as the e500 machine?

> > Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> > ---
> >   MAINTAINERS | 5 +++++
> >   1 file changed, 5 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 05c057e793..313630ac1e 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -2250,6 +2250,11 @@ F: net/can/*
> >   F: hw/net/can/*
> >   F: include/net/can_*.h
> > +OpenPIC interrupt controller
> > +S: Orphan
> > +F: hw/intc/openpic*.c
> > +F: include/hw/ppc/openpic*.h
> > +
> >   Subsystems
> >   ----------
> >   Overall Audio backends


ATB,

Mark.