Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Received-SPF: Pass (protection.outlook.com: domain of cern.ch designates
 188.184.36.50 as permitted sender) receiver=protection.outlook.com;
 client-ip=188.184.36.50; helo=cernmxgwlb4.cern.ch;
From:   Federico Vaga <federico.vaga@cern.ch>
To:     Vinod Koul <vkoul@kernel.org>, <linux-kernel@vger.kernel.org>
Reply-To: <federico.vaga@cern.ch>
CC:     <dmaengine@vger.kernel.org>
Subject: dmaengine: usage of dmaengine_get() and dma_find_channel()
Date:   Mon, 21 Jan 2019 17:41:58 +0100
Message-ID: <23108003.0nNWatJzJB@pcbe13614>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Jan 2019 16:42:02.3321
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: 0895405f-6990-4a31-9f65-08d67fbf5ed4
X-MS-Exchange-CrossTenant-Id: c80d3499-4a40-4a8c-986e-abce017d6b19
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=c80d3499-4a40-4a8c-986e-abce017d6b19;Ip=[188.184.36.50];Helo=[cernmxgwlb4.cern.ch]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR06MB4009
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hello,

I am a bit puzzle about the proper usage these two functions:

void dmaengine_get(void)
struct dma_chan *dma_find_channel(enum dma_transaction_type tx_type)

Looking into the kernel sources dma_find_channel is not used, despite the fact 
that it looks as one of the suggested mechanism to access a DMA engine. So, I 
cannot find a valid example of its usage.

The same for dmaengine_get(). On the other hand, at least this function is 
used in: drivers/video/fbdev/mx3fb.c. But I believe that its usage is wrong 
because it uses: dmaengine_get() + dma_request_channel(), instead of 
dmaengine_get() + dma_find_channel().

Reading the code my understanding is that dmaengine_get() + dma_find_channel() 
should be used for "public" (not private) DMA engine channels.

It's here where things are getting obscure to me. My understanding is that DMA 
channels which are not private can be used by several drivers at the same 
time. In principle they submit all their transfers and the DMA engine will 
process them and then the DMA engine will notify the drivers about their 
completion.

My understanding is also that the dmaengine subsystem assumes that "public" 
channels of the same type are interchangeable.

In my use case I wrote a DMA engine which is specific for a subset of driver, 
and it cannot be used by others (so is sharable among a group of drivers). All 
these drivers, potentially, they can submit their transfers to the DMA engine 
in parallel; but I cannot do it because I'm forced to use 
dma_request_channel() in order to be able to filter the DMA channels, and this 
will make automatically the channel PRIVATE which prevents other drivers to 
use the same channel.

With the current API I do not see alternative for me. I have to use 
dma_request_channel() in order to filter DMA channels. But I am wondering if 
it make sense for you, and if it does, is there a solution today? Or, is it 
possibile to design one?

Thanks (I do not know if this message is clear enough)