Merge branch 'upstream-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jgarzik/libata-dev

* 'upstream-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jgarzik/libata-dev:
  sata_fsl,mv,nv: prepare for NCQ command completion update
  ata: Convert pci_table entries to PCI_VDEVICE (if PCI_ANY_ID is used)
  libata: more PCI IDs for jmicron controllers
  ata_piix: fix locking around SIDPR access
  [libata] update blacklist for new hyphenated pattern ranges (v2)
  libata: allow hyphenated pattern ranges
  ata_generic: drop hard coded DMA force logic for CENATEK
  [libata] ahci: Fix warning: comparison between 'enum <anonymous>' and 'enum <anonymous>'
  [libata] add ATA_CMD_DSM to ata_get_cmd_descript
  [libata] Add Samsung PATA controller driver, pata_samsung_cf
  [libata] Add 460EX on-chip SATA driver, sata_dwc_460ex
  libata: reduce blacklist size even more (v2)
  libata: reduce blacklist size (v2)
  libata: glob_match for ata_device_blacklist (v2)
  ahci_platform: Remove unneeded ahci_driver.probe assignment
  ahci_platform: Provide for vendor specific init

1 files changed, 24 insertions, 23 deletions

diff --git a/drivers/ata/sata_mv.c b/drivers/ata/sata_mv.c
index a476cd99b95d..9463c71dd38e 100644
--- a/drivers/ata/sata_mv.c
+++ b/drivers/ata/sata_mv.c
@@ -2716,34 +2716,35 @@ static void mv_err_intr(struct ata_port *ap)
 static void mv_process_crpb_response(struct ata_port *ap,
                 struct mv_crpb *response, unsigned int tag, int ncq_enabled)
 {
+        u8 ata_status;
+        u16 edma_status = le16_to_cpu(response->flags);
         struct ata_queued_cmd *qc = ata_qc_from_tag(ap, tag);
 
-        if (qc) {
-                u8 ata_status;
-                u16 edma_status = le16_to_cpu(response->flags);
-                /*
-                 * edma_status from a response queue entry:
-                 *   LSB is from EDMA_ERR_IRQ_CAUSE (non-NCQ only).
-                 *   MSB is saved ATA status from command completion.
-                 */
-                if (!ncq_enabled) {
-                        u8 err_cause = edma_status & 0xff & ~EDMA_ERR_DEV;
-                        if (err_cause) {
-                                /*
-                                 * Error will be seen/handled by mv_err_intr().
-                                 * So do nothing at all here.
-                                 */
-                                return;
-                        }
-                }
-                ata_status = edma_status >> CRPB_FLAG_STATUS_SHIFT;
-                if (!ac_err_mask(ata_status))
-                        ata_qc_complete(qc);
-                /* else: leave it for mv_err_intr() */
-        } else {
+        if (unlikely(!qc)) {
                 ata_port_printk(ap, KERN_ERR, "%s: no qc for tag=%d\n",
                                 __func__, tag);
+                return;
+        }
+
+        /*
+         * edma_status from a response queue entry:
+         *   LSB is from EDMA_ERR_IRQ_CAUSE (non-NCQ only).
+         *   MSB is saved ATA status from command completion.
+         */
+        if (!ncq_enabled) {
+                u8 err_cause = edma_status & 0xff & ~EDMA_ERR_DEV;
+                if (err_cause) {
+                        /*
+                         * Error will be seen/handled by
+                         * mv_err_intr().  So do nothing at all here.
+                         */
+                        return;
+                }
         }
+        ata_status = edma_status >> CRPB_FLAG_STATUS_SHIFT;
+        if (!ac_err_mask(ata_status))
+                ata_qc_complete(qc);
+        /* else: leave it for mv_err_intr() */
 }
 
 static void mv_process_crpb_entries(struct ata_port *ap, struct mv_port_priv *pp)