RE: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
> -Original Message- > From: andriy.shevche...@linux.intel.com > > Sent: 19 October 2020 7:39 PM > To: Sia, Jee Heng > Cc: Eugeniy Paltsev ; > dmaeng...@vger.kernel.org; linux-kernel@vger.kernel.org; vk...@kernel.org; > Alexey Brodkin > Subject: Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay > AxiDMA > > On Mon, Oct 19, 2020 at 01:22:03AM +, Sia, Jee Heng wrote: > > > From: Eugeniy Paltsev > > > Sent: 16 October 2020 10:51 PM > > > > Hi Sia, > > > > > > Is this patch series available in some public git repo? > > [>>] We do not have public git repo, but the patch series are tested > > on kernel v5.9 > > Sia, can you fork a kernel repository on GitHub or GitLab and create there a > branch with this series based on v5.9? [>>] Thanks Andy to help to create branch at https://gitlab.com/andy-shev/next/-/tree/topic/dw-dma-axi. Eugeniy, you can start to use this branch and I shall learn to create a public repo. Do let me know if you need anything else. > > > > I want to test it on our HW with DW AXI DMAC. > > Eugeniy, to be honest, it's not a big deal to create one either with help of > lore.kernel.org or patchwork [1]. > > For your convenience (disclaimer, I can't guarantee I haven't missed something > here) I published it here [2]. Note, I didn't compile it. > > [1]: https://patchwork.kernel.org/project/linux- > dmaengine/cover/20201012042200.29787-1-jee.heng@intel.com/ > [2]: https://gitlab.com/andy-shev/next/-/tree/topic/dw-dma-axi > > -- > With Best Regards, > Andy Shevchenko >
Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
On Mon, Oct 19, 2020 at 01:22:03AM +, Sia, Jee Heng wrote: > > From: Eugeniy Paltsev > > Sent: 16 October 2020 10:51 PM > > Hi Sia, > > > > Is this patch series available in some public git repo? > [>>] We do not have public git repo, but the patch series are tested on > kernel v5.9 Sia, can you fork a kernel repository on GitHub or GitLab and create there a branch with this series based on v5.9? > > I want to test it on our HW with DW AXI DMAC. Eugeniy, to be honest, it's not a big deal to create one either with help of lore.kernel.org or patchwork [1]. For your convenience (disclaimer, I can't guarantee I haven't missed something here) I published it here [2]. Note, I didn't compile it. [1]: https://patchwork.kernel.org/project/linux-dmaengine/cover/20201012042200.29787-1-jee.heng@intel.com/ [2]: https://gitlab.com/andy-shev/next/-/tree/topic/dw-dma-axi -- With Best Regards, Andy Shevchenko
RE: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
> -Original Message- > From: Eugeniy Paltsev > Sent: 16 October 2020 10:51 PM > To: Sia, Jee Heng > Cc: andriy.shevche...@linux.intel.com; dmaeng...@vger.kernel.org; linux- > ker...@vger.kernel.org; vk...@kernel.org; Alexey Brodkin > > Subject: Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay > AxiDMA > > Hi Sia, > > Is this patch series available in some public git repo? [>>] We do not have public git repo, but the patch series are tested on kernel v5.9 > I want to test it on our HW with DW AXI DMAC. > > Thanks. > --- > Eugeniy Paltsev > > > > From: Sia Jee Heng > Sent: Monday, October 12, 2020 07:21 > To: vk...@kernel.org; Eugeniy Paltsev > Cc: andriy.shevche...@linux.intel.com; dmaeng...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay > AxiDMA > > The below patch series are to support AxiDMA running on Intel KeemBay SoC. > The base driver is dw-axi-dmac but code refactoring is needed, for example: > - Support YAML Schemas DT binding. > - Replacing Linked List with virtual descriptor management. > - Remove unrelated hw desc stuff from dma memory pool. > - Manage dma memory pool alloc/destroy based on channel activity. > - Support dmaengine device_sync() callback. > - Support dmaengine device_config(). > - Support dmaegnine device_prep_slave_sg(). > - Support dmaengine device_prep_dma_cyclic(). > - Support of_dma_controller_register(). > - Support burst residue granularity. > - Support Intel KeemBay AxiDMA registers. > - Support Intel KeemBay AxiDMA device handshake. > - Support Intel KeemBay AxiDMA BYTE and HALFWORD device operation. > - Add constraint to Max segment size. > > This patch set is to replace the patch series submitted at: > https://urldefense.com/v3/__https://lore.kernel.org/dmaengine/1599213094- > 30144-1-git-send-email- > jee.heng@intel.com/__;!!A4F2R9G_pg!Nemc1rSHID2X4d8pr0LNF0nD9Odrn4 > 25GRV8MSTPDvPwE6a3iWPeylAJSaxwqXjfPapMO4U$ > > This patch series are tested on Intel KeemBay platform. > > > Sia Jee Heng (15): > dt-bindings: dma: Add YAML schemas for dw-axi-dmac > dmaengine: dw-axi-dmac: simplify descriptor management > dmaengine: dw-axi-dmac: move dma_pool_create() to > alloc_chan_resources() > dmaengine: dw-axi-dmac: Add device_synchronize() callback > dmaengine: dw-axi-dmac: Add device_config operation > dmaengine: dw-axi-dmac: Support device_prep_slave_sg > dmaegine: dw-axi-dmac: Support device_prep_dma_cyclic() > dmaengine: dw-axi-dmac: Support of_dma_controller_register() > dmaengine: dw-axi-dmac: Support burst residue granularity > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA support > dt-binding: dma: dw-axi-dmac: Add support for Intel KeemBay AxiDMA > dmaengine: dw-axi-dmac: Add Intel KeemBay DMA register fields > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA handshake > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA BYTE and HALFWORD > registers > dmaengine: dw-axi-dmac: Set constraint to the Max segment size > > .../bindings/dma/snps,dw-axi-dmac.txt | 39 - > .../bindings/dma/snps,dw-axi-dmac.yaml| 149 > .../dma/dw-axi-dmac/dw-axi-dmac-platform.c| 696 +++--- > drivers/dma/dw-axi-dmac/dw-axi-dmac.h | 33 +- > 4 files changed, 783 insertions(+), 134 deletions(-) delete mode 100644 > Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.txt > create mode 100644 Documentation/devicetree/bindings/dma/snps,dw-axi- > dmac.yaml > > -- > 2.18.0
Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
Hi Sia, Is this patch series available in some public git repo? I want to test it on our HW with DW AXI DMAC. Thanks. --- Eugeniy Paltsev From: Sia Jee Heng Sent: Monday, October 12, 2020 07:21 To: vk...@kernel.org; Eugeniy Paltsev Cc: andriy.shevche...@linux.intel.com; dmaeng...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA The below patch series are to support AxiDMA running on Intel KeemBay SoC. The base driver is dw-axi-dmac but code refactoring is needed, for example: - Support YAML Schemas DT binding. - Replacing Linked List with virtual descriptor management. - Remove unrelated hw desc stuff from dma memory pool. - Manage dma memory pool alloc/destroy based on channel activity. - Support dmaengine device_sync() callback. - Support dmaengine device_config(). - Support dmaegnine device_prep_slave_sg(). - Support dmaengine device_prep_dma_cyclic(). - Support of_dma_controller_register(). - Support burst residue granularity. - Support Intel KeemBay AxiDMA registers. - Support Intel KeemBay AxiDMA device handshake. - Support Intel KeemBay AxiDMA BYTE and HALFWORD device operation. - Add constraint to Max segment size. This patch set is to replace the patch series submitted at: https://urldefense.com/v3/__https://lore.kernel.org/dmaengine/1599213094-30144-1-git-send-email-jee.heng@intel.com/__;!!A4F2R9G_pg!Nemc1rSHID2X4d8pr0LNF0nD9Odrn425GRV8MSTPDvPwE6a3iWPeylAJSaxwqXjfPapMO4U$ This patch series are tested on Intel KeemBay platform. Sia Jee Heng (15): dt-bindings: dma: Add YAML schemas for dw-axi-dmac dmaengine: dw-axi-dmac: simplify descriptor management dmaengine: dw-axi-dmac: move dma_pool_create() to alloc_chan_resources() dmaengine: dw-axi-dmac: Add device_synchronize() callback dmaengine: dw-axi-dmac: Add device_config operation dmaengine: dw-axi-dmac: Support device_prep_slave_sg dmaegine: dw-axi-dmac: Support device_prep_dma_cyclic() dmaengine: dw-axi-dmac: Support of_dma_controller_register() dmaengine: dw-axi-dmac: Support burst residue granularity dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA support dt-binding: dma: dw-axi-dmac: Add support for Intel KeemBay AxiDMA dmaengine: dw-axi-dmac: Add Intel KeemBay DMA register fields dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA handshake dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA BYTE and HALFWORD registers dmaengine: dw-axi-dmac: Set constraint to the Max segment size .../bindings/dma/snps,dw-axi-dmac.txt | 39 - .../bindings/dma/snps,dw-axi-dmac.yaml| 149 .../dma/dw-axi-dmac/dw-axi-dmac-platform.c| 696 +++--- drivers/dma/dw-axi-dmac/dw-axi-dmac.h | 33 +- 4 files changed, 783 insertions(+), 134 deletions(-) delete mode 100644 Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.txt create mode 100644 Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml -- 2.18.0
Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
On 13-10-20, 07:12, Sia, Jee Heng wrote: > > > > -Original Message- > > From: Vinod Koul > > Sent: 13 October 2020 3:01 PM > > To: Sia, Jee Heng > > Cc: Andy Shevchenko ; > > eugeniy.palt...@synopsys.com; dmaeng...@vger.kernel.org; linux- > > ker...@vger.kernel.org > > Subject: Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay > > AxiDMA > > > > On 13-10-20, 05:49, Sia, Jee Heng wrote: > > > > > > > > > > This patch set is to replace the patch series submitted at: > > > > > https://lore.kernel.org/dmaengine/1599213094-30144-1-git-send-emai > > > > > l-je > > > > > e.heng@intel.com/ > > > > > > > > And it means effectively the bumped version, besides the fact that > > > > you double sent this one... > > > > > > > > > > > > Please fix and resend. Note, now is merge window is open. Depends on > > > > maintainer's flow it may be good or bad time to resend with properly > > > > formed changelog and version of the series. > > > > yeah sorry I wont look at it till merge window closes > [>>] Sorry. > > > > > [>>] Thanks. Will resend the patch set with v1 in the header. > > > > Should this be v1, v1 was the first post, this would be v2! > > > [>>] Thanks for the correction. I meant v2. > > Please use git format-patch -v2 to autogenerate version headers in patches.. > > > > I thought Intel folks had internal review list to take care of these > > things, is it no > > longer used..? > [>>] We are, but I am confuse on what should I do to the existing patch set > sent in > https://lore.kernel.org/dmaengine/1599213094-30144-1-git-send-email-jee.heng@intel.com/. > There is no response from anyone. Sorry my bad looks like I have missed this one, can you please repost this after merge window closes and I will look into it Thanks -- ~Vinod
RE: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
> -Original Message- > From: Vinod Koul > Sent: 13 October 2020 3:01 PM > To: Sia, Jee Heng > Cc: Andy Shevchenko ; > eugeniy.palt...@synopsys.com; dmaeng...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay > AxiDMA > > On 13-10-20, 05:49, Sia, Jee Heng wrote: > > > > > > > > This patch set is to replace the patch series submitted at: > > > > https://lore.kernel.org/dmaengine/1599213094-30144-1-git-send-emai > > > > l-je > > > > e.heng@intel.com/ > > > > > > And it means effectively the bumped version, besides the fact that > > > you double sent this one... > > > > > > > > > Please fix and resend. Note, now is merge window is open. Depends on > > > maintainer's flow it may be good or bad time to resend with properly > > > formed changelog and version of the series. > > yeah sorry I wont look at it till merge window closes [>>] Sorry. > > > [>>] Thanks. Will resend the patch set with v1 in the header. > > Should this be v1, v1 was the first post, this would be v2! > [>>] Thanks for the correction. I meant v2. > Please use git format-patch -v2 to autogenerate version headers in patches.. > > I thought Intel folks had internal review list to take care of these things, > is it no > longer used..? [>>] We are, but I am confuse on what should I do to the existing patch set sent in https://lore.kernel.org/dmaengine/1599213094-30144-1-git-send-email-jee.heng@intel.com/. There is no response from anyone. > > -- > ~Vinod
Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
On 13-10-20, 05:49, Sia, Jee Heng wrote: > > > > > > This patch set is to replace the patch series submitted at: > > > https://lore.kernel.org/dmaengine/1599213094-30144-1-git-send-email-je > > > e.heng@intel.com/ > > > > And it means effectively the bumped version, besides the fact that you > > double > > sent this one... > > > > > > Please fix and resend. Note, now is merge window is open. Depends on > > maintainer's flow it may be good or bad time to resend with properly formed > > changelog and version of the series. yeah sorry I wont look at it till merge window closes > [>>] Thanks. Will resend the patch set with v1 in the header. Should this be v1, v1 was the first post, this would be v2! Please use git format-patch -v2 to autogenerate version headers in patches.. I thought Intel folks had internal review list to take care of these things, is it no longer used..? -- ~Vinod
RE: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
> -Original Message- > From: Andy Shevchenko > Sent: 12 October 2020 9:59 PM > To: Sia, Jee Heng > Cc: vk...@kernel.org; eugeniy.palt...@synopsys.com; > dmaeng...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay > AxiDMA > > On Mon, Oct 12, 2020 at 12:21:45PM +0800, Sia Jee Heng wrote: > > The below patch series are to support AxiDMA running on Intel KeemBay SoC. > > The base driver is dw-axi-dmac but code refactoring is needed, for example: > > - Support YAML Schemas DT binding. > > - Replacing Linked List with virtual descriptor management. > > - Remove unrelated hw desc stuff from dma memory pool. > > - Manage dma memory pool alloc/destroy based on channel activity. > > - Support dmaengine device_sync() callback. > > - Support dmaengine device_config(). > > - Support dmaegnine device_prep_slave_sg(). > > - Support dmaengine device_prep_dma_cyclic(). > > - Support of_dma_controller_register(). > > - Support burst residue granularity. > > - Support Intel KeemBay AxiDMA registers. > > - Support Intel KeemBay AxiDMA device handshake. > > - Support Intel KeemBay AxiDMA BYTE and HALFWORD device operation. > > - Add constraint to Max segment size. > > > > This patch set is to replace the patch series submitted at: > > https://lore.kernel.org/dmaengine/1599213094-30144-1-git-send-email-je > > e.heng@intel.com/ > > And it means effectively the bumped version, besides the fact that you double > sent this one... > > > Please fix and resend. Note, now is merge window is open. Depends on > maintainer's flow it may be good or bad time to resend with properly formed > changelog and version of the series. [>>] Thanks. Will resend the patch set with v1 in the header. > > > This patch series are tested on Intel KeemBay platform. > > > > > > Sia Jee Heng (15): > > dt-bindings: dma: Add YAML schemas for dw-axi-dmac > > dmaengine: dw-axi-dmac: simplify descriptor management > > dmaengine: dw-axi-dmac: move dma_pool_create() to > > alloc_chan_resources() > > dmaengine: dw-axi-dmac: Add device_synchronize() callback > > dmaengine: dw-axi-dmac: Add device_config operation > > dmaengine: dw-axi-dmac: Support device_prep_slave_sg > > dmaegine: dw-axi-dmac: Support device_prep_dma_cyclic() > > dmaengine: dw-axi-dmac: Support of_dma_controller_register() > > dmaengine: dw-axi-dmac: Support burst residue granularity > > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA support > > dt-binding: dma: dw-axi-dmac: Add support for Intel KeemBay AxiDMA > > dmaengine: dw-axi-dmac: Add Intel KeemBay DMA register fields > > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA handshake > > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA BYTE and HALFWORD > > registers > > dmaengine: dw-axi-dmac: Set constraint to the Max segment size > > > > .../bindings/dma/snps,dw-axi-dmac.txt | 39 - > > .../bindings/dma/snps,dw-axi-dmac.yaml| 149 > > .../dma/dw-axi-dmac/dw-axi-dmac-platform.c| 696 +++--- > > drivers/dma/dw-axi-dmac/dw-axi-dmac.h | 33 +- > > 4 files changed, 783 insertions(+), 134 deletions(-) delete mode > > 100644 Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.txt > > create mode 100644 > > Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml > > > > -- > > 2.18.0 > > > > -- > With Best Regards, > Andy Shevchenko >
Re: [PATCH 00/15] dmaengine: dw-axi-dmac: support Intel KeemBay AxiDMA
On Mon, Oct 12, 2020 at 12:21:45PM +0800, Sia Jee Heng wrote: > The below patch series are to support AxiDMA running on Intel KeemBay SoC. > The base driver is dw-axi-dmac but code refactoring is needed, for example: > - Support YAML Schemas DT binding. > - Replacing Linked List with virtual descriptor management. > - Remove unrelated hw desc stuff from dma memory pool. > - Manage dma memory pool alloc/destroy based on channel activity. > - Support dmaengine device_sync() callback. > - Support dmaengine device_config(). > - Support dmaegnine device_prep_slave_sg(). > - Support dmaengine device_prep_dma_cyclic(). > - Support of_dma_controller_register(). > - Support burst residue granularity. > - Support Intel KeemBay AxiDMA registers. > - Support Intel KeemBay AxiDMA device handshake. > - Support Intel KeemBay AxiDMA BYTE and HALFWORD device operation. > - Add constraint to Max segment size. > > This patch set is to replace the patch series submitted at: > https://lore.kernel.org/dmaengine/1599213094-30144-1-git-send-email-jee.heng@intel.com/ And it means effectively the bumped version, besides the fact that you double sent this one... Please fix and resend. Note, now is merge window is open. Depends on maintainer's flow it may be good or bad time to resend with properly formed changelog and version of the series. > This patch series are tested on Intel KeemBay platform. > > > Sia Jee Heng (15): > dt-bindings: dma: Add YAML schemas for dw-axi-dmac > dmaengine: dw-axi-dmac: simplify descriptor management > dmaengine: dw-axi-dmac: move dma_pool_create() to > alloc_chan_resources() > dmaengine: dw-axi-dmac: Add device_synchronize() callback > dmaengine: dw-axi-dmac: Add device_config operation > dmaengine: dw-axi-dmac: Support device_prep_slave_sg > dmaegine: dw-axi-dmac: Support device_prep_dma_cyclic() > dmaengine: dw-axi-dmac: Support of_dma_controller_register() > dmaengine: dw-axi-dmac: Support burst residue granularity > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA support > dt-binding: dma: dw-axi-dmac: Add support for Intel KeemBay AxiDMA > dmaengine: dw-axi-dmac: Add Intel KeemBay DMA register fields > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA handshake > dmaengine: dw-axi-dmac: Add Intel KeemBay AxiDMA BYTE and HALFWORD > registers > dmaengine: dw-axi-dmac: Set constraint to the Max segment size > > .../bindings/dma/snps,dw-axi-dmac.txt | 39 - > .../bindings/dma/snps,dw-axi-dmac.yaml| 149 > .../dma/dw-axi-dmac/dw-axi-dmac-platform.c| 696 +++--- > drivers/dma/dw-axi-dmac/dw-axi-dmac.h | 33 +- > 4 files changed, 783 insertions(+), 134 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.txt > create mode 100644 > Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml > > -- > 2.18.0 > -- With Best Regards, Andy Shevchenko