[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aWTedhkOclAVvktO@ryzen>
Date: Mon, 12 Jan 2026 12:43:50 +0100
From: Niklas Cassel <cassel@...nel.org>
To: Koichiro Den <den@...inux.co.jp>
Cc: jingoohan1@...il.com, mani@...nel.org, lpieralisi@...nel.org,
kwilczynski@...nel.org, robh@...nel.org, bhelgaas@...gle.com,
vigneshr@...com, s-vadapalli@...com, hongxing.zhu@....com,
l.stach@...gutronix.de, shawnguo@...nel.org, s.hauer@...gutronix.de,
kernel@...gutronix.de, festevam@...il.com, minghuan.Lian@....com,
mingkai.hu@....com, roy.zang@....com, jesper.nilsson@...s.com,
heiko@...ech.de, srikanth.thokala@...el.com,
marek.vasut+renesas@...il.com, yoshihiro.shimoda.uh@...esas.com,
geert+renesas@...der.be, magnus.damm@...il.com,
christian.bruel@...s.st.com, mcoquelin.stm32@...il.com,
alexandre.torgue@...s.st.com, thierry.reding@...il.com,
jonathanh@...dia.com, hayashi.kunihiko@...ionext.com,
mhiramat@...nel.org, kishon@...nel.org, jirislaby@...nel.org,
rongqianfeng@...o.com, 18255117159@....com,
shawn.lin@...k-chips.com, nicolas.frattaroli@...labora.com,
linux.amoon@...il.com, vidyas@...dia.com, Frank.Li@....com,
linux-omap@...r.kernel.org, linux-pci@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
imx@...ts.linux.dev, linuxppc-dev@...ts.ozlabs.org,
linux-arm-kernel@...s.com, linux-rockchip@...ts.infradead.org,
linux-arm-msm@...r.kernel.org, linux-renesas-soc@...r.kernel.org,
linux-stm32@...md-mailman.stormreply.com,
linux-tegra@...r.kernel.org
Subject: Re: [PATCH v5 3/3] PCI: dwc: ep: Support BAR subrange inbound
mapping via Address Match Mode iATU
Hello Koichiro,
On Sat, Jan 10, 2026 at 11:29:06PM +0900, Koichiro Den wrote:
(snip)
> > Your new feature (epc_features->subrange_mapping) in epc_features appears
> > to depend on epc_features->dynamic_inbound_mapping, so it is a shame that
> > we don't have a epc_features->dynamic_inbound_mapping bit, so that this new
> > feature could have depended on that bit.
> >
> > if (epf_bar->use_submap &&
> > !(epc_features->dynamic_inbound_mapping &&
> > epc_features->subrange_mapping))
> > return -EINVAL;
> >
> >
> > I think adding some documentation is a good step.
> >
> > Perhaps we should also introduce a epc_features->dynamic_inbound_mapping bit?
> > Since you are making DWC glue drivers return a mutable EPC features, we could
> > set this bit in the DWC driver after that commit. What do you think?
>
> As you pointed out, support for dynamic_inbound_mapping is needed
> independently of my series. Given that, it would make sense to handle it
> either before this series, or to fold it into the next iteration (=v6) of
> the series if that is preferred.
Please fold it into the next iteration (=v6).
It should be a one liner patch in the DWC driver, at least if you put it
after your "PCI: dwc: Allow glue drivers to return mutable EPC features"
patch.
Thank you for all your efforts on improving the endpoint framework.
Kind regards,
Niklas
Powered by blists - more mailing lists