From: Paolo Bonzini <pbonzini@redhat.com>
Date: Fri, 15 Jul 2016 16:42:53 +0000 (+0200)
Subject: megasas: remove useless check for cmd->frame
X-Git-Tag: qemu-xen-4.8.0-rc1~102^2~7
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=8cc46787b5b58f01a11c919c7ff939ed009e27fc;p=qemu-xen.git

megasas: remove useless check for cmd->frame

megasas_enqueue_frame always returns with non-NULL cmd->frame.
Remove the "else" part as it is dead code.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---

diff --git a/hw/scsi/megasas.c b/hw/scsi/megasas.c
index 52a41239cf..e968302fdc 100644
--- a/hw/scsi/megasas.c
+++ b/hw/scsi/megasas.c
@@ -1981,11 +1981,7 @@ static void megasas_handle_frame(MegasasState *s, uint64_t frame_addr,
         break;
     }
     if (frame_status != MFI_STAT_INVALID_STATUS) {
-        if (cmd->frame) {
-            cmd->frame->header.cmd_status = frame_status;
-        } else {
-            megasas_frame_set_cmd_status(s, frame_addr, frame_status);
-        }
+        cmd->frame->header.cmd_status = frame_status;
         megasas_unmap_frame(s, cmd);
         megasas_complete_frame(s, cmd->context);
     }