Received-SPF: pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH 0/4] Fix the parameter of dma_map_sg()
To:     <clabbe.montjoie@gmail.com>, <clabbe@baylibre.com>,
        <gcherian@marvell.com>, <herbert@gondor.apana.org.au>,
        <davem@davemloft.net>
References: <1612853965-67777-1-git-send-email-chenxiang66@hisilicon.com>
CC:     <linux-crypto@vger.kernel.org>, <linuxarm@openeuler.org>,
        <prime.zeng@huawei.com>
From:   "chenxiang (M)" <chenxiang66@hisilicon.com>
Message-ID: <54b73ba3-54f9-bb73-e398-4f12bc359b26@hisilicon.com>
Date:   Sat, 20 Feb 2021 09:51:17 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.2.0
MIME-Version: 1.0
In-Reply-To: <1612853965-67777-1-git-send-email-chenxiang66@hisilicon.com>
Content-Type: text/plain; charset="gbk"; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

Ping...

?? 2021/2/9 14:59, chenxiang д??:
> From: Xiang Chen <chenxiang66@hisilicon.com>
>
> According to Documentation/core-api/dma-api-howto.rst, the parameters
> of dma_unmap_sg() must be the same as those which are passed in to the
> scatter/gather mapping API.
> But for some drivers under crypto, the <nents> parameter of dma_unmap_sg()
> is number of elements after mapping. So fix them.
>
> Part of the document is as follows:
>
> To unmap a scatterlist, just call::
>
>          dma_unmap_sg(dev, sglist, nents, direction);
> 	
> Again, make sure DMA activity has already finished.
> 	
> 	.. note::
> 	
> 	        The 'nents' argument to the dma_unmap_sg call must be
> 		the _same_ one you passed into the dma_map_sg call,
> 		it should _NOT_ be the 'count' value _returned_ from the
> 		dma_map_sg call.
>
> chenxiang (4):
>    crypto: amlogic - Fix the parameter of dma_unmap_sg()
>    crypto: cavium - Fix the parameter of dma_unmap_sg()
>    crypto: ux500 - Fix the parameter of dma_unmap_sg()
>    crypto: allwinner - Fix the parameter of dma_unmap_sg()
>
>   drivers/crypto/allwinner/sun8i-ce/sun8i-ce-cipher.c | 9 ++++++---
>   drivers/crypto/allwinner/sun8i-ce/sun8i-ce-hash.c   | 3 ++-
>   drivers/crypto/allwinner/sun8i-ss/sun8i-ss-cipher.c | 9 ++++++---
>   drivers/crypto/allwinner/sun8i-ss/sun8i-ss-hash.c   | 3 ++-
>   drivers/crypto/amlogic/amlogic-gxl-cipher.c         | 6 +++---
>   drivers/crypto/cavium/nitrox/nitrox_reqmgr.c        | 8 ++++----
>   drivers/crypto/ux500/cryp/cryp_core.c               | 4 ++--
>   drivers/crypto/ux500/hash/hash_core.c               | 2 +-
>   8 files changed, 26 insertions(+), 18 deletions(-)
>