IB/mad: Eliminate redundant SM class version defines for OPA

and rename class version define to indicate SM rather than SMP or SMI

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>

diff --git a/drivers/infiniband/core/mad.c b/drivers/infiniband/core/mad.c
index 40cbd6bdb73be1f23095387dd95f58c8508227b1..bc2a9c2caa3c0561f8e1f72f3996769afa25c201 100644 (file)
--- a/drivers/infiniband/core/mad.c
+++ b/drivers/infiniband/core/mad.c
@@ -769,7 +769,7 @@ static int handle_outgoing_dr_smp(struct ib_mad_agent_private *mad_agent_priv,
         * If we are at the start of the LID routed part, don't update the
         * hop_ptr or hop_cnt.  See section 14.2.2, Vol 1 IB spec.
         */
-       if (opa && smp->class_version == OPA_SMP_CLASS_VERSION) {
+       if (opa && smp->class_version == OPA_SM_CLASS_VERSION) {
                u32 opa_drslid;
 
                if ((opa_get_smp_direction(opa_smp)
@@ -2167,7 +2167,7 @@ handle_smi(struct ib_mad_port_private *port_priv,
        struct ib_mad_hdr *mad_hdr = (struct ib_mad_hdr *)recv->mad;
 
        if (opa && mad_hdr->base_version == OPA_MGMT_BASE_VERSION &&
-           mad_hdr->class_version == OPA_SMI_CLASS_VERSION)
+           mad_hdr->class_version == OPA_SM_CLASS_VERSION)
                return handle_opa_smi(port_priv, qp_info, wc, port_num, recv,
                                      response);
 

diff --git a/drivers/infiniband/hw/hfi1/mad.c b/drivers/infiniband/hw/hfi1/mad.c
index 9487c9bb892087d4dd3b994576fa2eaec2db7d0e..320e4daa5fc6cdf83000a607e3e29edd55ff8b2f 100644 (file)
--- a/drivers/infiniband/hw/hfi1/mad.c
+++ b/drivers/infiniband/hw/hfi1/mad.c
@@ -128,7 +128,7 @@ static void send_trap(struct hfi1_ibport *ibp, void *data, unsigned len)
        smp = send_buf->mad;
        smp->base_version = OPA_MGMT_BASE_VERSION;
        smp->mgmt_class = IB_MGMT_CLASS_SUBN_LID_ROUTED;
-       smp->class_version = OPA_SMI_CLASS_VERSION;
+       smp->class_version = OPA_SM_CLASS_VERSION;
        smp->method = IB_MGMT_METHOD_TRAP;
        ibp->rvp.tid++;
        smp->tid = cpu_to_be64(ibp->rvp.tid);
@@ -343,7 +343,7 @@ static int __subn_get_opa_nodeinfo(struct opa_smp *smp, u32 am, u8 *data,
 
        ni->port_guid = cpu_to_be64(dd->pport[pidx].guid);
        ni->base_version = OPA_MGMT_BASE_VERSION;
-       ni->class_version = OPA_SMI_CLASS_VERSION;
+       ni->class_version = OPA_SM_CLASS_VERSION;
        ni->node_type = 1;     /* channel adapter */
        ni->num_ports = ibdev->phys_port_cnt;
        /* This is already in network order */
@@ -379,7 +379,7 @@ static int subn_get_nodeinfo(struct ib_smp *smp, struct ib_device *ibdev,
                nip->port_guid = cpu_to_be64(dd->pport[pidx].guid);
 
        nip->base_version = OPA_MGMT_BASE_VERSION;
-       nip->class_version = OPA_SMI_CLASS_VERSION;
+       nip->class_version = OPA_SM_CLASS_VERSION;
        nip->node_type = 1;     /* channel adapter */
        nip->num_ports = ibdev->phys_port_cnt;
        /* This is already in network order */
@@ -2302,7 +2302,7 @@ static int pma_get_opa_classportinfo(struct opa_pma_mad *pmp,
                pmp->mad_hdr.status |= IB_SMP_INVALID_FIELD;
 
        p->base_version = OPA_MGMT_BASE_VERSION;
-       p->class_version = OPA_SMI_CLASS_VERSION;
+       p->class_version = OPA_SM_CLASS_VERSION;
        /*
         * Expected response time is 4.096 usec. * 2^18 == 1.073741824 sec.
         */
@@ -4022,7 +4022,7 @@ static int process_subn_opa(struct ib_device *ibdev, int mad_flags,
 
        am = be32_to_cpu(smp->attr_mod);
        attr_id = smp->attr_id;
-       if (smp->class_version != OPA_SMI_CLASS_VERSION) {
+       if (smp->class_version != OPA_SM_CLASS_VERSION) {
                smp->status |= IB_SMP_UNSUP_VERSION;
                ret = reply((struct ib_mad_hdr *)smp);
                return ret;
@@ -4232,7 +4232,7 @@ static int process_perf_opa(struct ib_device *ibdev, u8 port,
 
        *out_mad = *in_mad;
 
-       if (pmp->mad_hdr.class_version != OPA_SMI_CLASS_VERSION) {
+       if (pmp->mad_hdr.class_version != OPA_SM_CLASS_VERSION) {
                pmp->mad_hdr.status |= IB_SMP_UNSUP_VERSION;
                return reply((struct ib_mad_hdr *)pmp);
        }

diff --git a/include/rdma/ib_mad.h b/include/rdma/ib_mad.h
index c8a773ffe23b7f53d6a74b486cc37d4c19e949cf..981214b3790cf4226d68bd5010aa87e14506be27 100644 (file)
--- a/include/rdma/ib_mad.h
+++ b/include/rdma/ib_mad.h
@@ -46,7 +46,7 @@
 #define IB_MGMT_BASE_VERSION                   1
 #define OPA_MGMT_BASE_VERSION                  0x80
 
-#define OPA_SMP_CLASS_VERSION                  0x80
+#define OPA_SM_CLASS_VERSION                   0x80
 
 /* Management classes */
 #define IB_MGMT_CLASS_SUBN_LID_ROUTED          0x01

diff --git a/include/rdma/opa_smi.h b/include/rdma/opa_smi.h
index 4a529ef479951b1a8726f5f52fcc82216a6e514e..f7896117936e67bcc0f368f4edbdc39e17893cac 100644 (file)
--- a/include/rdma/opa_smi.h
+++ b/include/rdma/opa_smi.h
@@ -44,8 +44,6 @@
 #define OPA_MAX_SLS                            32
 #define OPA_MAX_SCS                            32
 
-#define OPA_SMI_CLASS_VERSION                  0x80
-
 #define OPA_LID_PERMISSIVE                     cpu_to_be32(0xFFFFFFFF)
 
 struct opa_smp {