From: Joshua Crofts <joshua.crofts1@gmail.com>
To: "Maxwell Doose" <maxwell@maxwelld.cc>
Cc: <linux-iio@vger.kernel.org>, <jic23@kernel.org>,
<andy@kernel.org>, <dlechner@baylibre.com>, <nuno.sa@analog.com>
Subject: Re: [RFC] Maintainer entry profile/contributor guide for IIO
Date: Tue, 18 Aug 2026 09:22:40 +0200 [thread overview]
Message-ID: <20260818092240.000061f2@gmail.com> (raw)
In-Reply-To: <DKRG0P23P5NA.171S6RGSLSI3W@maxwelld.cc>
On Mon, 17 Aug 2026 13:50:29 -0500
"Maxwell Doose" <maxwell@maxwelld.cc> wrote:
> On Mon Aug 17, 2026 at 4:18 AM CDT
> Joshua Crofts <joshua.crofts1@gmail.com> wrote:
>
> > Hi all,
> >
> > I was browsing lore and checked out the ksummit mailing list, where the
> > topic about guiding new contributors arose [1]. New contributors tend to
> > make the same mistakes when sending patches, causing reviewers to point
> > these out all the time over and over again. For IIO, this is definitely the
> > case (I myself send an email telling people not to send a v2 in reply to a
> > v1 several times a week). Other subsystems have a "Maintainer entry profile"
> > that contains subsystem-specific process info (DAMON for example [2]) and
> > (sometimes even [3]) a document describing the code style of the subsystem
> > (this would be a great place where to mention things like not using
> > kernel.h in new drivers etc.). I'm happy to create both of the documents
> > but it's always great to hear other people's ideas!
> >
>
> Took a look at [1] and it looks like these are issues across multiple
> mailing lists so maybe we should add these to the main
> submitting-patches documentation? And then for other IIO-specific things
> we can put those in a maintainer entry profile.
Fair, the "don't reply with a v2 to a v1" comment was a bit off considering
I was mainly talking about IIO specific documentation, however having a central
document (or documents) of sorts for the IIO code style and process is a nice
thing to have, I'm sure even the most seasoned reviewers forget stuff occasionally.
>
> > Second of all, the idea of having a bot that would automatically detect new
> > contributors (i.e. the email they're submitting the patch with doesn't show
> > up in `git log --author`) and send an email reminding them of the basic rules
> > (while also referring to the entry profile mentioned above) sounds like a
> > great idea, but of course hosting and maintaining are pain points.
> >
>
> Hm...this one is more difficult. I guess a good first question to ask
> would be "can this be hosted on kernel.org infastructure?" But maybe we
> can also ask Greg KH about how he does his automated bot.
From what I gathered, kernel.org doesn't really provide infrastructure for this
type of stuff :( If you look at [1] you see that netdev got overloaded with patches
so they made their own bot for this exact purpose. One could just spin up a
Docker container that periodically pulls from lore using lei and runs a git log
with the author's email against mainline/testing/togreg.
I believe it was David who originally proposed an IIO bot [1], primarily for code
inspection.
[1] https://lore.kernel.org/all/4742ea18-f3df-46dd-aad4-18d1e5f03f7f@baylibre.com/
--
Kind regards,
Joshua Crofts
next prev parent reply other threads:[~2026-08-18 7:22 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-17 9:18 [RFC] Maintainer entry profile/contributor guide for IIO Joshua Crofts
2026-08-17 18:50 ` Maxwell Doose
2026-08-18 7:22 ` Joshua Crofts [this message]
2026-08-18 0:31 ` David Lechner
2026-08-18 7:06 ` Joshua Crofts
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=20260818092240.000061f2@gmail.com \
--to=joshua.crofts1@gmail.com \
--cc=andy@kernel.org \
--cc=dlechner@baylibre.com \
--cc=jic23@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=maxwell@maxwelld.cc \
--cc=nuno.sa@analog.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.