diff mbox series

[-next] scsi: bfa: Remove set but not used variable 'fchs'

Message ID 20200418071057.96699-1-yuehaibing@huawei.com (mailing list archive)
State Mainlined
Commit 0745c834f7937bee2fe7de5c500ea4d6aa38ce35
Headers show
Series [-next] scsi: bfa: Remove set but not used variable 'fchs' | expand

Commit Message

YueHaibing April 18, 2020, 7:10 a.m. UTC
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/scsi/bfa/bfa_svc.c: In function 'uf_recv':
drivers/scsi/bfa/bfa_svc.c:5520:17: warning:
 variable 'fchs' set but not used [-Wunused-but-set-variable]
  struct fchs_s *fchs;
                 ^

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

Comments

Martin K. Petersen April 27, 2020, 8:21 p.m. UTC | #1
On Sat, 18 Apr 2020 07:10:57 +0000, YueHaibing wrote:

> Fixes gcc '-Wunused-but-set-variable' warning:
> 
> drivers/scsi/bfa/bfa_svc.c: In function 'uf_recv':
> drivers/scsi/bfa/bfa_svc.c:5520:17: warning:
>  variable 'fchs' set but not used [-Wunused-but-set-variable]
>   struct fchs_s *fchs;
>                  ^
> 
> [...]

Applied to 5.8/scsi-queue, thanks!

[1/1] scsi: bfa: Remove set but not used variable 'fchs'
      https://git.kernel.org/mkp/scsi/c/0745c834f793
diff mbox series

Patch

diff --git a/drivers/scsi/bfa/bfa_svc.c b/drivers/scsi/bfa/bfa_svc.c
index d8a9e40fa257..0b7d2e8f4a66 100644
--- a/drivers/scsi/bfa/bfa_svc.c
+++ b/drivers/scsi/bfa/bfa_svc.c
@@ -5517,7 +5517,6 @@  uf_recv(struct bfa_s *bfa, struct bfi_uf_frm_rcvd_s *m)
 	struct bfa_uf_s *uf = &ufm->uf_list[uf_tag];
 	struct bfa_uf_buf_s *uf_buf;
 	uint8_t *buf;
-	struct fchs_s *fchs;
 
 	uf_buf = (struct bfa_uf_buf_s *)
 			bfa_mem_get_dmabuf_kva(ufm, uf_tag, uf->pb_len);
@@ -5526,8 +5525,6 @@  uf_recv(struct bfa_s *bfa, struct bfi_uf_frm_rcvd_s *m)
 	m->frm_len = be16_to_cpu(m->frm_len);
 	m->xfr_len = be16_to_cpu(m->xfr_len);
 
-	fchs = (struct fchs_s *)uf_buf;
-
 	list_del(&uf->qe);	/* dequeue from posted queue */
 
 	uf->data_ptr = buf;