public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Niklas Cassel <cassel@kernel.org>
To: Koichiro Den <den@valinux.co.jp>
Cc: Manivannan Sadhasivam <mani@kernel.org>,
	bhelgaas@google.com, kwilczynski@kernel.org, frank.li@nxp.com,
	linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v8 0/5] PCI: endpoint: BAR subrange mapping support
Date: Thu, 22 Jan 2026 09:45:13 +0100	[thread overview]
Message-ID: <aXHfUyd0EPFnoVBO@ryzen> (raw)
In-Reply-To: <waapztvy6jyjqtfcoo3rbgvagi4z3p5afw6x2acgf5bxatcui6@nkodhtqqtetr>

Hello Koichiro,

On Thu, Jan 22, 2026 at 10:52:31AM +0900, Koichiro Den wrote:
> > 
> > In this series you didn't add a consumer of this subrange mapping, which I
> > believe is the epf-vntb driver [1]. Even so I don't see a justification of why
> > the BAR needs to be split into multiple subranges. Can you explain that?
> 
> Yes, the first consumer I have in mind is epf-vntb / remote eDMA-backed NTB
> work referenced in [1].

Another consumer in your RFC series is pci-epf-test, specifically:
[RFC PATCH v4 35/38] PCI: endpoint: pci-epf-test: Add pci_epf_test_next_free_bar() helper
[RFC PATCH v4 36/38] PCI: endpoint: pci-epf-test: Add remote eDMA-backed mode
[RFC PATCH v4 37/38] misc: pci_endpoint_test: Add remote eDMA transfer test mode
[RFC PATCH v4 38/38] selftests: pci_endpoint: Add remote eDMA transfer coverage

where the pci-epf-test exports the eDMA registers (using subranges) over a
BAR, and then in the host side driver, calls dw_edma_probe() on this subrange
and uses the host side driver to control the eDMA residing in the endpoint.

Most of your patches in [RFC PATCH v4 35/38] have NTB prefix.

Would it not be possible to simply include patches 35-38 in this series,
so we actually have a consumer, or do you need some other patches as well?

E.g., perhaps
[RFC PATCH v4 01/38] dmaengine: dw-edma: Export helper to get integrated register window

Would also be required?



Kind regards,
Niklas

  parent reply	other threads:[~2026-01-22  8:45 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-15  8:49 [PATCH v8 0/5] PCI: endpoint: BAR subrange mapping support Koichiro Den
2026-01-15  8:49 ` [PATCH v8 1/5] PCI: endpoint: Add dynamic_inbound_mapping EPC feature Koichiro Den
2026-01-15  8:49 ` [PATCH v8 2/5] PCI: endpoint: Add BAR subrange mapping support Koichiro Den
2026-01-15 14:52   ` Frank Li
2026-01-15 15:21     ` Niklas Cassel
2026-01-15 19:44       ` Frank Li
2026-01-19  8:42     ` Koichiro Den
2026-01-15  8:49 ` [PATCH v8 3/5] PCI: dwc: Advertise dynamic inbound " Koichiro Den
2026-01-15 14:54   ` Frank Li
2026-01-15  8:49 ` [PATCH v8 4/5] PCI: dwc: ep: Support BAR subrange inbound mapping via Address Match Mode iATU Koichiro Den
2026-01-15 15:22   ` Frank Li
2026-01-15  8:49 ` [PATCH v8 5/5] Documentation: PCI: endpoint: Clarify pci_epc_set_bar() usage Koichiro Den
2026-01-15 15:23   ` Frank Li
2026-01-19 13:00 ` [PATCH v8 0/5] PCI: endpoint: BAR subrange mapping support Koichiro Den
2026-01-21 15:38   ` Manivannan Sadhasivam
2026-01-22  1:52     ` Koichiro Den
2026-01-22  6:46       ` Manivannan Sadhasivam
2026-01-22  8:45       ` Niklas Cassel [this message]
2026-01-22 14:02         ` Koichiro Den
2026-01-22 15:17           ` Niklas Cassel
2026-01-23  0:36             ` Koichiro Den
2026-01-23  7:11               ` Koichiro Den
2026-01-23  8:35                 ` Niklas Cassel
2026-01-23 10:16                   ` Niklas Cassel
2026-01-23 14:08                   ` Koichiro Den
2026-01-23 14:57                     ` Niklas Cassel
2026-01-26 14:26                       ` Koichiro Den
2026-01-26 19:08                         ` Niklas Cassel
2026-01-23 15:26                   ` 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=aXHfUyd0EPFnoVBO@ryzen \
    --to=cassel@kernel.org \
    --cc=bhelgaas@google.com \
    --cc=den@valinux.co.jp \
    --cc=frank.li@nxp.com \
    --cc=kwilczynski@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=mani@kernel.org \
    /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