crypto: crypto4xx - get rid of redundant using_sd variable
authorChristian Lamparter <chunkeey@gmail.com>
Mon, 22 Apr 2019 11:26:01 +0000 (13:26 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 3 May 2019 06:03:29 +0000 (14:03 +0800)
using_sd is used as a stand-in for sa_command_0.bf.scatter
that we need to set anyway, so we might as well just prevent
double-accounting.

Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/amcc/crypto4xx_core.c
drivers/crypto/amcc/crypto4xx_core.h

index 3e7d24ff3fa6a90b5178501ff1c7d43619458098..3934c2523762c576b2d1e870cc90f22033cd9ae8 100644 (file)
@@ -539,7 +539,7 @@ static void crypto4xx_cipher_done(struct crypto4xx_device *dev,
 
        req = skcipher_request_cast(pd_uinfo->async_req);
 
-       if (pd_uinfo->using_sd) {
+       if (pd_uinfo->sa_va->sa_command_0.bf.scatter) {
                crypto4xx_copy_pkt_to_dst(dev, pd, pd_uinfo,
                                          req->cryptlen, req->dst);
        } else {
@@ -593,7 +593,7 @@ static void crypto4xx_aead_done(struct crypto4xx_device *dev,
        u32 icv[AES_BLOCK_SIZE];
        int err = 0;
 
-       if (pd_uinfo->using_sd) {
+       if (pd_uinfo->sa_va->sa_command_0.bf.scatter) {
                crypto4xx_copy_pkt_to_dst(dev, pd, pd_uinfo,
                                          pd->pd_ctl_len.bf.pkt_len,
                                          dst);
@@ -887,7 +887,6 @@ int crypto4xx_build_pd(struct crypto_async_request *req,
                 * we know application give us dst a whole piece of memory
                 * no need to use scatter ring.
                 */
-               pd_uinfo->using_sd = 0;
                pd_uinfo->first_sd = 0xffffffff;
                sa->sa_command_0.bf.scatter = 0;
                pd->dest = (u32)dma_map_page(dev->core_dev->device,
@@ -901,7 +900,6 @@ int crypto4xx_build_pd(struct crypto_async_request *req,
                u32 sd_idx = fst_sd;
                nbytes = datalen;
                sa->sa_command_0.bf.scatter = 1;
-               pd_uinfo->using_sd = 1;
                pd_uinfo->first_sd = fst_sd;
                sd = crypto4xx_get_sdp(dev, &sd_dma, sd_idx);
                pd->dest = sd_dma;
index 4ecc34fa8ebd0236b5d8108d58c6cfc473c1c4f4..c624f8cd3d2e1375f5b51141c582fe388e7ebb7d 100644 (file)
@@ -64,7 +64,6 @@ union shadow_sa_buf {
 struct pd_uinfo {
        struct crypto4xx_device *dev;
        u32   state;
-       u32 using_sd;
        u32 first_gd;           /* first gather discriptor
                                used by this packet */
        u32 num_gd;             /* number of gather discriptor