* [PATCH -next] crypto: brcm - Fix some set-but-not-used warning
@ 2018-12-28 3:44 ` YueHaibing
0 siblings, 0 replies; 10+ messages in thread
From: YueHaibing @ 2018-12-28 3:44 UTC (permalink / raw)
To: Herbert Xu, Eric Biggers, Gilad Ben-Yossef, Raveendra Padasalagi,
Stefan Agner
Cc: YueHaibing, linux-crypto, kernel-janitors
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]
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
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 c9393ff..9a66c29 100644
--- a/drivers/crypto/bcm/cipher.c
+++ b/drivers/crypto/bcm/cipher.c
@@ -717,7 +717,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 */
@@ -1675,8 +1675,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;
@@ -1686,8 +1684,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);
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH -next] crypto: brcm - Fix some set-but-not-used warning
@ 2018-12-28 3:44 ` YueHaibing
0 siblings, 0 replies; 10+ messages in thread
From: YueHaibing @ 2018-12-28 3:44 UTC (permalink / raw)
To: Herbert Xu, Eric Biggers, Gilad Ben-Yossef, Raveendra Padasalagi,
Stefan Agner
Cc: YueHaibing, linux-crypto, kernel-janitors
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]
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
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 c9393ff..9a66c29 100644
--- a/drivers/crypto/bcm/cipher.c
+++ b/drivers/crypto/bcm/cipher.c
@@ -717,7 +717,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 */
@@ -1675,8 +1675,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;
@@ -1686,8 +1684,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);
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH -next] crypto: brcm - Fix some set-but-not-used warning
2018-12-28 3:44 ` YueHaibing
@ 2019-01-09 5:47 ` Raveendra Padasalagi
-1 siblings, 0 replies; 10+ messages in thread
From: Raveendra Padasalagi @ 2019-01-09 5:35 UTC (permalink / raw)
To: YueHaibing
Cc: Herbert Xu, Eric Biggers, Gilad Ben-Yossef, Stefan Agner,
linux-crypto, kernel-janitors
Hi YueHaibing,
Please add below fixes tag in the commit message,
Fixes: 9d12ba86f818 ("crypto: brcm - Add Broadcom SPU driver")
Otherwise patch looks good to me.
-Raveendra
On Fri, Dec 28, 2018 at 9:07 AM YueHaibing <yuehaibing@huawei.com> wrote:
>
> 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]
>
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
> ---
> 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 c9393ff..9a66c29 100644
> --- a/drivers/crypto/bcm/cipher.c
> +++ b/drivers/crypto/bcm/cipher.c
> @@ -717,7 +717,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 */
> @@ -1675,8 +1675,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;
> @@ -1686,8 +1684,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);
>
>
>
>
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH -next] crypto: brcm - Fix some set-but-not-used warning
@ 2019-01-09 5:47 ` Raveendra Padasalagi
0 siblings, 0 replies; 10+ messages in thread
From: Raveendra Padasalagi @ 2019-01-09 5:47 UTC (permalink / raw)
To: YueHaibing
Cc: Herbert Xu, Eric Biggers, Gilad Ben-Yossef, Stefan Agner,
linux-crypto, kernel-janitors
Hi YueHaibing,
Please add below fixes tag in the commit message,
Fixes: 9d12ba86f818 ("crypto: brcm - Add Broadcom SPU driver")
Otherwise patch looks good to me.
-Raveendra
On Fri, Dec 28, 2018 at 9:07 AM YueHaibing <yuehaibing@huawei.com> wrote:
>
> 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]
>
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
> ---
> 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 c9393ff..9a66c29 100644
> --- a/drivers/crypto/bcm/cipher.c
> +++ b/drivers/crypto/bcm/cipher.c
> @@ -717,7 +717,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 */
> @@ -1675,8 +1675,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;
> @@ -1686,8 +1684,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);
>
>
>
>
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH -next] crypto: brcm - Fix some set-but-not-used warning
2019-01-09 5:47 ` Raveendra Padasalagi
@ 2019-01-09 5:58 ` YueHaibing
-1 siblings, 0 replies; 10+ messages in thread
From: YueHaibing @ 2019-01-09 5:58 UTC (permalink / raw)
To: Raveendra Padasalagi
Cc: Herbert Xu, Eric Biggers, Gilad Ben-Yossef, Stefan Agner,
linux-crypto, kernel-janitors
On 2019/1/9 13:35, Raveendra Padasalagi wrote:
> Hi YueHaibing,
>
> Please add below fixes tag in the commit message,
> Fixes: 9d12ba86f818 ("crypto: brcm - Add Broadcom SPU driver")
>
> Otherwise patch looks good to me.
Ok, will send v2, Thanks.
>
> -Raveendra
>
> On Fri, Dec 28, 2018 at 9:07 AM YueHaibing <yuehaibing@huawei.com> wrote:
>>
>> 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]
>>
>> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
>> ---
>> 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 c9393ff..9a66c29 100644
>> --- a/drivers/crypto/bcm/cipher.c
>> +++ b/drivers/crypto/bcm/cipher.c
>> @@ -717,7 +717,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 */
>> @@ -1675,8 +1675,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;
>> @@ -1686,8 +1684,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);
>>
>>
>>
>>
>>
>
> .
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH -next] crypto: brcm - Fix some set-but-not-used warning
@ 2019-01-09 5:58 ` YueHaibing
0 siblings, 0 replies; 10+ messages in thread
From: YueHaibing @ 2019-01-09 5:58 UTC (permalink / raw)
To: Raveendra Padasalagi
Cc: Herbert Xu, Eric Biggers, Gilad Ben-Yossef, Stefan Agner,
linux-crypto, kernel-janitors
On 2019/1/9 13:35, Raveendra Padasalagi wrote:
> Hi YueHaibing,
>
> Please add below fixes tag in the commit message,
> Fixes: 9d12ba86f818 ("crypto: brcm - Add Broadcom SPU driver")
>
> Otherwise patch looks good to me.
Ok, will send v2, Thanks.
>
> -Raveendra
>
> On Fri, Dec 28, 2018 at 9:07 AM YueHaibing <yuehaibing@huawei.com> wrote:
>>
>> 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]
>>
>> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
>> ---
>> 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 c9393ff..9a66c29 100644
>> --- a/drivers/crypto/bcm/cipher.c
>> +++ b/drivers/crypto/bcm/cipher.c
>> @@ -717,7 +717,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 */
>> @@ -1675,8 +1675,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;
>> @@ -1686,8 +1684,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);
>>
>>
>>
>>
>>
>
> .
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 -next] crypto: brcm - Fix some set-but-not-used warning
2018-12-28 3:44 ` YueHaibing
@ 2019-01-09 6:11 ` YueHaibing
-1 siblings, 0 replies; 10+ messages in thread
From: YueHaibing @ 2019-01-09 6:11 UTC (permalink / raw)
To: Herbert Xu, Eric Biggers, Gilad Ben-Yossef, Raveendra Padasalagi,
Stefan Agner
Cc: YueHaibing, linux-crypto, kernel-janitors
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>
---
v2: add Fixes tag
---
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 c9393ff..9a66c29 100644
--- a/drivers/crypto/bcm/cipher.c
+++ b/drivers/crypto/bcm/cipher.c
@@ -717,7 +717,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 */
@@ -1675,8 +1675,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;
@@ -1686,8 +1684,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);
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 -next] crypto: brcm - Fix some set-but-not-used warning
@ 2019-01-09 6:11 ` YueHaibing
0 siblings, 0 replies; 10+ messages in thread
From: YueHaibing @ 2019-01-09 6:11 UTC (permalink / raw)
To: Herbert Xu, Eric Biggers, Gilad Ben-Yossef, Raveendra Padasalagi,
Stefan Agner
Cc: YueHaibing, linux-crypto, kernel-janitors
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>
---
v2: add Fixes tag
---
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 c9393ff..9a66c29 100644
--- a/drivers/crypto/bcm/cipher.c
+++ b/drivers/crypto/bcm/cipher.c
@@ -717,7 +717,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 */
@@ -1675,8 +1675,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;
@@ -1686,8 +1684,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);
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2 -next] crypto: brcm - Fix some set-but-not-used warning
2019-01-09 6:11 ` YueHaibing
@ 2019-01-18 10:56 ` Herbert Xu
-1 siblings, 0 replies; 10+ messages in thread
From: Herbert Xu @ 2019-01-18 10:56 UTC (permalink / raw)
To: YueHaibing
Cc: Eric Biggers, Gilad Ben-Yossef, Raveendra Padasalagi,
Stefan Agner, linux-crypto, kernel-janitors
On Wed, Jan 09, 2019 at 06:11:18AM +0000, YueHaibing wrote:
> 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>
> ---
> v2: add Fixes tag
> ---
> drivers/crypto/bcm/cipher.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 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
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 -next] crypto: brcm - Fix some set-but-not-used warning
@ 2019-01-18 10:56 ` Herbert Xu
0 siblings, 0 replies; 10+ messages in thread
From: Herbert Xu @ 2019-01-18 10:56 UTC (permalink / raw)
To: YueHaibing
Cc: Eric Biggers, Gilad Ben-Yossef, Raveendra Padasalagi,
Stefan Agner, linux-crypto, kernel-janitors
On Wed, Jan 09, 2019 at 06:11:18AM +0000, YueHaibing wrote:
> 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>
> ---
> v2: add Fixes tag
> ---
> drivers/crypto/bcm/cipher.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 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
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2019-01-18 10:57 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-28 3:44 [PATCH -next] crypto: brcm - Fix some set-but-not-used warning YueHaibing
2018-12-28 3:44 ` YueHaibing
2019-01-09 5:35 ` Raveendra Padasalagi
2019-01-09 5:47 ` Raveendra Padasalagi
2019-01-09 5:58 ` YueHaibing
2019-01-09 5:58 ` YueHaibing
2019-01-09 6:11 ` [PATCH v2 " YueHaibing
2019-01-09 6:11 ` YueHaibing
2019-01-18 10:56 ` Herbert Xu
2019-01-18 10:56 ` Herbert Xu
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.