Linux ATA/IDE development
 help / color / mirror / Atom feed
From: Niklas Cassel <cassel@kernel.org>
To: Matthias Goergens <matthias.goergens@gmail.com>
Cc: dlemoal@kernel.org, linux-ide@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: name the libata/linux for-next branch
Date: Thu, 24 Sep 2026 15:33:32 +0200	[thread overview]
Message-ID: <arUmrIiHwevxjjqK@ryzen> (raw)
In-Reply-To: <20260924101844.2403008-1-matthias.goergens@gmail.com>

Hello Matthias,

On Thu, Sep 24, 2026 at 06:18:44PM +0800, Matthias Goergens wrote:
> On Thu, Sep 24, 2026 at 11:32:19AM +0200, Niklas Cassel wrote:
> > I think this could have been written simpler, without references to dates.
> 
> Agreed, your wording is better.  I'll use it, together with the
> s/tree/branch/ fix, in a v2.
> 
> > But in reality, if you want to clean things up, why limit it to these three?
> >
> > Write a script that clones all trees, compares HEAD against what is
> > is Linux Next (Next/Trees file). If it diverges, add the branch name to
> > the tree entry in MAINTAINERS.
> 
> That script is how I found these: about 160 T: lines diverge.  libata
> isn't a one-off; it's in the first small batch of the plan I described
> in the ext4 thread [1].
> 
> I'm sending them in small batches rather than all at once because I'd
> invariably get something wrong, or miss a subsystem's conventions, and
> I'd rather learn that from a handful of maintainers than from 160.
> Exhibit A is the KVM/arm64 reply [2]: that for-next is purely an
> integration branch and patches there must be based on a tag from
> Linus's tree, so naming it would have sent people to the wrong place.
> Comparing HEAD with Next/Trees can't tell that apart from a branch like
> yours.  So I now also look at how each branch is built, and I'm holding
> back the ones that look like integration branches, block included,
> until I know how to word them.
> 
> As for the other two: SCSI went out in the same first batch [3], and
> nvme.git isn't in Next/Trees at all, so the script never flagged it.
> Thanks for the offered Acked-by; I'll Cc you when block comes up.

Okay, nice to see that you are actually putting in the effort to fix all
entries and not just one.

I guess I would have know if you sent it as a series, but that would
probably have been worse than sending a single patch per subsystem,
as I am sure many people will have comments.

Perhaps mention that you are doing a cleanup of all entries after the:

---

line after the commit message, with the links that you shared just now.
That way maintainers will know that you are actually doing a longer
work and not just a one-off.


Kind regards,
Niklas

  reply	other threads:[~2026-09-24 13:33 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-24  3:35 [PATCH] MAINTAINERS: name the libata/linux for-next branch Matthias Goergens
2026-09-24  3:52 ` Damien Le Moal
2026-09-24  9:32 ` Niklas Cassel
2026-09-24 10:18   ` Matthias Goergens
2026-09-24 13:33     ` Niklas Cassel [this message]
2026-09-25  5:23   ` [PATCH v2] " Matthias Goergens
2026-09-25  9:18     ` Niklas Cassel

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=arUmrIiHwevxjjqK@ryzen \
    --to=cassel@kernel.org \
    --cc=dlemoal@kernel.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=matthias.goergens@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox