Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8;
Date:   Mon, 11 Dec 2023 17:19:12 +0000
From:   William Breathitt Gray <william.gray@linaro.org>
To:     Fabrice Gasnier <fabrice.gasnier@foss.st.com>
Cc:     linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 2/2] MAINTAINERS: add myself as counter watch events
 tool maintainer
Message-ID: <ZXdEkO5u/IDKEgaV@ubuntu-server-vm-macos>
References: <20231206164726.418990-1-fabrice.gasnier@foss.st.com>
 <20231206164726.418990-3-fabrice.gasnier@foss.st.com>
 <ZXcx+94QW2pWDBxM@ubuntu-server-vm-macos>
 <81bbf2f7-6db2-47de-9761-0b196b5348d2@foss.st.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="DqZxG5w8JvyzHbBp"
Content-Disposition: inline
In-Reply-To: <81bbf2f7-6db2-47de-9761-0b196b5348d2@foss.st.com>
Precedence: bulk


--DqZxG5w8JvyzHbBp
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Dec 11, 2023 at 06:09:44PM +0100, Fabrice Gasnier wrote:
> On 12/11/23 16:59, William Breathitt Gray wrote:
> > On Wed, Dec 06, 2023 at 05:47:26PM +0100, Fabrice Gasnier wrote:
> >> Add MAINTAINERS entry for the counter watch events tool. William has
> >> been asking to add at least me as the point of contact for this utilit=
y.
> >>
> >> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com>
> >> ---
> >> Changes in v3: Add MAINTAINERS entry. This is a split of another patch
> >> series[1].
> >> [1] https://lore.kernel.org/lkml/20230922143920.3144249-1-fabrice.gasn=
ier@foss.st.com/
> >> ---
> >>  MAINTAINERS | 5 +++++
> >>  1 file changed, 5 insertions(+)
> >>
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index dd5de540ec0b..b8541ab7866a 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -5322,6 +5322,11 @@ F:	include/linux/counter.h
> >>  F:	include/uapi/linux/counter.h
> >>  F:	tools/counter/
> >> =20
> >> +COUNTER WATCH EVENTS TOOL
> >> +M:	Fabrice Gasnier <fabrice.gasnier@foss.st.com>
> >> +S:	Maintained
> >> +F:	tools/counter/counter_watch_events.c
> >=20
> > Add an L line as well for the linux-iio@vger.kernel.org address so
> > discussions get sent to our public mailing list.
>=20
> Hi William,
>=20
> I can add it, yes. But just to be sure, with current patch,
> get_maintainer.pl gives me:
>=20
> ./scripts/get_maintainer.pl tools/counter/counter_watch_events.c
> Fabrice Gasnier <fabrice.gasnier@foss.st.com> (maintainer:COUNTER WATCH
> EVENTS TOOL)
> William Breathitt Gray <william.gray@linaro.org> (maintainer:COUNTER
> SUBSYSTEM)
> linux-iio@vger.kernel.org (open list:COUNTER SUBSYSTEM)
> linux-kernel@vger.kernel.org (open list)
>=20
> So is it really needed to add an L line ?
>=20
> Best Regards,
> Fabrice

I think the get_maintainer.pl script tries its best to find a list when
one isn't specified, so when it doesn't find a list under the COUNTER
WATCH EVENTS TOOL entry, it finds the "tools/counter" directory
specified under the COUNTER SUBSYSTEM entry and pulls in that list.

Although we end up with the same list, that level of indirection makes
it somewhat ambiguous to a user whether that's the correct list to mail.
So just to be proper and clear, we should provide an explicit L line so
there is no confusion.

William Breathitt Gray

--DqZxG5w8JvyzHbBp
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQSNN83d4NIlKPjon7a1SFbKvhIjKwUCZXdEkAAKCRC1SFbKvhIj
K7IpAQCwOWCY90BHIn4sKetysj1BC1aPzwZTDdzLCYI+4tKFbAEArcPeysYkeWXY
eqnjLqD+msPvPKVxDRMOLJZBn70lcQs=
=GTfg
-----END PGP SIGNATURE-----

--DqZxG5w8JvyzHbBp--