linux-kernel - Re: [PATCH V2] dmaengine: sprd: Support two-stage dma interrupt

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 20 Oct 2022 16:15:04 +0800
From:   Baolin Wang <baolin.wang@...ux.alibaba.com>
To:     Cixi Geng <gengcixi@...il.com>, Vinod Koul <vkoul@...nel.org>
Cc:     orsonzhai@...il.com, zhang.lyra@...il.com,
        dmaengine@...r.kernel.org, linux-kernel@...r.kernel.org,
        Cixi Geng <cixi.geng1@...soc.com>
Subject: Re: [PATCH V2] dmaengine: sprd: Support two-stage dma interrupt



On 10/20/2022 3:33 PM, Cixi Geng wrote:
> Vinod Koul <vkoul@...nel.org> 于2022年10月19日周三 22:07写道：
>>
>> On 04-10-22, 07:49, Cixi Geng wrote:
>>> From: Cixi Geng <cixi.geng1@...soc.com>
>>>
>>> Audio need to request Audio CP global dma interrupt, so Audio CP
>>> DMA should support two-stage interrupt to adapte it.
>>> It will occur interrupt when two-stage dma channel transfer done.
>>
>> The patch looks fine to me but...
>>
>>> diff --git a/include/linux/dma/sprd-dma.h b/include/linux/dma/sprd-dma.h
>>> index d09c6f6f6da5..26de41d6d915 100644
>>> --- a/include/linux/dma/sprd-dma.h
>>> +++ b/include/linux/dma/sprd-dma.h
>>
>>>   enum sprd_dma_int_type {
>>>        SPRD_DMA_NO_INT,
>>> @@ -112,6 +120,10 @@ enum sprd_dma_int_type {
>>>        SPRD_DMA_TRANS_BLK_INT,
>>>        SPRD_DMA_LIST_INT,
>>>        SPRD_DMA_CFGERR_INT,
>>> +     SPRD_DMA_SRC_CHN0_INT,
>>> +     SPRD_DMA_SRC_CHN1_INT,
>>> +     SPRD_DMA_DST_CHN0_INT,
>>> +     SPRD_DMA_DST_CHN1_INT,
>>
>> why is sprd_dma_int_type part of driver interface. sprd_dma_int_type is
>> used only by this driver and should be moved into the driver..

Now we can not move this into dma driver, since we have some drivers in 
the mainline will set the DMA interrupt type, such as spi-sprd.c, 
sprd_serial.c and sprd-pcm-compress.c.

>>
>> Can you change that as well please
> the two-stage interrupts added need more discuss and test,
> anyway, I can create a new patch for the change to move init_type into driver,
>>
>> --
>> ~Vinod