summaryrefslogtreecommitdiffstats
path: root/drivers/scsi/bfa
diff options
context:
space:
mode:
authorJulia Lawall <julia@diku.dk>2010-03-11 14:09:55 -0800
committerJames Bottomley <James.Bottomley@suse.de>2010-04-11 09:23:32 -0500
commit4644efabde172808c0a8b6e3f17e4c204a4e52e7 (patch)
treebda7fdf7b3d26a738f12f9f15bd423fce82c5f69 /drivers/scsi/bfa
parentf3d6e1dcd291fd0da3accb0d60fbd0d26d2189ed (diff)
downloadop-kernel-dev-4644efabde172808c0a8b6e3f17e4c204a4e52e7.zip
op-kernel-dev-4644efabde172808c0a8b6e3f17e4c204a4e52e7.tar.gz
[SCSI] bfa: eliminate useless code
The variable bfa_itnim is initialized twice to the same (side effect-free) expression. Drop one initialization. A simplified version of the semantic match that finds this problem is: (http://coccinelle.lip6.fr/) // <smpl> @forall@ idexpression *x; identifier f!=ERR_PTR; @@ x = f(...) ... when != x ( x = f(...,<+...x...+>,...) | * x = f(...) ) // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Acked-by: Jing Huang <huangj@brocade.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/bfa')
-rw-r--r--drivers/scsi/bfa/bfad_im.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/scsi/bfa/bfad_im.c b/drivers/scsi/bfa/bfad_im.c
index 78f42aa..e2c70de 100644
--- a/drivers/scsi/bfa/bfad_im.c
+++ b/drivers/scsi/bfa/bfad_im.c
@@ -252,7 +252,6 @@ bfad_im_target_reset_send(struct bfad_s *bfad, struct scsi_cmnd *cmnd,
struct bfa_itnim_s *bfa_itnim;
bfa_status_t rc = BFA_STATUS_OK;
- bfa_itnim = bfa_fcs_itnim_get_halitn(&itnim->fcs_itnim);
tskim = bfa_tskim_alloc(&bfad->bfa, (struct bfad_tskim_s *) cmnd);
if (!tskim) {
BFA_DEV_PRINTF(bfad, BFA_ERR,
OpenPOWER on IntegriCloud