[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CADrjBPpNpv-xaWri1KuFPN+7BAAu90Ti9t=fVth7_cjXnuj4Jw@mail.gmail.com>
Date: Wed, 18 Sep 2024 15:52:40 +0100
From: Peter Griffin <peter.griffin@...aro.org>
To: Conor Dooley <conor@...nel.org>
Cc: André Draszik <andre.draszik@...aro.org>,
Catalin Marinas <catalin.marinas@....com>, Will Deacon <will@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Alim Akhtar <alim.akhtar@...sung.com>, Tudor Ambarus <tudor.ambarus@...aro.org>,
Will McVicker <willmcvicker@...gle.com>, kernel-team@...roid.com,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-samsung-soc@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH 3/3] MAINTAINERS: add myself for Google Tensor SoC
Hi Conor,
On Tue, 17 Sept 2024 at 22:19, Conor Dooley <conor@...nel.org> wrote:
>
> On Mon, Sep 16, 2024 at 08:42:03PM +0100, Peter Griffin wrote:
> > Hi André,
> >
> > On Mon, 16 Sept 2024 at 17:58, André Draszik <andre.draszik@...aro.org> wrote:
> > >
> > > Add myself as maintainer for the Google Tensor SoC alongside Peter.
> > >
> > > Signed-off-by: André Draszik <andre.draszik@...aro.org>
> > > ---
> > > MAINTAINERS | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 2cdd7cacec86..b6edb21b4f2d 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -9669,6 +9669,7 @@ T: git git://git.kernel.org/pub/scm/linux/kernel/git/chrome-platform/linux.git
> > > F: drivers/firmware/google/
> > >
> > > GOOGLE TENSOR SoC SUPPORT
> > > +M: André Draszik <andre.draszik@...aro.org>
> >
> > Please update this to: -
> >
> > +R: André Draszik <andre.draszik@...aro.org>
> >
> > The definition of which is
> >
> > R: Designated *Reviewer*: FullName <address@...ain>
> > These reviewers should be CCed on patches.
>
> I find this email really weird. If you discussed something off-list and
> Andre misunderstood that you wanted him as a reviewer not a maintainer,
> that's fine - but you need to explain why to the rest of us.
Apologies for the brevity, it wasn't my intention for it to come
across as weird.
We have a small handful of files here currently, so I don't think it
requires a team of maintainers, it's not particularly high volume.
Having said that, Andre, Tudor and myself usually CC each other on our
patches for gs101 and Pixel 6 so it makes sense for the tooling to
include them as reviewers on patches that may only come to me
otherwise. Other ICs like Maxim 20339 on the board have their own
dedicated MAINTAINERS entry.
Peter.
Powered by blists - more mailing lists