crypto: brcm - Fix some set-but-not-used warning
authorYueHaibing <yuehaibing@huawei.com>
Wed, 9 Jan 2019 06:11:18 +0000 (06:11 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 27 Jan 2020 13:46:13 +0000 (14:46 +0100)
[ Upstream commit 707d0cf8f7cff6dfee9197002859912310532c4f ]

Fixes gcc '-Wunused-but-set-variable' warning:

drivers/crypto/bcm/cipher.c: In function 'handle_ahash_req':
drivers/crypto/bcm/cipher.c:720:15: warning:
 variable 'chunk_start' set but not used [-Wunused-but-set-variable]

drivers/crypto/bcm/cipher.c: In function 'spu_rx_callback':
drivers/crypto/bcm/cipher.c:1679:31: warning:
 variable 'areq' set but not used [-Wunused-but-set-variable]

drivers/crypto/bcm/cipher.c:1678:22: warning:
 variable 'ctx' set but not used [-Wunused-but-set-variable]

Fixes: 9d12ba86f818 ("crypto: brcm - Add Broadcom SPU driver")
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Reviewed-by: Raveendra Padasalagi <raveendra.padasalagi@broadcom.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/crypto/bcm/cipher.c

index 84422435f39b4aa42c6f9f8c413047880b5fc3d0..279e907590e9887819dc2f53da3a6d0c63402b08 100644 (file)
@@ -718,7 +718,7 @@ static int handle_ahash_req(struct iproc_reqctx_s *rctx)
         */
        unsigned int new_data_len;
 
-       unsigned int chunk_start = 0;
+       unsigned int __maybe_unused chunk_start = 0;
        u32 db_size;     /* Length of data field, incl gcm and hash padding */
        int pad_len = 0; /* total pad len, including gcm, hash, stat padding */
        u32 data_pad_len = 0;   /* length of GCM/CCM padding */
@@ -1676,8 +1676,6 @@ static void spu_rx_callback(struct mbox_client *cl, void *msg)
        struct spu_hw *spu = &iproc_priv.spu;
        struct brcm_message *mssg = msg;
        struct iproc_reqctx_s *rctx;
-       struct iproc_ctx_s *ctx;
-       struct crypto_async_request *areq;
        int err = 0;
 
        rctx = mssg->ctx;
@@ -1687,8 +1685,6 @@ static void spu_rx_callback(struct mbox_client *cl, void *msg)
                err = -EFAULT;
                goto cb_finish;
        }
-       areq = rctx->parent;
-       ctx = rctx->ctx;
 
        /* process the SPU status */
        err = spu->spu_status_process(rctx->msg_buf.rx_stat);