From: Andy Shevchenko <andy.shevchenko@gmail.com> To: Peter Ujfalusi <peter.ujfalusi@ti.com> Cc: Vinod Koul <vinod.koul@intel.com>, Arnd Bergmann <arnd@arndb.de>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, dmaengine <dmaengine@vger.kernel.org>, Linux OMAP Mailing List <linux-omap@vger.kernel.org>, linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>, "linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>, Sekhar Nori <nsekhar@ti.com>, linux-spi <linux-spi@vger.kernel.org> Subject: Re: [RFC v02 01/15] dmaengine: core: Allow NULL mask pointer in __dma_device_satisfies_mask() Date: Tue, 1 Dec 2015 14:58:35 +0200 [thread overview] Message-ID: <CAHp75Vfk_GAANRYdbHNxtCy-Ni18FppsKhPPX1X84v1Zu6JXvw@mail.gmail.com> (raw) In-Reply-To: <565D6CB1.8030808@ti.com> On Tue, Dec 1, 2015 at 11:47 AM, Peter Ujfalusi <peter.ujfalusi@ti.com> wrote: > On 11/30/2015 04:35 PM, Andy Shevchenko wrote: >> On Mon, Nov 30, 2015 at 3:45 PM, Peter Ujfalusi <peter.ujfalusi@ti.com> wrote: >>> Treat as true condition the case when the mask is NULL. >> >> What do you think about setting some default (all "on") mask when mask >> is not supplied? > > Probably rephrasing the commit message to say that when the mask is NULL it > means that the caller does not care about the capabilities of the dma device > thus return with true in such a case. > > We could also drop this patch and in private_candidate() : > > - if (!__dma_device_satisfies_mask(dev, mask)) { > + if (mask && !__dma_device_satisfies_mask(dev, mask)) { > pr_debug("%s: wrong capabilities\n", __func__); > return NULL; > } Between patch and above proposal I would choose the latter one. >> I don't know for sure but there might be cases when you don't want >> literally *any* channel to satisfy. > > Or set DMA_SLAVE only in dma_request_chan()? What happens if we have cases > when we are able to request channel for memcpy via dma_request_chan() > (dedicated memcpy channel/DMA engine?) in that case we will have the SLAVE > set, but not MEMCPY, or any other variation we do not know yet? Frankly, have no idea. -- With Best Regards, Andy Shevchenko
WARNING: multiple messages have this Message-ID (diff)
From: andy.shevchenko@gmail.com (Andy Shevchenko) To: linux-arm-kernel@lists.infradead.org Subject: [RFC v02 01/15] dmaengine: core: Allow NULL mask pointer in __dma_device_satisfies_mask() Date: Tue, 1 Dec 2015 14:58:35 +0200 [thread overview] Message-ID: <CAHp75Vfk_GAANRYdbHNxtCy-Ni18FppsKhPPX1X84v1Zu6JXvw@mail.gmail.com> (raw) In-Reply-To: <565D6CB1.8030808@ti.com> On Tue, Dec 1, 2015 at 11:47 AM, Peter Ujfalusi <peter.ujfalusi@ti.com> wrote: > On 11/30/2015 04:35 PM, Andy Shevchenko wrote: >> On Mon, Nov 30, 2015 at 3:45 PM, Peter Ujfalusi <peter.ujfalusi@ti.com> wrote: >>> Treat as true condition the case when the mask is NULL. >> >> What do you think about setting some default (all "on") mask when mask >> is not supplied? > > Probably rephrasing the commit message to say that when the mask is NULL it > means that the caller does not care about the capabilities of the dma device > thus return with true in such a case. > > We could also drop this patch and in private_candidate() : > > - if (!__dma_device_satisfies_mask(dev, mask)) { > + if (mask && !__dma_device_satisfies_mask(dev, mask)) { > pr_debug("%s: wrong capabilities\n", __func__); > return NULL; > } Between patch and above proposal I would choose the latter one. >> I don't know for sure but there might be cases when you don't want >> literally *any* channel to satisfy. > > Or set DMA_SLAVE only in dma_request_chan()? What happens if we have cases > when we are able to request channel for memcpy via dma_request_chan() > (dedicated memcpy channel/DMA engine?) in that case we will have the SLAVE > set, but not MEMCPY, or any other variation we do not know yet? Frankly, have no idea. -- With Best Regards, Andy Shevchenko
next prev parent reply other threads:[~2015-12-01 12:58 UTC|newest] Thread overview: 147+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-30 13:45 [RFC v02 00/15] dmaengine: New 'universal' API for requesting channel Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 01/15] dmaengine: core: Allow NULL mask pointer in __dma_device_satisfies_mask() Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 14:35 ` Andy Shevchenko 2015-11-30 14:35 ` Andy Shevchenko 2015-11-30 14:35 ` Andy Shevchenko 2015-12-01 9:47 ` Peter Ujfalusi 2015-12-01 9:47 ` Peter Ujfalusi 2015-12-01 12:58 ` Andy Shevchenko [this message] 2015-12-01 12:58 ` Andy Shevchenko 2015-12-01 17:06 ` Vinod Koul 2015-12-01 17:06 ` Vinod Koul 2015-11-30 13:45 ` [RFC v02 02/15] dmaengine: core: Move and merge the code paths using private_candidate Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 14:42 ` Andy Shevchenko 2015-11-30 14:42 ` Andy Shevchenko 2015-12-01 8:41 ` Peter Ujfalusi 2015-12-01 8:41 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 03/15] dmaengine: core: Introduce new, universal API to request a channel Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 14:09 ` Arnd Bergmann 2015-11-30 14:09 ` Arnd Bergmann 2015-11-30 14:09 ` Arnd Bergmann 2015-12-01 9:48 ` Peter Ujfalusi 2015-12-01 9:48 ` Peter Ujfalusi 2015-12-01 9:48 ` Peter Ujfalusi 2015-11-30 14:51 ` Andy Shevchenko 2015-11-30 14:51 ` Andy Shevchenko 2015-11-30 14:51 ` Andy Shevchenko 2015-12-01 9:56 ` Peter Ujfalusi 2015-12-01 9:56 ` Peter Ujfalusi 2015-12-01 10:07 ` Andy Shevchenko 2015-12-01 10:07 ` Andy Shevchenko 2015-11-30 15:51 ` Tony Lindgren 2015-11-30 15:51 ` Tony Lindgren 2015-11-30 15:51 ` Tony Lindgren 2015-12-01 8:13 ` Peter Ujfalusi 2015-12-01 8:13 ` Peter Ujfalusi 2015-12-01 8:13 ` Peter Ujfalusi 2015-12-01 17:00 ` Tony Lindgren 2015-12-01 17:00 ` Tony Lindgren 2015-12-02 10:00 ` Peter Ujfalusi 2015-12-02 10:00 ` Peter Ujfalusi 2015-12-02 10:00 ` Peter Ujfalusi 2015-12-02 10:00 ` Peter Ujfalusi 2015-12-02 15:00 ` Tony Lindgren 2015-12-02 15:00 ` Tony Lindgren 2015-12-02 15:00 ` Tony Lindgren 2015-11-30 13:45 ` [RFC v02 04/15] dmaengine: edma: Add support for DMA filter mapping to slave devices Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 14:11 ` Arnd Bergmann 2015-11-30 14:11 ` Arnd Bergmann 2015-12-01 9:58 ` Peter Ujfalusi 2015-12-01 9:58 ` Peter Ujfalusi 2015-12-01 9:58 ` Peter Ujfalusi 2015-12-01 10:14 ` Arnd Bergmann 2015-12-01 10:14 ` Arnd Bergmann 2015-12-01 17:22 ` Vinod Koul 2015-12-01 17:22 ` Vinod Koul 2015-12-01 20:20 ` Arnd Bergmann 2015-12-01 20:20 ` Arnd Bergmann 2015-12-01 20:20 ` Arnd Bergmann 2015-12-02 4:37 ` Vinod Koul 2015-12-02 4:37 ` Vinod Koul 2015-12-02 4:37 ` Vinod Koul 2015-12-02 10:02 ` Peter Ujfalusi 2015-12-02 10:02 ` Peter Ujfalusi 2015-12-02 10:02 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 05/15] ARM: davinci: devices-da8xx: Add dma_filter_map to edma Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 06/15] ARM: davinci: dm355: " Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 07/15] ARM: davinci: dm365: " Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 08/15] ARM: davinci: dm644x: " Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 09/15] ARM: davinci: dm646x: " Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 10/15] mmc: davinci_mmc: Use dma_request_chan() to requesting DMA channel Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 11/15] spi: davinci: " Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2016-05-02 11:04 ` Applied "spi: davinci: Use dma_request_chan() for requesting DMA channel" to the spi tree Mark Brown 2016-05-02 11:04 ` Mark Brown 2015-11-30 13:45 ` [RFC v02 12/15] ARM: davinci: devices-da8xx: Remove DMA resources for MMC and SPI Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 13/15] ARM: davinci: devices: Remove DMA resources for MMC Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 14/15] ARM: davinci: dm355: Remove DMA resources for SPI Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` [RFC v02 15/15] ARM: davinci: dm365: " Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 13:45 ` Peter Ujfalusi 2015-11-30 14:18 ` [RFC v02 00/15] dmaengine: New 'universal' API for requesting channel Arnd Bergmann 2015-11-30 14:18 ` Arnd Bergmann 2015-12-01 10:12 ` Peter Ujfalusi 2015-12-01 10:12 ` Peter Ujfalusi 2015-12-01 10:12 ` Peter Ujfalusi 2015-12-01 10:12 ` Peter Ujfalusi 2015-12-01 10:15 ` Arnd Bergmann 2015-12-01 10:15 ` Arnd Bergmann 2015-12-01 10:15 ` Arnd Bergmann 2015-11-30 14:32 ` Andy Shevchenko 2015-11-30 14:32 ` Andy Shevchenko 2015-12-01 13:45 ` Peter Ujfalusi 2015-12-01 13:45 ` Peter Ujfalusi 2015-12-01 14:24 ` Arnd Bergmann 2015-12-01 14:24 ` Arnd Bergmann 2015-12-02 10:51 ` Peter Ujfalusi 2015-12-02 10:51 ` Peter Ujfalusi 2015-12-02 13:38 ` Arnd Bergmann 2015-12-02 13:38 ` Arnd Bergmann 2015-12-02 13:38 ` Arnd Bergmann 2015-12-01 16:59 ` Vinod Koul 2015-12-01 16:59 ` Vinod Koul 2015-12-01 16:59 ` Vinod Koul 2015-12-01 20:17 ` Arnd Bergmann 2015-12-01 20:17 ` Arnd Bergmann 2015-12-01 20:17 ` Arnd Bergmann 2015-12-02 4:52 ` Vinod Koul 2015-12-02 4:52 ` Vinod Koul 2015-12-02 8:23 ` Arnd Bergmann 2015-12-02 8:23 ` Arnd Bergmann 2015-12-02 12:29 ` Peter Ujfalusi 2015-12-02 12:29 ` Peter Ujfalusi 2015-12-02 12:29 ` Peter Ujfalusi
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=CAHp75Vfk_GAANRYdbHNxtCy-Ni18FppsKhPPX1X84v1Zu6JXvw@mail.gmail.com \ --to=andy.shevchenko@gmail.com \ --cc=arnd@arndb.de \ --cc=dmaengine@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mmc@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux-spi@vger.kernel.org \ --cc=nsekhar@ti.com \ --cc=peter.ujfalusi@ti.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.