diff mbox series

crypto: hisilicon - use sgl API to get sgl dma addr and len

Message ID 1572058641-173376-1-git-send-email-wangzhou1@hisilicon.com
State Accepted
Commit f0c8b6a1e1454f1645463e8ffb3e027fc597867c
Headers show
Series crypto: hisilicon - use sgl API to get sgl dma addr and len | expand

Commit Message

Zhou Wang Oct. 26, 2019, 2:57 a.m. UTC
Use sgl API to get sgl dma addr and len, this will help to avoid compile
error in some platforms. So NEED_SG_DMA_LENGTH can be removed here, which
can only be selected by arch code.

Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>

Suggested-by: Herbert Xu <herbert@gondor.apana.org.au>
---
 drivers/crypto/hisilicon/Kconfig | 1 -
 drivers/crypto/hisilicon/sgl.c   | 4 ++--
 2 files changed, 2 insertions(+), 3 deletions(-)

-- 
2.8.1

Comments

Ard Biesheuvel Oct. 26, 2019, 7:32 a.m. UTC | #1
On Sat, 26 Oct 2019 at 05:01, Zhou Wang <wangzhou1@hisilicon.com> wrote:
>

> Use sgl API to get sgl dma addr and len, this will help to avoid compile

> error in some platforms. So NEED_SG_DMA_LENGTH can be removed here, which

> can only be selected by arch code.

>

> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>

> Suggested-by: Herbert Xu <herbert@gondor.apana.org.au>


Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>


> ---

>  drivers/crypto/hisilicon/Kconfig | 1 -

>  drivers/crypto/hisilicon/sgl.c   | 4 ++--

>  2 files changed, 2 insertions(+), 3 deletions(-)

>

> diff --git a/drivers/crypto/hisilicon/Kconfig b/drivers/crypto/hisilicon/Kconfig

> index a71f2bf..82fb810d 100644

> --- a/drivers/crypto/hisilicon/Kconfig

> +++ b/drivers/crypto/hisilicon/Kconfig

> @@ -18,7 +18,6 @@ config CRYPTO_DEV_HISI_QM

>         tristate

>         depends on ARM64 || COMPILE_TEST

>         depends on PCI && PCI_MSI

> -       select NEED_SG_DMA_LENGTH

>         help

>           HiSilicon accelerator engines use a common queue management

>           interface. Specific engine driver may use this module.

> diff --git a/drivers/crypto/hisilicon/sgl.c b/drivers/crypto/hisilicon/sgl.c

> index bf72603..012023c 100644

> --- a/drivers/crypto/hisilicon/sgl.c

> +++ b/drivers/crypto/hisilicon/sgl.c

> @@ -164,8 +164,8 @@ static struct hisi_acc_hw_sgl *acc_get_sgl(struct hisi_acc_sgl_pool *pool,

>  static void sg_map_to_hw_sg(struct scatterlist *sgl,

>                             struct acc_hw_sge *hw_sge)

>  {

> -       hw_sge->buf = sgl->dma_address;

> -       hw_sge->len = cpu_to_le32(sgl->dma_length);

> +       hw_sge->buf = sg_dma_address(sgl);

> +       hw_sge->len = cpu_to_le32(sg_dma_len(sgl));

>  }

>

>  static void inc_hw_sgl_sge(struct hisi_acc_hw_sgl *hw_sgl)

> --

> 2.8.1

>
Herbert Xu Nov. 1, 2019, 6:13 a.m. UTC | #2
On Sat, Oct 26, 2019 at 10:57:21AM +0800, Zhou Wang wrote:
> Use sgl API to get sgl dma addr and len, this will help to avoid compile

> error in some platforms. So NEED_SG_DMA_LENGTH can be removed here, which

> can only be selected by arch code.

> 

> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>

> Suggested-by: Herbert Xu <herbert@gondor.apana.org.au>

> ---

>  drivers/crypto/hisilicon/Kconfig | 1 -

>  drivers/crypto/hisilicon/sgl.c   | 4 ++--

>  2 files changed, 2 insertions(+), 3 deletions(-)


Patch applied.  Thanks.
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
Zhou Wang Nov. 1, 2019, 7:04 a.m. UTC | #3
On 2019/11/1 14:13, Herbert Xu wrote:
> On Sat, Oct 26, 2019 at 10:57:21AM +0800, Zhou Wang wrote:

>> Use sgl API to get sgl dma addr and len, this will help to avoid compile

>> error in some platforms. So NEED_SG_DMA_LENGTH can be removed here, which

>> can only be selected by arch code.

>>

>> Signed-off-by: Zhou Wang <wangzhou1@hisilicon.com>

>> Suggested-by: Herbert Xu <herbert@gondor.apana.org.au>

>> ---

>>  drivers/crypto/hisilicon/Kconfig | 1 -

>>  drivers/crypto/hisilicon/sgl.c   | 4 ++--

>>  2 files changed, 2 insertions(+), 3 deletions(-)

> 

> Patch applied.  Thanks.


Thanks.

>
diff mbox series

Patch

diff --git a/drivers/crypto/hisilicon/Kconfig b/drivers/crypto/hisilicon/Kconfig
index a71f2bf..82fb810d 100644
--- a/drivers/crypto/hisilicon/Kconfig
+++ b/drivers/crypto/hisilicon/Kconfig
@@ -18,7 +18,6 @@  config CRYPTO_DEV_HISI_QM
 	tristate
 	depends on ARM64 || COMPILE_TEST
 	depends on PCI && PCI_MSI
-	select NEED_SG_DMA_LENGTH
 	help
 	  HiSilicon accelerator engines use a common queue management
 	  interface. Specific engine driver may use this module.
diff --git a/drivers/crypto/hisilicon/sgl.c b/drivers/crypto/hisilicon/sgl.c
index bf72603..012023c 100644
--- a/drivers/crypto/hisilicon/sgl.c
+++ b/drivers/crypto/hisilicon/sgl.c
@@ -164,8 +164,8 @@  static struct hisi_acc_hw_sgl *acc_get_sgl(struct hisi_acc_sgl_pool *pool,
 static void sg_map_to_hw_sg(struct scatterlist *sgl,
 			    struct acc_hw_sge *hw_sge)
 {
-	hw_sge->buf = sgl->dma_address;
-	hw_sge->len = cpu_to_le32(sgl->dma_length);
+	hw_sge->buf = sg_dma_address(sgl);
+	hw_sge->len = cpu_to_le32(sg_dma_len(sgl));
 }
 
 static void inc_hw_sgl_sge(struct hisi_acc_hw_sgl *hw_sgl)