Re: [PATCH RESEND v5 00/24] dmaengine: dw-edma: Add RP/EP local DMA controllers support

From: Serge Semin <fancer.lancer@gmail.com>
To: Bjorn Helgaas <helgaas@kernel.org>
Cc: "Serge Semin" <Sergey.Semin@baikalelectronics.ru>,
	"Gustavo Pimentel" <gustavo.pimentel@synopsys.com>,
	"Vinod Koul" <vkoul@kernel.org>, "Rob Herring" <robh@kernel.org>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>,
	"Jingoo Han" <jingoohan1@gmail.com>,
	"Frank Li" <Frank.Li@nxp.com>,
	"Manivannan Sadhasivam" <manivannan.sadhasivam@linaro.org>,
	"Alexey Malahov" <Alexey.Malahov@baikalelectronics.ru>,
	"Pavel Parkhomenko" <Pavel.Parkhomenko@baikalelectronics.ru>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	linux-pci@vger.kernel.org, dmaengine@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH RESEND v5 00/24] dmaengine: dw-edma: Add RP/EP local DMA controllers support
Date: Wed, 24 Aug 2022 21:00:16 +0300	[thread overview]
Message-ID: <20220824180016.ckkf7jugycyai345@mobilestation> (raw)
In-Reply-To: <20220824163916.GA2784109@bhelgaas>

On Wed, Aug 24, 2022 at 11:39:16AM -0500, Bjorn Helgaas wrote:
> On Mon, Aug 22, 2022 at 09:53:08PM +0300, Serge Semin wrote:
> > This is a final patchset in the series created in the framework of
> > my Baikal-T1 PCIe/eDMA-related work:
> > 
> > [1: Done v5] PCI: dwc: Various fixes and cleanups
> > Link: https://lore.kernel.org/linux-pci/20220624143428.8334-1-Sergey.Semin@baikalelectronics.ru/
> > Merged: kernel 6.0-rc1
> > [2: Done v4] PCI: dwc: Add hw version and dma-ranges support
> > Link: https://lore.kernel.org/linux-pci/20220624143947.8991-1-Sergey.Semin@baikalelectronics.ru
> > Merged: kernel 6.0-rc1
> > [3: In-review v5] PCI: dwc: Add generic resources and Baikal-T1 support
> > Link: https://lore.kernel.org/linux-pci/20220822184701.25246-1-Sergey.Semin@baikalelectronics.ru/
> > [4: Done v5] dmaengine: dw-edma: Add RP/EP local DMA support
> > Link: ---you are looking at it---
> > ...
> 
> > Please note originally this series was self content, but due to Frank
> > being a bit faster in his work submission I had to rebase my patchset onto
> > his one. So now this patchset turns to be dependent on the Frank' work:
> > 
> > Link: https://lore.kernel.org/linux-pci/20220524152159.2370739-1-Frank.Li@nxp.com/
> 

> I think this paragraph is obsolete, since the "Enable designware PCI
> EP EDMA locally" series you reference is already upstream:
> https://git.kernel.org/linus/94d13317bef3

Right.

> 
> What remains are items 3 and 4.
> 

Right.

> 3 is mostly drivers/pci/ and DT bindings (Lorenzo).  4 is mostly
> drivers/dma/dw-edma/ stuff (Gustavo).  I guess Lorenzo and Gustavo can
> figure out where it makes the most sense to merge it.

As I already said to you two months ago:
Link: https://lore.kernel.org/linux-pci/20220617104143.yj2mlnj4twoxoeld@mobilestation/
Neither Gustavo Pimentel nor Jingoo Han have given any sign of
activity during all the time the patches being on review, discussed,
resubmitted while both of them are the DW PCIe and eDMA drivers
maintainers. Moreover none of them have been active in kernel for more
than a year:
Last Gustavo' tag could be tracked at Apr' 2021.
Last Han' tag could be found at Nov' 2020 commit.
So it's very much unluckily they'll just get back any time soon. We
can't wait for their opinion especially seeing that my patches have
been on review for almost half a year.

So as we already agreed with you the best solution would be to merge
this and the patchset #3 via your repo (or Lorenzo' repo if he is back
from his long-term absence). Vinod already acked the DMA-patches of
this series:
Link: https://lore.kernel.org/linux-pci/YtlDivjaXfSEK1Xg@matsya/

So the only thing left to settle is the DT-related part of the
patchset #3. Which I hope will be done before the next kernel merge
window. After it's done I'll ping you and Lorenzo. Ok?

-Sergey

> 
> Bjorn