From: Vijaya Mohan Guvva <vmo...@brocade.com>

Observed Auto mode in the HBA side while doing manual D-Port test on the
switch side. Mode is not passed to BFA from firmware when the test is
triggered by switch side. BFA just blindly using Auto mode.

Signed-off-by: Vijaya Mohan Guvva <vmo...@brocade.com>
---
 drivers/scsi/bfa/bfa_svc.c | 2 +-
 drivers/scsi/bfa/bfi.h     | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/scsi/bfa/bfa_svc.c b/drivers/scsi/bfa/bfa_svc.c
index 6c41e57..625225f 100644
--- a/drivers/scsi/bfa/bfa_svc.c
+++ b/drivers/scsi/bfa/bfa_svc.c
@@ -6758,7 +6758,7 @@ bfa_dport_scn(struct bfa_dport_s *dport, struct 
bfi_diag_dport_scn_s *msg)
                dport->rp_pwwn = msg->info.teststart.pwwn;
                dport->rp_nwwn = msg->info.teststart.nwwn;
                dport->lpcnt = cpu_to_be32(msg->info.teststart.numfrm);
-               bfa_dport_result_start(dport, BFA_DPORT_OPMODE_AUTO);
+               bfa_dport_result_start(dport, msg->info.teststart.mode);
                break;
 
        case BFI_DPORT_SCN_SUBTESTSTART:
diff --git a/drivers/scsi/bfa/bfi.h b/drivers/scsi/bfa/bfi.h
index 7192e99..1d97a22 100644
--- a/drivers/scsi/bfa/bfi.h
+++ b/drivers/scsi/bfa/bfi.h
@@ -1144,7 +1144,8 @@ struct bfi_diag_dport_scn_teststart_s {
        wwn_t   pwwn;   /* switch port wwn. 8 bytes */
        wwn_t   nwwn;   /* switch node wwn. 8 bytes */
        u8      type;   /* bfa_diag_dport_test_type_e */
-       u8      rsvd[3];
+       u8      mode;   /* bfa_diag_dport_test_opmode */
+       u8      rsvd[2];
        u32     numfrm; /* from switch uint in 1M */
 };
 
-- 
1.8.2.1

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to