From: Niklas Cassel <cassel@kernel.org>
To: Matthias Goergens <matthias.goergens@gmail.com>
Cc: Damien Le Moal <dlemoal@kernel.org>, linux-ide@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: name the libata/linux for-next branch
Date: Thu, 24 Sep 2026 11:32:19 +0200 [thread overview]
Message-ID: <arTuIxpEsaYIQXU4@ryzen> (raw)
In-Reply-To: <20260924033545.3624716-1-matthias.goergens@gmail.com>
Hello Matthias,
On Thu, Sep 24, 2026 at 11:35:45AM +0800, Matthias Goergens wrote:
> The T: entry for LIBATA SUBSYSTEM (Serial and Parallel ATA drivers)
> names libata/linux without a branch. The repository's HEAD (master,
> 8d3ae59288f1) is a commit from 2026-08-16 that mainline already
> contains. linux-next pulls for-next from this repository (Next/Trees,
> next-20260923), and on 2026-09-24 that branch carried work not yet in
> mainline. Name the branch so the entry identifies where development
> happens.
I think this could have been written simpler, without references to dates.
Something like:
"
The T: entry for LIBATA SUBSYSTEM (Serial and Parallel ATA drivers)
names libata/linux without a branch. The repository's HEAD pointer
points to branch master, which has no active development.
Active development is on the for-next branch. Name the branch so the
entry identifies where development happens.
"
That said, if you are only sending a patch for libata, I don't see why we
should pick this up.
If I look at some entries for other trees (block, scsi, nvme):
T: git git://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux.git
T: git git://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git
T: git git://git.infradead.org/nvme.git
$ git symbolic-ref refs/remotes/block/HEAD
refs/remotes/block/master
$ git symbolic-ref refs/remotes/scsi/HEAD
refs/remotes/scsi/master
$ git symbolic-ref refs/remotes/nvme/HEAD
refs/remotes/nvme/master
They all point to branch master, which is a copy of Linus's master branch
(often some really copy too).
My point is that, only changing libata seems to make things more inconsistent.
Send a series to Jens fixing all of them, and I would give my Acked-by,
so that he can pick up the patch.
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.
>
> Documentation/process/submitting-patches.rst sends contributors to the
> T: entry to find the tree to prepare patches against, so a branch-less
> entry whose HEAD is already in mainline points them at the wrong tree.
s/points them to the wrong tree/points them to the wrong branch/
Kind regards,
Niklas
next prev parent reply other threads:[~2026-09-24 9:32 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 [this message]
2026-09-24 10:18 ` Matthias Goergens
2026-09-24 13:33 ` Niklas Cassel
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=arTuIxpEsaYIQXU4@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 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.