From: Bjorn Helgaas <helgaas@kernel.org>
To: Jon Mason <jdmason@kudzu.us>
Cc: Tom Rix <trix@redhat.com>,
kishon@ti.com, lpieralisi@kernel.org, kw@linux.com,
bhelgaas@google.com, Frank.Li@nxp.com, linux-pci@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] PCI: endpoint: pci-epf-vntb: reduce several globals to statics
Date: Fri, 12 Aug 2022 14:21:24 -0500 [thread overview]
Message-ID: <20220812192124.GA1678588@bhelgaas> (raw)
In-Reply-To: <YvZgJ4IGEG8levOA@kudzu.us>
On Fri, Aug 12, 2022 at 10:13:59AM -0400, Jon Mason wrote:
> On Tue, Jul 12, 2022 at 03:05:27PM -0500, Bjorn Helgaas wrote:
> > Handled via Jon, I guess?
> >
> > I'm unclear on the future direction of pci-epf-vntb.c. Jon, are you
> > signing up to maintain this? MAINTAINERS doesn't seem to reflect
> > that, even in next-20220712, so you're not being copied on everything.
> >
> > If you are planning to merge and maintain this file, it would be
> > helpful to me if you acknowledge patches you merge so I know to ignore
> > them.
>
> I massively dropped the ball on all the EPF stuff. I appologize profusely.
No worries, sounds like you're getting everything sorted out :)
> I'm pulling it into my ntb tree bcause of the patch dependencies. If
> you want me to own this stuff because it has ntb in it, then I can do
> a matainers entry to reflect it. My assumption is that because it is
> under the drivers/pci umbrella it is yours (unless you want me to own
> it). 100% defer to your decision.
This patch dependency thing feels like a one-time or at least unusual
situation. Unless it becomes a problem, I think it makes sense to
keep the drivers/pci umbrella instead of carving bits out of the
middle.
Even if I continue to merge everything under drivers/pci, maybe we
should consider an update like this just so you get cc'd on updates to
these files?
diff --git a/MAINTAINERS b/MAINTAINERS
index 08a5d70ceef9..5bafe7e8c2b2 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14466,6 +14466,7 @@ W: https://github.com/jonmason/ntb/wiki
T: git git://github.com/jonmason/ntb.git
F: drivers/net/ntb_netdev.c
F: drivers/ntb/
+F: drivers/pci/endpoint/functions/pci-epf-*ntb.c
F: include/linux/ntb.h
F: include/linux/ntb_transport.h
F: tools/testing/selftests/ntb/
next prev parent reply other threads:[~2022-08-12 19:21 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-04 13:25 [PATCH v3] PCI: endpoint: pci-epf-vntb: reduce several globals to statics Tom Rix
2022-07-05 15:14 ` [EXT] " Frank Li
2022-07-12 20:05 ` Bjorn Helgaas
2022-08-12 14:13 ` Jon Mason
2022-08-12 19:21 ` Bjorn Helgaas [this message]
2022-08-12 19:28 ` Jon Mason
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=20220812192124.GA1678588@bhelgaas \
--to=helgaas@kernel.org \
--cc=Frank.Li@nxp.com \
--cc=bhelgaas@google.com \
--cc=jdmason@kudzu.us \
--cc=kishon@ti.com \
--cc=kw@linux.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=trix@redhat.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.