Date:   Fri, 25 Nov 2022 15:58:52 +0300
From:   Serge Semin <fancer.lancer@...il.com>
To:     Lorenzo Pieralisi <lpieralisi@...nel.org>,
        Robin Murphy <robin.murphy@....com>,
        Vinod Koul <vkoul@...nel.org>,
        Manivannan Sadhasivam <mani@...nel.org>
Cc:     Gustavo Pimentel <gustavo.pimentel@...opsys.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        Bjorn Helgaas <bhelgaas@...gle.com>,
        Rob Herring <robh@...nel.org>,
        Serge Semin <Sergey.Semin@...kalelectronics.ru>,
        Jingoo Han <jingoohan1@...il.com>,
        Cai Huoqing <cai.huoqing@...ux.dev>, linux-pci@...r.kernel.org,
        Pavel Parkhomenko <Pavel.Parkhomenko@...kalelectronics.ru>,
        Frank Li <Frank.Li@....com>,
        Alexey Malahov <Alexey.Malahov@...kalelectronics.ru>,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        Krzysztof Wilczyński <kw@...ux.com>
Subject: Re: [PATCH v7 00/20] PCI: dwc: Add generic resources and Baikal-T1
 support

On Wed, Nov 23, 2022 at 04:09:12PM +0100, Lorenzo Pieralisi wrote:
> On Sun, 13 Nov 2022 22:12:41 +0300, Serge Semin wrote:
> > This patchset is a third one 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 v7] PCI: dwc: Add generic resources and Baikal-T1 support
> > Link: ---you are looking at it---
> > [4: Done v6] dmaengine: dw-edma: Add RP/EP local DMA support
> > Link: https://lore.kernel.org/linux-pci/20221107210438.1515-1-Sergey.Semin@baikalelectronics.ru/
> > 
> > [...]
> 
> I think it is time we merged this series - we went through
> several rounds of reviews and it should be ready for
> mainline (in particular wrt using the generic infrastructure
> it puts in place).
> 
> Applied to pci/dwc, thank you.

Thanks. We've finally done that!

Could you please merge the DW eDMA part in too?
Link: https://lore.kernel.org/linux-pci/20221107210438.1515-1-Sergey.Semin@baikalelectronics.ru/
Due to the dependencies we agreed to hold it on for until the last DW
PCIe patchset is merged in. See discussion here:
https://lore.kernel.org/dmaengine/20220616183900.ww7ora37kmve7av2@mobilestation/
and here (Vinod ab-tag):
https://lore.kernel.org/linux-pci/YuKFnjrxnyNa+98X@matsya/
and here (the last thread mentioned the dependencies):
https://lore.kernel.org/linux-pci/20220825112843.4pbh37x6wemsdmmp@mobilestation/

The eDMA series has got all the ab/rb/tb-tags from @Vinod and @Mani side.
The only tiny exception is
[PATCH v6 22/24] dmaengine: dw-edma: Bypass dma-ranges mapping for the local setup
for which I had to drop the tags due to an update per the @Robin request
in the framework of the dma-ranges/DMA-mask discussion:
https://lore.kernel.org/linux-pci/20220927104831.bovlzl74osb4t5d3@mobilestation/
https://lore.kernel.org/linux-pci/20221007224515.sseyabdfa2phcsdz@mobilestation/

I failed to reach @Robin since November 8, 2022. If you are able to
effectively draw his attention so he looked at the updated patch, that
would be great. Other than that I see no barrier to merge the DW eDMA
series in too.

-Serge(y)

> 
> [01/20] dt-bindings: imx6q-pcie: Fix clock names for imx6sx and imx8mq
>         https://git.kernel.org/lpieralisi/pci/c/b8a83e600bdd
> [02/20] dt-bindings: visconti-pcie: Fix interrupts array max constraints
>         https://git.kernel.org/lpieralisi/pci/c/4cf4b9b70ab2
> [03/20] dt-bindings: PCI: dwc: Detach common RP/EP DT bindings
>         https://git.kernel.org/lpieralisi/pci/c/057646a5db2f
> [04/20] dt-bindings: PCI: dwc: Remove bus node from the examples
>         https://git.kernel.org/lpieralisi/pci/c/b9fe9985aee2
> [05/20] dt-bindings: PCI: dwc: Add phys/phy-names common properties
>         https://git.kernel.org/lpieralisi/pci/c/875596361910
> [06/20] dt-bindings: PCI: dwc: Add max-link-speed common property
>         https://git.kernel.org/lpieralisi/pci/c/eaa9d8865287
> [07/20] dt-bindings: PCI: dwc: Apply generic schema for generic device only
>         https://git.kernel.org/lpieralisi/pci/c/f133396e2d00
> [08/20] dt-bindings: PCI: dwc: Add max-functions EP property
>         https://git.kernel.org/lpieralisi/pci/c/12f7936c7a0e
> [09/20] dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties
>         https://git.kernel.org/lpieralisi/pci/c/35486813c41b
> [10/20] dt-bindings: PCI: dwc: Add reg/reg-names common properties
>         https://git.kernel.org/lpieralisi/pci/c/4cc13eedb892
> [11/20] dt-bindings: PCI: dwc: Add clocks/resets common properties
>         https://git.kernel.org/lpieralisi/pci/c/bd9504af9169
> [12/20] dt-bindings: PCI: dwc: Add dma-coherent property
>         https://git.kernel.org/lpieralisi/pci/c/4a8972542a6d
> [13/20] dt-bindings: PCI: dwc: Apply common schema to Rockchip DW PCIe nodes
>         https://git.kernel.org/lpieralisi/pci/c/98b59129cb9f
> [14/20] dt-bindings: PCI: dwc: Add Baikal-T1 PCIe Root Port bindings
>         https://git.kernel.org/lpieralisi/pci/c/ce27c4e61f2d
> [15/20] PCI: dwc: Introduce dma-ranges property support for RC-host
>         https://git.kernel.org/lpieralisi/pci/c/8522e17d4cab
> [16/20] PCI: dwc: Introduce generic controller capabilities interface
>         https://git.kernel.org/lpieralisi/pci/c/7f9e982dc4fc
> [17/20] PCI: dwc: Introduce generic resources getter
>         https://git.kernel.org/lpieralisi/pci/c/ef8c58877fe7
> [18/20] PCI: dwc: Combine iATU detection procedures
>         https://git.kernel.org/lpieralisi/pci/c/9f67ecdd9579
> [19/20] PCI: dwc: Introduce generic platform clocks and resets
>         https://git.kernel.org/lpieralisi/pci/c/ef69f852a978
> [20/20] PCI: dwc: Add Baikal-T1 PCIe controller support
>         https://git.kernel.org/lpieralisi/pci/c/ba6ed462dcf4
> 
> Thanks,
> Lorenzo

Powered by blists - more mailing lists