From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
To: Ulf Hansson <ulf.hansson@linaro.org>
Cc: Vincent Guittot <vincent.guittot@linaro.org>,
Benjamin Gaignard <benjamin.gaignard@st.com>,
Linux PM <linux-pm@vger.kernel.org>,
Stephen Boyd <sboyd@kernel.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
"Rafael J . Wysocki" <rjw@rjwysocki.net>,
Lina Iyer <ilina@codeaurora.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Linux ARM <linux-arm-kernel@lists.infradead.org>,
Sudeep Holla <sudeep.holla@arm.com>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Lukasz Luba <lukasz.luba@arm.com>
Subject: Re: [PATCH] MAINTAINERS: Update section for cpuidle-psci
Date: Wed, 30 Sep 2020 10:12:54 +0100 [thread overview]
Message-ID: <20200930091254.GA16862@e121166-lin.cambridge.arm.com> (raw)
In-Reply-To: <CAPDyKFrPRjgqzN2TXCh8A8P3H8g7FFQKRo8WZeLuCnMbMsT3bA@mail.gmail.com>
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@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@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@arm.com>
> > > > Cc: Lorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>
> > > > Signed-off-by: Ulf Hansson <ulf.hansson@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@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@arm.com>
> > > > M: Sudeep Holla <sudeep.holla@arm.com>
> > > > +M: Ulf Hansson <ulf.hansson@linaro.org>
> > > > L: linux-pm@vger.kernel.org
> > > > L: linux-arm-kernel@lists.infradead.org
> > > > S: Supported
> > > > -F: drivers/cpuidle/cpuidle-psci.c
> > > > +F: drivers/cpuidle/cpuidle-psci*
> > > >
> > > > CRAMFS FILESYSTEM
> > > > M: Nicolas Pitre <nico@fluxnic.net>
> > > > --
> > > > 2.25.1
> > > >
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-09-30 9:14 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-14 12:37 [PATCH] MAINTAINERS: Update section for cpuidle-psci Ulf Hansson
2020-09-21 15:49 ` Rafael J. Wysocki
2020-09-23 8:11 ` Lorenzo Pieralisi
2020-09-23 13:44 ` Ulf Hansson
2020-09-30 9:12 ` Lorenzo Pieralisi [this message]
2020-09-30 9:46 ` Ulf Hansson
2020-09-30 10:18 ` Sudeep Holla
2020-09-30 10:16 ` Sudeep Holla
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200930091254.GA16862@e121166-lin.cambridge.arm.com \
--to=lorenzo.pieralisi@arm.com \
--cc=benjamin.gaignard@st.com \
--cc=bjorn.andersson@linaro.org \
--cc=daniel.lezcano@linaro.org \
--cc=ilina@codeaurora.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=lukasz.luba@arm.com \
--cc=rafael@kernel.org \
--cc=rjw@rjwysocki.net \
--cc=sboyd@kernel.org \
--cc=sudeep.holla@arm.com \
--cc=ulf.hansson@linaro.org \
--cc=vincent.guittot@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).