pm8001: Fix to remove null pointer checks that could never happen

Removal of null pointer checks that could never happen

Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Acked-by: Suresh Thiagarajan <Suresh.Thiagarajan@pmcs.com>
Acked-by: Jack Wang <xjtuwjp@gmail.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

diff --git a/drivers/scsi/pm8001/pm8001_hwi.c b/drivers/scsi/pm8001/pm8001_hwi.c
index 3ef7b50..cc89d18 100644 (file)
--- a/drivers/scsi/pm8001/pm8001_hwi.c
+++ b/drivers/scsi/pm8001/pm8001_hwi.c
@@ -4409,7 +4409,7 @@ static int pm8001_chip_sata_req(struct pm8001_hba_info *pm8001_ha,
 
        /* Check for read log for failed drive and return */
        if (sata_cmd.sata_fis.command == 0x2f) {
-               if (pm8001_ha_dev && ((pm8001_ha_dev->id & NCQ_READ_LOG_FLAG) ||
+               if (((pm8001_ha_dev->id & NCQ_READ_LOG_FLAG) ||
                        (pm8001_ha_dev->id & NCQ_ABORT_ALL_FLAG) ||
                        (pm8001_ha_dev->id & NCQ_2ND_RLE_FLAG))) {
                        struct task_status_struct *ts;