diff mbox series

[-next] scsi: csiostor: Remove set but not used variable 'pln'

Message ID 1548381439-97080-1-git-send-email-yuehaibing@huawei.com (mailing list archive)
State Mainlined
Commit e18e9dac9cdc9f4d048115cd4b9b0ff40c190555
Headers show
Series [-next] scsi: csiostor: Remove set but not used variable 'pln' | expand

Commit Message

Yue Haibing Jan. 25, 2019, 1:57 a.m. UTC
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/scsi/csiostor/csio_attr.c: In function 'csio_fcoe_free_vnp':
drivers/scsi/csiostor/csio_attr.c:500:21: warning:
 variable 'pln' set but not used [-Wunused-but-set-variable]

It never used since introduction.

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 drivers/scsi/csiostor/csio_attr.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Varun Prakash Jan. 29, 2019, 3:29 p.m. UTC | #1
On Fri, Jan 25, 2019 at 01:57:19AM +0000, YueHaibing wrote:
> Fixes gcc '-Wunused-but-set-variable' warning:
> 
> drivers/scsi/csiostor/csio_attr.c: In function 'csio_fcoe_free_vnp':
> drivers/scsi/csiostor/csio_attr.c:500:21: warning:
>  variable 'pln' set but not used [-Wunused-but-set-variable]
> 
> It never used since introduction.
> 
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
> ---
>  drivers/scsi/csiostor/csio_attr.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/scsi/csiostor/csio_attr.c b/drivers/scsi/csiostor/csio_attr.c
> index 9bd2bd8..200e500 100644
> --- a/drivers/scsi/csiostor/csio_attr.c
> +++ b/drivers/scsi/csiostor/csio_attr.c
> @@ -497,7 +497,6 @@
>  static int
>  csio_fcoe_free_vnp(struct csio_hw *hw, struct csio_lnode *ln)
>  {
> -	struct csio_lnode *pln;
>  	struct csio_mb  *mbp;
>  	struct fw_fcoe_vnp_cmd *rsp;
>  	int ret = 0;
> @@ -514,8 +513,6 @@
>  		goto out;
>  	}
>  
> -	pln = ln->pln;
> -
>  	csio_fcoe_vnp_free_init_mb(ln, mbp, CSIO_MB_DEFAULT_TMO,
>  				   ln->fcf_flowid, ln->vnp_flowid,
>  				   NULL);
> 
>

Acked-by: Varun Prakash <varun@chelsio.com>
Martin K. Petersen Feb. 1, 2019, 11:20 p.m. UTC | #2
YueHaibing,

> Fixes gcc '-Wunused-but-set-variable' warning:
>
> drivers/scsi/csiostor/csio_attr.c: In function 'csio_fcoe_free_vnp':
> drivers/scsi/csiostor/csio_attr.c:500:21: warning:
>  variable 'pln' set but not used [-Wunused-but-set-variable]

Applied to 5.1/scsi-queue. Thanks!
diff mbox series

Patch

diff --git a/drivers/scsi/csiostor/csio_attr.c b/drivers/scsi/csiostor/csio_attr.c
index 9bd2bd8..200e500 100644
--- a/drivers/scsi/csiostor/csio_attr.c
+++ b/drivers/scsi/csiostor/csio_attr.c
@@ -497,7 +497,6 @@ 
 static int
 csio_fcoe_free_vnp(struct csio_hw *hw, struct csio_lnode *ln)
 {
-	struct csio_lnode *pln;
 	struct csio_mb  *mbp;
 	struct fw_fcoe_vnp_cmd *rsp;
 	int ret = 0;
@@ -514,8 +513,6 @@ 
 		goto out;
 	}
 
-	pln = ln->pln;
-
 	csio_fcoe_vnp_free_init_mb(ln, mbp, CSIO_MB_DEFAULT_TMO,
 				   ln->fcf_flowid, ln->vnp_flowid,
 				   NULL);