Re: [PATCH v12 0/8] Enable designware PCI EP EDMA locally

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 31, 2022 at 9:02 AM Zhi Li <lznuaa@xxxxxxxxx> wrote:
>
> On Wed, May 25, 2022 at 9:41 AM Zhi Li <lznuaa@xxxxxxxxx> wrote:
> >
> > On Wed, May 25, 2022 at 4:23 AM Serge Semin <fancer.lancer@xxxxxxxxx> wrote:
> > >
> > > Hello Vinod
> > >
> > > On Tue, May 24, 2022 at 10:21:51AM -0500, Frank Li wrote:
> > > > Default Designware EDMA just probe remotely at host side.
> > > > This patch allow EDMA driver can probe at EP side.
> > > >
> > > > 1. Clean up patch
> > > >    dmaengine: dw-edma: Detach the private data and chip info structures
> > > >    dmaengine: dw-edma: Remove unused field irq in struct dw_edma_chip
> > > >    dmaengine: dw-edma: Change rg_region to reg_base in struct
> > > >    dmaengine: dw-edma: rename wr(rd)_ch_cnt to ll_wr(rd)_cnt in struct
> > > >
> > > > 2. Enhance EDMA driver to allow prode eDMA at EP side
> > > >    dmaengine: dw-edma: Add support for chip specific flags
> > > >    dmaengine: dw-edma: Add DW_EDMA_CHIP_32BIT_DBI for chip specific
> > > > flags (this patch removed at v11 because dma tree already have fixed
> > > > patch)
> > > >
> > > > 3. Bugs fix at EDMA driver when probe eDMA at EP side
> > > >    dmaengine: dw-edma: Fix programming the source & dest addresses for
> > > > ep
> > > >    dmaengine: dw-edma: Don't rely on the deprecated "direction" member
> > > >
> > > > 4. change pci-epf-test to use EDMA driver to transfer data.
> > > >    PCI: endpoint: Add embedded DMA controller test
> > > >
> > > > 5. Using imx8dxl to do test, but some EP functions still have not
> > > > upstream yet. So below patch show how probe eDMA driver at EP
> > > > controller driver.
> > > > https://lore.kernel.org/linux-pci/20220309120149.GB134091@thinkpad/T/#m979eb506c73ab3cfca2e7a43635ecdaec18d8097
> > >
> > > This series has been on review for over three months now. It has got
> > > several acks, rb and tb tags from me, Manivannan and Kishon (the last
> > > patch in the series). Seeing Gustavo hasn't been active for all that time
> > > at all and hasn't performed any review for more than a year the
> > > probability of getting his attention soon enough is almost zero. Thus
> > > could you please give your acks if you are ok with the series content. Due
> > > to having several more patchsets dependent on this one, Bjorn has agreed
> > > to merge this series in through the PCI tree:
> > > https://lore.kernel.org/linux-pci/20220524155201.GA247821@bhelgaas/
> > > So the only thing we need is your ack tags.
> > >
> > > @Frank. Should there be a new patchset revision could you please add a
> > > request to merge the series in to the PCI tree? I am a bit tired repeating
> > > the same messages each time the new mailing review lap.)
> >
> > The key is to need Vinod to say something
> >
> > Best regards
> > Frank Li.
>
> @Vinod Kou:
>        These patches were well reviewed by Serge Semin,  Bjorn,
> Manivannan Sadhasivam, Kishon Vijay Abraham and tested on 3 platforms.
>        Pending on your opinion because it touch file under /driver/dma/dw_edma/*

@vinod kou:
      friendly ping.

>
> best regards
> Frank Li
>
> >
> > >
> > > -Sergey
> > >
> > > >
> > > > Frank Li (6):
> > > >   dmaengine: dw-edma: Remove unused field irq in struct dw_edma_chip
> > > >   dmaengine: dw-edma: Detach the private data and chip info structures
> > > >   dmaengine: dw-edma: Change rg_region to reg_base in struct
> > > >     dw_edma_chip
> > > >   dmaengine: dw-edma: Rename wr(rd)_ch_cnt to ll_wr(rd)_cnt in struct
> > > >     dw_edma_chip
> > > >   dmaengine: dw-edma: Add support for chip specific flags
> > > >   PCI: endpoint: Enable DMA tests for endpoints with DMA capabilities
> > > >
> > > > Serge Semin (2):
> > > >   dmaengine: dw-edma: Drop dma_slave_config.direction field usage
> > > >   dmaengine: dw-edma: Fix eDMA Rd/Wr-channels and DMA-direction
> > > >     semantics
> > > >
> > > >  drivers/dma/dw-edma/dw-edma-core.c            | 141 +++++++++++-------
> > > >  drivers/dma/dw-edma/dw-edma-core.h            |  31 +---
> > > >  drivers/dma/dw-edma/dw-edma-pcie.c            |  83 +++++------
> > > >  drivers/dma/dw-edma/dw-edma-v0-core.c         |  41 ++---
> > > >  drivers/dma/dw-edma/dw-edma-v0-core.h         |   4 +-
> > > >  drivers/dma/dw-edma/dw-edma-v0-debugfs.c      |  18 +--
> > > >  drivers/dma/dw-edma/dw-edma-v0-debugfs.h      |   8 +-
> > > >  drivers/pci/endpoint/functions/pci-epf-test.c | 112 ++++++++++++--
> > > >  include/linux/dma/edma.h                      |  59 +++++++-
> > > >  9 files changed, 317 insertions(+), 180 deletions(-)
> > > >
> > > > --
> > > > 2.35.1
> > > >



[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux