From: Paolo Bonzini <pbonzini@redhat.com>
Date: Tue, 16 Feb 2016 07:49:15 +0000 (+0100)
Subject: mptsas: fix wrong formula
X-Git-Tag: TizenStudio_2.0_p2.4~27^2~6^2~8^2~182^2~17
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9155b7606a17967b1e056aa5d0433a047f23ae51;p=sdk%2Femulator%2Fqemu.git

mptsas: fix wrong formula

MPI_DOORBELL_WHO_INIT_SHIFT is being repeated twice.  Reported
by Coverity.

Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---

diff --git a/hw/scsi/mptsas.c b/hw/scsi/mptsas.c
index 1ce3226..499c146 100644
--- a/hw/scsi/mptsas.c
+++ b/hw/scsi/mptsas.c
@@ -824,7 +824,7 @@ static uint32_t mptsas_doorbell_read(MPTSASState *s)
 {
     uint32_t ret;
 
-    ret = (s->who_init << MPI_DOORBELL_WHO_INIT_SHIFT) & MPI_DOORBELL_WHO_INIT_SHIFT;
+    ret = (s->who_init << MPI_DOORBELL_WHO_INIT_SHIFT) & MPI_DOORBELL_WHO_INIT_MASK;
     ret |= s->state;
     switch (s->doorbell_state) {
     case DOORBELL_NONE: