[PATCH] MAINTAINERS: Update section for cpuidle-psci

Lorenzo Pieralisi lorenzo.pieralisi at arm.com
Wed Sep 30 05:12:54 EDT 2020


On Wed, Sep 23, 2020 at 03:44:53PM +0200, Ulf Hansson wrote:
> On Wed, 23 Sep 2020 at 10:11, Lorenzo Pieralisi
> <lorenzo.pieralisi at arm.com> wrote:
> >
> > On Mon, Sep 21, 2020 at 05:49:52PM +0200, Rafael J. Wysocki wrote:
> > > On Fri, Aug 14, 2020 at 2:37 PM Ulf Hansson <ulf.hansson at linaro.org> wrote:
> > > >
> > > > Extend the file pattern to also include cpuidle-psci.h and
> > > > cpuidle-psci-domain.c. Moreover, add myself as a co-maintainer,
> > > > particularly to help with the support for PM domains.
> > > >
> > > > Cc: Sudeep Holla <sudeep.holla at arm.com>
> > > > Cc: Lorenzo Pieralisi <Lorenzo.Pieralisi at arm.com>
> > > > Signed-off-by: Ulf Hansson <ulf.hansson at linaro.org>
> > >
> > > Any objections from the original maintainers?
> >
> > The PSCI driver providing support for platform coordinated idle states
> > (that this patch is targeting AFAICS) does not require more maintainers,
> > that support is complete and Sudeep and I will take care of refactoring
> > and fixes.
> >
> > If the OSI bits in it require a specific maintainer I would prefer those
> > bits to be moved into a specific file and maintainership added
> > _explicitly_ for that file only.
> 
> My main concern is that the OSI support (and some related changes for
> genpd) is still being worked upon. The corresponding code deserves to
> be looked after by someone. I offer my help with this.

All OSI changes can go into cpuidle-psci-domain.c, you can add a
maintainer entry for that.

> I guess we can split up the code as you suggest, but perhaps another
> way would be to explicitly state my part of the responsibility in
> MAINTAINERS. Along the lines of this:
> 
> M:     Ulf Hansson <ulf.hansson at linaro.org> # OSI support / PM domains
> 
> Would that be okay to you?

No because there should be no need, as I have already said. If you need
to change cpuidle-psci.c send patches please and we will review them.

Thanks,
Lorenzo

> > Thanks,
> > Lorenzo
> >
> > > >  MAINTAINERS | 3 ++-
> > > >  1 file changed, 2 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index f9ae8ea54e27..66821197175e 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -4556,10 +4556,11 @@ F:      drivers/cpuidle/cpuidle-exynos.c
> > > >  CPUIDLE DRIVER - ARM PSCI
> > > >  M:     Lorenzo Pieralisi <lorenzo.pieralisi at arm.com>
> > > >  M:     Sudeep Holla <sudeep.holla at arm.com>
> > > > +M:     Ulf Hansson <ulf.hansson at linaro.org>
> > > >  L:     linux-pm at vger.kernel.org
> > > >  L:     linux-arm-kernel at lists.infradead.org
> > > >  S:     Supported
> > > > -F:     drivers/cpuidle/cpuidle-psci.c
> > > > +F:     drivers/cpuidle/cpuidle-psci*
> > > >
> > > >  CRAMFS FILESYSTEM
> > > >  M:     Nicolas Pitre <nico at fluxnic.net>
> > > > --
> > > > 2.25.1
> > > >



More information about the linux-arm-kernel mailing list