lkml.org 
[lkml]   [2020]   [Jan]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 4.14 060/371] crypto: brcm - Fix some set-but-not-used warning
    Date
    From: YueHaibing <yuehaibing@huawei.com>

    [ 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 | 6 +-----
    1 file changed, 1 insertion(+), 5 deletions(-)

    diff --git a/drivers/crypto/bcm/cipher.c b/drivers/crypto/bcm/cipher.c
    index 84422435f39b..279e907590e9 100644
    --- a/drivers/crypto/bcm/cipher.c
    +++ b/drivers/crypto/bcm/cipher.c
    @@ -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);
    --
    2.20.1
    \
     
     \ /
      Last update: 2020-01-16 18:25    [W:4.025 / U:0.024 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site