From: Sergei Shtylyov <sshtylyov@mvista.com> To: Matt Porter <mporter@ti.com> Cc: Tony Lindgren <tony@atomide.com>, Linux DaVinci Kernel List <davinci-linux-open-source@linux.davincidsp.com>, Linux OMAP List <linux-omap@vger.kernel.org>, Russell King <linux@arm.linux.org.uk>, "Cousson, Benoit" <b-cousson@ti.com>, Arnd Bergmann <arnd@arndb.de>, Linux Documentation List <linux-doc@vger.kernel.org>, Vinod Koul <vinod.koul@intel.com>, Linux MMC List <linux-mmc@vger.kernel.org>, Devicetree Discuss <devicetree-discuss@lists.ozlabs.org>, Mark Brown <broonie@opensource.wolfsonmicro.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Rob Herring <rob.herring@calxeda.com>, Grant Likely <grant.likely@secretlab.ca>, Rob Landley <rob@landley.net>, Dan Williams <djbw@fb.com>, Linux SPI Devel List <spi-devel-general@lists.sourceforge.net>, Chris Ball <cjb@laptop.org>, Linux ARM Kernel List <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH v7 01/10] ARM: davinci: move private EDMA API to arm/common Date: Sat, 02 Feb 2013 04:01:37 +0400 [thread overview] Message-ID: <510C5761.7050401@mvista.com> (raw) In-Reply-To: <20130201185959.GQ2244@beef> Hello. On 01-02-2013 22:59, Matt Porter wrote: >>>>> Move mach-davinci/dma.c to common/edma.c so it can be used >>>>> by OMAP (specifically AM33xx) as well. >>>> I think this should rather go to drivers/dma/? >>> No, this is the private EDMA API. It's the analogous thing to >>> the private OMAP dma API that is in plat-omap/dma.c. The actual >>> dmaengine driver is in drivers/dma/edma.c as a wrapper around >>> this...same way OMAP DMA engine conversion is being done. >> Keeps me wondering why we couldn't have the same with CPPI 4.1 when I proposed >> that, instead of waiting indefinitely for TI to convert it to drivers/dma/ >> directly. We could have working MUSB DMA on OMAP-L1x/Sitara all this time... Sigh. > That is a shame. Yeah, I've pointed out that I was doing this exactly > the same way as was acceptable for the OMAP DMA conversion since it was > in RFC. The reasons are sound since in both cases, we have many drivers > to convert that need to continue using the private DMA APIs. In case of CPPI 4.1, we'd only have to convert MUSB DMA driver. Other in-tree CPPI 4.1 having SoCs don't use it for anything but MUSB -- it even is sub-block of their MUSB device, AFAIK (I maybe wrong about Sitaras -- I don't know them well). > -Matt WBR, Sergei
WARNING: multiple messages have this Message-ID (diff)
From: sshtylyov@mvista.com (Sergei Shtylyov) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v7 01/10] ARM: davinci: move private EDMA API to arm/common Date: Sat, 02 Feb 2013 04:01:37 +0400 [thread overview] Message-ID: <510C5761.7050401@mvista.com> (raw) In-Reply-To: <20130201185959.GQ2244@beef> Hello. On 01-02-2013 22:59, Matt Porter wrote: >>>>> Move mach-davinci/dma.c to common/edma.c so it can be used >>>>> by OMAP (specifically AM33xx) as well. >>>> I think this should rather go to drivers/dma/? >>> No, this is the private EDMA API. It's the analogous thing to >>> the private OMAP dma API that is in plat-omap/dma.c. The actual >>> dmaengine driver is in drivers/dma/edma.c as a wrapper around >>> this...same way OMAP DMA engine conversion is being done. >> Keeps me wondering why we couldn't have the same with CPPI 4.1 when I proposed >> that, instead of waiting indefinitely for TI to convert it to drivers/dma/ >> directly. We could have working MUSB DMA on OMAP-L1x/Sitara all this time... Sigh. > That is a shame. Yeah, I've pointed out that I was doing this exactly > the same way as was acceptable for the OMAP DMA conversion since it was > in RFC. The reasons are sound since in both cases, we have many drivers > to convert that need to continue using the private DMA APIs. In case of CPPI 4.1, we'd only have to convert MUSB DMA driver. Other in-tree CPPI 4.1 having SoCs don't use it for anything but MUSB -- it even is sub-block of their MUSB device, AFAIK (I maybe wrong about Sitaras -- I don't know them well). > -Matt WBR, Sergei
next prev parent reply other threads:[~2013-02-02 0:02 UTC|newest] Thread overview: 283+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-02-01 18:22 [PATCH v7 00/10] DMA Engine support for AM33XX Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 01/10] ARM: davinci: move private EDMA API to arm/common Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:41 ` Tony Lindgren 2013-02-01 18:41 ` Tony Lindgren 2013-02-01 18:41 ` Tony Lindgren 2013-02-02 12:49 ` Russell King - ARM Linux 2013-02-02 12:49 ` Russell King - ARM Linux 2013-02-02 12:49 ` Russell King - ARM Linux 2013-02-02 14:44 ` Matt Porter 2013-02-02 14:44 ` Matt Porter 2013-02-02 14:44 ` Matt Porter [not found] ` <5022f635a527470dbd0be932063e9cd2@DFLE72.ent.ti.com> 2013-02-01 18:49 ` Matt Porter 2013-02-01 18:49 ` Matt Porter 2013-02-01 18:49 ` Matt Porter [not found] ` <2077c13e12314dc3adc8e5b653855da0@DFLE72.ent.ti.com> 2013-02-01 18:59 ` Matt Porter 2013-02-01 18:59 ` Matt Porter 2013-02-01 18:59 ` Matt Porter 2013-02-02 0:01 ` Sergei Shtylyov [this message] 2013-02-02 0:01 ` Sergei Shtylyov 2013-02-02 0:01 ` Sergei Shtylyov 2013-02-02 12:45 ` Russell King - ARM Linux 2013-02-02 12:45 ` Russell King - ARM Linux 2013-02-02 12:45 ` Russell King - ARM Linux 2013-02-02 17:27 ` Sergei Shtylyov 2013-02-02 17:27 ` Sergei Shtylyov 2013-02-02 17:27 ` Sergei Shtylyov [not found] ` <e9be6668da8b4372a04687847daa1d8c@DFLE72.ent.ti.com> 2013-02-02 18:07 ` Matt Porter 2013-02-02 18:07 ` Matt Porter 2013-02-02 18:07 ` Matt Porter 2013-02-02 18:16 ` Tony Lindgren 2013-02-02 18:16 ` Tony Lindgren 2013-02-02 18:16 ` Tony Lindgren 2013-02-02 19:48 ` Matt Porter 2013-02-02 19:48 ` Matt Porter 2013-02-02 19:48 ` Matt Porter 2013-02-02 21:02 ` Tony Lindgren 2013-02-02 21:02 ` Tony Lindgren 2013-02-02 21:02 ` Tony Lindgren 2013-02-02 19:06 ` Sergei Shtylyov 2013-02-02 19:06 ` Sergei Shtylyov 2013-02-02 19:06 ` Sergei Shtylyov [not found] ` <3245316d7aa94b2e823f98b69497547d@DLEE74.ent.ti.com> 2013-02-02 19:55 ` Matt Porter 2013-02-02 19:55 ` Matt Porter 2013-02-02 19:55 ` Matt Porter 2013-02-02 20:18 ` Sergei Shtylyov 2013-02-02 20:18 ` Sergei Shtylyov 2013-02-02 20:18 ` Sergei Shtylyov 2013-02-01 19:52 ` Sergei Shtylyov 2013-02-01 19:52 ` Sergei Shtylyov 2013-02-01 19:52 ` Sergei Shtylyov 2013-02-01 18:58 ` Felipe Balbi 2013-02-01 18:58 ` Felipe Balbi 2013-02-01 18:58 ` Felipe Balbi 2013-02-01 18:58 ` Felipe Balbi 2013-02-01 20:49 ` Sergei Shtylyov 2013-02-01 20:49 ` Sergei Shtylyov 2013-02-01 20:49 ` Sergei Shtylyov 2013-02-01 20:49 ` Sergei Shtylyov 2013-02-01 20:56 ` Felipe Balbi 2013-02-01 20:56 ` Felipe Balbi 2013-02-01 20:56 ` Felipe Balbi 2013-02-01 21:30 ` Russell King - ARM Linux 2013-02-01 21:30 ` Russell King - ARM Linux 2013-02-01 21:30 ` Russell King - ARM Linux 2013-02-01 21:30 ` Russell King - ARM Linux 2013-02-02 0:07 ` Sergei Shtylyov 2013-02-02 0:07 ` Sergei Shtylyov 2013-02-02 0:07 ` Sergei Shtylyov 2013-02-02 0:07 ` Sergei Shtylyov 2013-02-02 0:44 ` Russell King - ARM Linux 2013-02-02 0:44 ` Russell King - ARM Linux 2013-02-02 0:44 ` Russell King - ARM Linux 2013-02-02 0:44 ` Russell King - ARM Linux 2013-02-02 2:09 ` Sergei Shtylyov 2013-02-02 2:09 ` Sergei Shtylyov 2013-02-02 2:09 ` Sergei Shtylyov 2013-02-02 10:18 ` Russell King - ARM Linux 2013-02-02 10:18 ` Russell King - ARM Linux 2013-02-02 10:18 ` Russell King - ARM Linux 2013-02-02 10:18 ` Russell King - ARM Linux 2013-02-02 12:17 ` Russell King - ARM Linux 2013-02-02 12:17 ` Russell King - ARM Linux 2013-02-02 12:17 ` Russell King - ARM Linux 2013-02-02 12:17 ` Russell King - ARM Linux 2013-02-02 17:02 ` Sergei Shtylyov 2013-02-02 17:02 ` Sergei Shtylyov 2013-02-02 17:02 ` Sergei Shtylyov 2013-02-02 16:27 ` Sergei Shtylyov 2013-02-02 16:27 ` Sergei Shtylyov 2013-02-02 16:27 ` Sergei Shtylyov 2013-02-02 16:45 ` Russell King - ARM Linux 2013-02-02 16:45 ` Russell King - ARM Linux 2013-02-02 16:45 ` Russell King - ARM Linux 2013-02-02 16:45 ` Russell King - ARM Linux 2013-02-02 17:17 ` Sergei Shtylyov 2013-02-02 17:17 ` Sergei Shtylyov 2013-02-02 17:17 ` Sergei Shtylyov 2013-02-04 14:27 ` Arnd Bergmann 2013-02-04 14:27 ` Arnd Bergmann 2013-02-04 14:27 ` Arnd Bergmann 2013-02-02 0:13 ` Sergei Shtylyov 2013-02-02 0:13 ` Sergei Shtylyov 2013-02-02 0:13 ` Sergei Shtylyov 2013-02-02 0:13 ` Sergei Shtylyov 2013-02-04 15:41 ` Felipe Balbi 2013-02-04 15:41 ` Felipe Balbi 2013-02-04 15:41 ` Felipe Balbi 2013-02-04 15:41 ` Felipe Balbi 2013-02-04 15:45 ` Russell King - ARM Linux 2013-02-04 15:45 ` Russell King - ARM Linux 2013-02-04 15:45 ` Russell King - ARM Linux 2013-02-04 15:45 ` Russell King - ARM Linux 2013-02-04 17:36 ` Sergei Shtylyov 2013-02-04 17:36 ` Sergei Shtylyov 2013-02-04 17:36 ` Sergei Shtylyov 2013-02-04 17:36 ` Sergei Shtylyov 2013-02-04 16:47 ` Felipe Balbi 2013-02-04 16:47 ` Felipe Balbi 2013-02-04 16:47 ` Felipe Balbi 2013-02-04 16:47 ` Felipe Balbi 2013-02-04 17:10 ` Russell King - ARM Linux 2013-02-04 17:10 ` Russell King - ARM Linux 2013-02-04 17:10 ` Russell King - ARM Linux 2013-02-04 17:10 ` Russell King - ARM Linux 2013-02-04 17:54 ` Sergei Shtylyov 2013-02-04 17:54 ` Sergei Shtylyov 2013-02-04 17:54 ` Sergei Shtylyov 2013-02-04 17:54 ` Sergei Shtylyov 2013-02-04 17:02 ` Felipe Balbi 2013-02-04 17:02 ` Felipe Balbi 2013-02-04 17:02 ` Felipe Balbi 2013-02-04 17:02 ` Felipe Balbi 2013-02-04 18:22 ` Sergei Shtylyov 2013-02-04 18:22 ` Sergei Shtylyov 2013-02-04 18:22 ` Sergei Shtylyov 2013-02-04 18:22 ` Sergei Shtylyov 2013-02-04 19:22 ` Cyril Chemparathy 2013-02-04 19:22 ` Cyril Chemparathy 2013-02-04 19:22 ` Cyril Chemparathy 2013-02-04 19:22 ` Cyril Chemparathy 2013-02-04 20:29 ` Linus Walleij 2013-02-04 20:29 ` Linus Walleij 2013-02-04 20:29 ` Linus Walleij 2013-02-04 20:29 ` Linus Walleij 2013-02-04 20:33 ` Mark Brown 2013-02-04 20:33 ` Mark Brown 2013-02-04 20:33 ` Mark Brown 2013-02-04 20:33 ` Mark Brown 2013-02-04 21:11 ` Linus Walleij 2013-02-04 21:11 ` Linus Walleij 2013-02-04 21:11 ` Linus Walleij 2013-02-04 21:11 ` Linus Walleij 2013-02-04 21:47 ` Arnd Bergmann 2013-02-04 21:47 ` Arnd Bergmann 2013-02-04 21:47 ` Arnd Bergmann 2013-02-04 21:47 ` Arnd Bergmann 2013-02-05 12:38 ` Russell King - ARM Linux 2013-02-05 12:38 ` Russell King - ARM Linux 2013-02-05 12:38 ` Russell King - ARM Linux 2013-02-05 12:38 ` Russell King - ARM Linux 2013-02-05 15:37 ` Cyril Chemparathy 2013-02-05 15:37 ` Cyril Chemparathy 2013-02-05 15:37 ` Cyril Chemparathy 2013-02-05 15:37 ` Cyril Chemparathy 2013-02-04 21:54 ` Cyril Chemparathy 2013-02-04 21:54 ` Cyril Chemparathy 2013-02-04 21:54 ` Cyril Chemparathy 2013-02-04 21:54 ` Cyril Chemparathy 2013-02-05 12:41 ` Russell King - ARM Linux 2013-02-05 12:41 ` Russell King - ARM Linux 2013-02-05 12:41 ` Russell King - ARM Linux 2013-02-05 12:41 ` Russell King - ARM Linux 2013-02-05 15:42 ` Cyril Chemparathy 2013-02-05 15:42 ` Cyril Chemparathy 2013-02-05 15:42 ` Cyril Chemparathy 2013-02-05 15:30 ` Linus Walleij 2013-02-05 15:30 ` Linus Walleij 2013-02-05 15:30 ` Linus Walleij 2013-02-05 15:30 ` Linus Walleij 2013-02-05 17:14 ` Russell King - ARM Linux 2013-02-05 17:14 ` Russell King - ARM Linux 2013-02-05 17:14 ` Russell King - ARM Linux 2013-02-05 17:14 ` Russell King - ARM Linux 2013-02-05 18:33 ` Linus Walleij 2013-02-05 18:33 ` Linus Walleij 2013-02-05 18:33 ` Linus Walleij 2013-02-05 18:33 ` Linus Walleij 2013-02-04 22:30 ` Cyril Chemparathy 2013-02-04 22:30 ` Cyril Chemparathy 2013-02-04 22:30 ` Cyril Chemparathy 2013-02-04 22:30 ` Cyril Chemparathy 2013-02-05 16:21 ` Linus Walleij 2013-02-05 16:21 ` Linus Walleij 2013-02-05 16:21 ` Linus Walleij 2013-02-05 16:21 ` Linus Walleij 2013-02-05 16:47 ` Mark Brown 2013-02-05 16:47 ` Mark Brown 2013-02-05 16:47 ` Mark Brown 2013-02-05 16:47 ` Mark Brown 2013-02-05 17:06 ` Russell King - ARM Linux 2013-02-05 17:06 ` Russell King - ARM Linux 2013-02-05 17:06 ` Russell King - ARM Linux 2013-02-05 17:06 ` Russell King - ARM Linux 2013-02-05 17:41 ` Mark Brown 2013-02-05 17:41 ` Mark Brown 2013-02-05 17:41 ` Mark Brown 2013-02-05 17:41 ` Mark Brown 2013-02-05 18:29 ` Linus Walleij 2013-02-05 18:29 ` Linus Walleij 2013-02-05 18:29 ` Linus Walleij 2013-02-05 18:29 ` Linus Walleij 2013-02-05 19:45 ` Cyril Chemparathy 2013-02-05 19:45 ` Cyril Chemparathy 2013-02-05 19:45 ` Cyril Chemparathy 2013-02-05 18:28 ` Tony Lindgren 2013-02-05 18:28 ` Tony Lindgren 2013-02-05 18:28 ` Tony Lindgren 2013-02-05 18:28 ` Tony Lindgren 2013-02-05 22:26 ` Arnd Bergmann 2013-02-05 22:26 ` Arnd Bergmann 2013-02-05 22:26 ` Arnd Bergmann 2013-02-06 7:45 ` Felipe Balbi 2013-02-06 7:45 ` Felipe Balbi 2013-02-06 7:45 ` Felipe Balbi 2013-02-01 23:10 ` Sergei Shtylyov 2013-02-01 23:10 ` Sergei Shtylyov 2013-02-01 23:10 ` Sergei Shtylyov 2013-02-01 23:10 ` Sergei Shtylyov 2013-02-09 16:05 ` Sekhar Nori 2013-02-09 16:05 ` Sekhar Nori 2013-02-09 16:05 ` Sekhar Nori 2013-02-09 20:08 ` Russell King - ARM Linux 2013-02-09 20:08 ` Russell King - ARM Linux 2013-02-09 20:08 ` Russell King - ARM Linux 2013-03-04 22:05 ` Matt Porter 2013-03-04 22:05 ` Matt Porter 2013-03-04 22:05 ` Matt Porter [not found] ` <e92425fefcc04bb4ab739ec8d4e82672@DLEE74.ent.ti.com> 2013-03-04 22:12 ` Matt Porter 2013-03-04 22:12 ` Matt Porter 2013-03-04 22:12 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 02/10] ARM: edma: remove unused transfer controller handlers Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 03/10] ARM: edma: add AM33XX support to the private EDMA API Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 04/10] dmaengine: edma: enable build for AM33XX Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 05/10] dmaengine: edma: Add TI EDMA device tree binding Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:26 ` Matt Porter 2013-02-01 18:26 ` Matt Porter 2013-02-01 18:26 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 06/10] ARM: dts: add AM33XX EDMA support Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 07/10] dmaengine: add dma_request_slave_channel_compat() Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:28 ` Matt Porter 2013-02-01 18:28 ` Matt Porter 2013-02-01 18:28 ` Matt Porter 2013-02-12 16:38 ` Vinod Koul 2013-02-12 16:38 ` Vinod Koul 2013-02-12 16:38 ` Vinod Koul 2013-02-01 18:22 ` [PATCH v7 08/10] spi: omap2-mcspi: convert to dma_request_slave_channel_compat() Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 09/10] spi: omap2-mcspi: add generic DMA request support to the DT binding Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` [PATCH v7 10/10] ARM: dts: add AM33XX SPI DMA support Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:22 ` Matt Porter 2013-02-01 18:32 ` [PATCH v7 00/10] DMA Engine support for AM33XX Matt Porter 2013-02-01 18:32 ` Matt Porter 2013-02-01 18:32 ` Matt Porter
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=510C5761.7050401@mvista.com \ --to=sshtylyov@mvista.com \ --cc=arnd@arndb.de \ --cc=b-cousson@ti.com \ --cc=broonie@opensource.wolfsonmicro.com \ --cc=cjb@laptop.org \ --cc=davinci-linux-open-source@linux.davincidsp.com \ --cc=devicetree-discuss@lists.ozlabs.org \ --cc=djbw@fb.com \ --cc=grant.likely@secretlab.ca \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mmc@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=mporter@ti.com \ --cc=rob.herring@calxeda.com \ --cc=rob@landley.net \ --cc=spi-devel-general@lists.sourceforge.net \ --cc=tony@atomide.com \ --cc=vinod.koul@intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.