[SCSI] pm80xx: remove unneeded NULL check
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 9 May 2013 12:48:13 +0000 (15:48 +0300)
committerJames Bottomley <JBottomley@Parallels.com>
Tue, 4 Jun 2013 18:16:18 +0000 (11:16 -0700)
Coccinelle complains about the inconsistent NULL checking on "t".  It
turns out the check isn't needed because we verified that "t" is
non-NULL at the start of the function.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Anand Kumar Santhanam <AnandKumar.Santhanam@pmcs.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/pm8001/pm8001_hwi.c

index a58546f73f0d3538f3632b4db50e8bb2003766a1..5456f5c73593490739c7998c82a192aa5b67d36c 100644 (file)
@@ -3740,7 +3740,7 @@ int pm8001_mpi_task_abort_resp(struct pm8001_hba_info *pm8001_ha, void *piomb)
        pm8001_ccb_task_free(pm8001_ha, t, ccb, tag);
        mb();
 
-       if ((pm8001_dev->id & NCQ_ABORT_ALL_FLAG) && t) {
+       if (pm8001_dev->id & NCQ_ABORT_ALL_FLAG) {
                pm8001_tag_free(pm8001_ha, tag);
                sas_free_task(t);
                /* clear the flag */