Linux ATA/IDE development
 help / color / mirror / Atom feed
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

  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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox