summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRickard Strandqvist <rickard_strandqvist@spectrumdigital.se>2014-07-09 17:19:38 +0530
committerChristoph Hellwig <hch@lst.de>2014-07-25 17:17:05 -0400
commitd98164461c885ef2655d8edceec4f8f403c0b41e (patch)
tree2309dcd9774a24bfa2a8a1b91a02a7ed4cb2836b
parent5b4ce882d56e5356ea38ab86f6da91df4ac57842 (diff)
downloadlinux-d98164461c885ef2655d8edceec4f8f403c0b41e.tar.gz
linux-d98164461c885ef2655d8edceec4f8f403c0b41e.tar.bz2
linux-d98164461c885ef2655d8edceec4f8f403c0b41e.zip
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>
-rw-r--r--drivers/scsi/pm8001/pm8001_hwi.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/pm8001/pm8001_hwi.c b/drivers/scsi/pm8001/pm8001_hwi.c
index 3ef7b504d8c1..cc89d18e1ae6 100644
--- 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;