[PATCH -next] [SCSI] csiostor: remove unused variable in csio_process_fwevtq_entry()

[PATCH -next] [SCSI] csiostor: remove unused variable in csio_process_fwevtq_entry()

[Wei Yongjun]

From: Wei Yongjun <yongjun_wei@trendmicro.com.cn>

The variable 'data' is initialized but never used
otherwise, so remove the unused variable.

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
---
 drivers/scsi/csiostor/csio_hw.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/scsi/csiostor/csio_hw.c b/drivers/scsi/csiostor/csio_hw.c
index a0b4c89..1936055 100644
--- a/drivers/scsi/csiostor/csio_hw.c
+++ b/drivers/scsi/csiostor/csio_hw.c
@@ -3565,7 +3565,6 @@ csio_process_fwevtq_entry(struct csio_hw *hw, void *wr, uint32_t len,
 			  struct csio_fl_dma_buf *flb, void *priv)
 {
 	__u8 op;
-	__be64 *data;
 	void *msg = NULL;
 	uint32_t msg_len = 0;
 	bool msg_sg = 0;
@@ -3581,8 +3580,6 @@ csio_process_fwevtq_entry(struct csio_hw *hw, void *wr, uint32_t len,
 		msg = (void *) flb;
 		msg_len = flb->totlen;
 		msg_sg = 1;
-
-		data = (__be64 *) msg;
 	} else if (op == CPL_FW6_MSG || op == CPL_FW4_MSG) {
 
 		CSIO_INC_STATS(hw, n_cpl_fw6_msg);
@@ -3590,8 +3587,6 @@ csio_process_fwevtq_entry(struct csio_hw *hw, void *wr, uint32_t len,
 		msg = (void *)((uintptr_t)wr + sizeof(__be64));
 		msg_len = (op == CPL_FW6_MSG) ? sizeof(struct cpl_fw6_msg) :
 			   sizeof(struct cpl_fw4_msg);
-
-		data = (__be64 *) msg;
 	} else {
 		csio_warn(hw, "unexpected CPL %#x on FW event queue\n", op);
 		CSIO_INC_STATS(hw, n_cpl_unexp);

Re: [PATCH -next] [SCSI] csiostor: remove unused variable in csio_process_fwevtq_entry()

[Naresh Kumar Inna]

On 3/15/2013 3:31 PM, Wei Yongjun wrote:
> From: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
> 
> The variable 'data' is initialized but never used
> otherwise, so remove the unused variable.
> 
> Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
> ---
>  drivers/scsi/csiostor/csio_hw.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/drivers/scsi/csiostor/csio_hw.c b/drivers/scsi/csiostor/csio_hw.c
> index a0b4c89..1936055 100644
> --- a/drivers/scsi/csiostor/csio_hw.c
> +++ b/drivers/scsi/csiostor/csio_hw.c
> @@ -3565,7 +3565,6 @@ csio_process_fwevtq_entry(struct csio_hw *hw, void *wr, uint32_t len,
>  			  struct csio_fl_dma_buf *flb, void *priv)
>  {
>  	__u8 op;
> -	__be64 *data;
>  	void *msg = NULL;
>  	uint32_t msg_len = 0;
>  	bool msg_sg = 0;
> @@ -3581,8 +3580,6 @@ csio_process_fwevtq_entry(struct csio_hw *hw, void *wr, uint32_t len,
>  		msg = (void *) flb;
>  		msg_len = flb->totlen;
>  		msg_sg = 1;
> -
> -		data = (__be64 *) msg;
>  	} else if (op == CPL_FW6_MSG || op == CPL_FW4_MSG) {
>  
>  		CSIO_INC_STATS(hw, n_cpl_fw6_msg);
> @@ -3590,8 +3587,6 @@ csio_process_fwevtq_entry(struct csio_hw *hw, void *wr, uint32_t len,
>  		msg = (void *)((uintptr_t)wr + sizeof(__be64));
>  		msg_len = (op == CPL_FW6_MSG) ? sizeof(struct cpl_fw6_msg) :
>  			   sizeof(struct cpl_fw4_msg);
> -
> -		data = (__be64 *) msg;
>  	} else {
>  		csio_warn(hw, "unexpected CPL %#x on FW event queue\n", op);
>  		CSIO_INC_STATS(hw, n_cpl_unexp);
> 
> 

Acked-by: Naresh Kumar Inna <naresh@chelsio.com>

Thanks,
Naresh.