Merge branch 'drm-nouveau-pony' of git://git.kernel.org/pub/scm/linux/kernel/git...

[linux-2.6-block.git] / drivers / scsi / bfa / fcbuild.c

diff --git a/drivers/scsi/bfa/fcbuild.c b/drivers/scsi/bfa/fcbuild.c
index d174706b9caa913c2a39d928aee7e7132e3708fa..fee5456451cb5a6c1993bff92f1793b8538b7e74 100644 (file)
--- a/drivers/scsi/bfa/fcbuild.c
+++ b/drivers/scsi/bfa/fcbuild.c
@@ -188,14 +188,14 @@ fc_els_rsp_parse(struct fchs_s *fchs, int len)
        switch (els_cmd->els_code) {
        case FC_ELS_LS_RJT:
                if (ls_rjt->reason_code == FC_LS_RJT_RSN_LOGICAL_BUSY)
-                       return (FC_PARSE_BUSY);
+                       return FC_PARSE_BUSY;
                else
-                       return (FC_PARSE_FAILURE);
+                       return FC_PARSE_FAILURE;
 
        case FC_ELS_ACC:
-               return (FC_PARSE_OK);
+               return FC_PARSE_OK;
        }
-       return (FC_PARSE_OK);
+       return FC_PARSE_OK;
 }
 
 static void
@@ -228,7 +228,7 @@ fc_plogi_x_build(struct fchs_s *fchs, void *pld, u32 d_id, u32 s_id,
        bfa_os_memcpy(&plogi->port_name, &port_name, sizeof(wwn_t));
        bfa_os_memcpy(&plogi->node_name, &node_name, sizeof(wwn_t));
 
-       return (sizeof(struct fc_logi_s));
+       return sizeof(struct fc_logi_s);
 }
 
 u16
@@ -267,7 +267,7 @@ fc_flogi_build(struct fchs_s *fchs, struct fc_logi_s *flogi, u32 s_id,
        flogi->csp.npiv_supp    = 1; /* @todo. field name is not correct */
        vvl_info[0]     = bfa_os_htonl(FLOGI_VVL_BRCD);
 
-       return (sizeof(struct fc_logi_s));
+       return sizeof(struct fc_logi_s);
 }
 
 u16
@@ -287,7 +287,7 @@ fc_flogi_acc_build(struct fchs_s *fchs, struct fc_logi_s *flogi, u32 s_id,
 
        flogi->csp.bbcred = bfa_os_htons(local_bb_credits);
 
-       return (sizeof(struct fc_logi_s));
+       return sizeof(struct fc_logi_s);
 }
 
 u16
@@ -306,7 +306,7 @@ fc_fdisc_build(struct fchs_s *fchs, struct fc_logi_s *flogi, u32 s_id,
        flogi->port_name = port_name;
        flogi->node_name = node_name;
 
-       return (sizeof(struct fc_logi_s));
+       return sizeof(struct fc_logi_s);
 }
 
 u16
@@ -338,26 +338,26 @@ fc_plogi_rsp_parse(struct fchs_s *fchs, int len, wwn_t port_name)
        case FC_ELS_LS_RJT:
                ls_rjt = (struct fc_ls_rjt_s *) (fchs + 1);
                if (ls_rjt->reason_code == FC_LS_RJT_RSN_LOGICAL_BUSY)
-                       return (FC_PARSE_BUSY);
+                       return FC_PARSE_BUSY;
                else
-                       return (FC_PARSE_FAILURE);
+                       return FC_PARSE_FAILURE;
        case FC_ELS_ACC:
                plogi = (struct fc_logi_s *) (fchs + 1);
                if (len < sizeof(struct fc_logi_s))
-                       return (FC_PARSE_FAILURE);
+                       return FC_PARSE_FAILURE;
 
                if (!wwn_is_equal(plogi->port_name, port_name))
-                       return (FC_PARSE_FAILURE);
+                       return FC_PARSE_FAILURE;
 
                if (!plogi->class3.class_valid)
-                       return (FC_PARSE_FAILURE);
+                       return FC_PARSE_FAILURE;
 
                if (bfa_os_ntohs(plogi->class3.rxsz) < (FC_MIN_PDUSZ))
-                       return (FC_PARSE_FAILURE);
+                       return FC_PARSE_FAILURE;
 
-               return (FC_PARSE_OK);
+               return FC_PARSE_OK;
        default:
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
        }
 }
 
@@ -372,7 +372,7 @@ fc_plogi_parse(struct fchs_s *fchs)
        if ((bfa_os_ntohs(plogi->class3.rxsz) < FC_MIN_PDUSZ)
            || (bfa_os_ntohs(plogi->class3.rxsz) > FC_MAX_PDUSZ)
            || (plogi->class3.rxsz == 0))
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        return FC_PARSE_OK;
 }
@@ -393,7 +393,7 @@ fc_prli_build(struct fchs_s *fchs, void *pld, u32 d_id, u32 s_id,
        prli->parampage.servparams.task_retry_id = 0;
        prli->parampage.servparams.confirm       = 1;
 
-       return (sizeof(struct fc_prli_s));
+       return sizeof(struct fc_prli_s);
 }
 
 u16
@@ -414,41 +414,41 @@ fc_prli_acc_build(struct fchs_s *fchs, void *pld, u32 d_id, u32 s_id,
 
        prli->parampage.rspcode = FC_PRLI_ACC_XQTD;
 
-       return (sizeof(struct fc_prli_s));
+       return sizeof(struct fc_prli_s);
 }
 
 enum fc_parse_status
 fc_prli_rsp_parse(struct fc_prli_s *prli, int len)
 {
        if (len < sizeof(struct fc_prli_s))
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if (prli->command != FC_ELS_ACC)
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if ((prli->parampage.rspcode != FC_PRLI_ACC_XQTD)
            && (prli->parampage.rspcode != FC_PRLI_ACC_PREDEF_IMG))
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if (prli->parampage.servparams.target != 1)
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
-       return (FC_PARSE_OK);
+       return FC_PARSE_OK;
 }
 
 enum fc_parse_status
 fc_prli_parse(struct fc_prli_s *prli)
 {
        if (prli->parampage.type != FC_TYPE_FCP)
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if (!prli->parampage.imagepair)
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if (!prli->parampage.servparams.initiator)
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
-       return (FC_PARSE_OK);
+       return FC_PARSE_OK;
 }
 
 u16
@@ -462,7 +462,7 @@ fc_logo_build(struct fchs_s *fchs, struct fc_logo_s *logo, u32 d_id,
        logo->nport_id = (s_id);
        logo->orig_port_name = port_name;
 
-       return (sizeof(struct fc_logo_s));
+       return sizeof(struct fc_logo_s);
 }
 
 static          u16
@@ -484,7 +484,7 @@ fc_adisc_x_build(struct fchs_s *fchs, struct fc_adisc_s *adisc, u32 d_id,
        adisc->orig_node_name = node_name;
        adisc->nport_id = (s_id);
 
-       return (sizeof(struct fc_adisc_s));
+       return sizeof(struct fc_adisc_s);
 }
 
 u16
@@ -511,15 +511,15 @@ fc_adisc_rsp_parse(struct fc_adisc_s *adisc, int len, wwn_t port_name,
 {
 
        if (len < sizeof(struct fc_adisc_s))
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if (adisc->els_cmd.els_code != FC_ELS_ACC)
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if (!wwn_is_equal(adisc->orig_port_name, port_name))
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
-       return (FC_PARSE_OK);
+       return FC_PARSE_OK;
 }
 
 enum fc_parse_status
@@ -529,14 +529,14 @@ fc_adisc_parse(struct fchs_s *fchs, void *pld, u32 host_dap,
        struct fc_adisc_s     *adisc = (struct fc_adisc_s *) pld;
 
        if (adisc->els_cmd.els_code != FC_ELS_ACC)
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if ((adisc->nport_id == (host_dap))
            && wwn_is_equal(adisc->orig_port_name, port_name)
            && wwn_is_equal(adisc->orig_node_name, node_name))
-               return (FC_PARSE_OK);
+               return FC_PARSE_OK;
 
-       return (FC_PARSE_FAILURE);
+       return FC_PARSE_FAILURE;
 }
 
 enum fc_parse_status
@@ -550,13 +550,13 @@ fc_pdisc_parse(struct fchs_s *fchs, wwn_t node_name, wwn_t port_name)
        if ((bfa_os_ntohs(pdisc->class3.rxsz) <
                 (FC_MIN_PDUSZ - sizeof(struct fchs_s)))
            || (pdisc->class3.rxsz == 0))
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if (!wwn_is_equal(pdisc->port_name, port_name))
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        if (!wwn_is_equal(pdisc->node_name, node_name))
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        return FC_PARSE_OK;
 }
@@ -570,7 +570,7 @@ fc_abts_build(struct fchs_s *fchs, u32 d_id, u32 s_id, u16 ox_id)
        fchs->s_id = (s_id);
        fchs->ox_id = bfa_os_htons(ox_id);
 
-       return (sizeof(struct fchs_s));
+       return sizeof(struct fchs_s);
 }
 
 enum fc_parse_status
@@ -578,9 +578,9 @@ fc_abts_rsp_parse(struct fchs_s *fchs, int len)
 {
        if ((fchs->cat_info == FC_CAT_BA_ACC)
            || (fchs->cat_info == FC_CAT_BA_RJT))
-               return (FC_PARSE_OK);
+               return FC_PARSE_OK;
 
-       return (FC_PARSE_FAILURE);
+       return FC_PARSE_FAILURE;
 }
 
 u16
@@ -597,7 +597,7 @@ fc_rrq_build(struct fchs_s *fchs, struct fc_rrq_s *rrq, u32 d_id,
        rrq->ox_id = bfa_os_htons(rrq_oxid);
        rrq->rx_id = FC_RXID_ANY;
 
-       return (sizeof(struct fc_rrq_s));
+       return sizeof(struct fc_rrq_s);
 }
 
 u16
@@ -611,7 +611,7 @@ fc_logo_acc_build(struct fchs_s *fchs, void *pld, u32 d_id, u32 s_id,
        memset(acc, 0, sizeof(struct fc_els_cmd_s));
        acc->els_code = FC_ELS_ACC;
 
-       return (sizeof(struct fc_els_cmd_s));
+       return sizeof(struct fc_els_cmd_s);
 }
 
 u16
@@ -627,7 +627,7 @@ fc_ls_rjt_build(struct fchs_s *fchs, struct fc_ls_rjt_s *ls_rjt, u32 d_id,
        ls_rjt->reason_code_expl = reason_code_expl;
        ls_rjt->vendor_unique = 0x00;
 
-       return (sizeof(struct fc_ls_rjt_s));
+       return sizeof(struct fc_ls_rjt_s);
 }
 
 u16
@@ -643,7 +643,7 @@ fc_ba_acc_build(struct fchs_s *fchs, struct fc_ba_acc_s *ba_acc, u32 d_id,
        ba_acc->ox_id = fchs->ox_id;
        ba_acc->rx_id = fchs->rx_id;
 
-       return (sizeof(struct fc_ba_acc_s));
+       return sizeof(struct fc_ba_acc_s);
 }
 
 u16
@@ -654,7 +654,7 @@ fc_ls_acc_build(struct fchs_s *fchs, struct fc_els_cmd_s *els_cmd,
        memset(els_cmd, 0, sizeof(struct fc_els_cmd_s));
        els_cmd->els_code = FC_ELS_ACC;
 
-       return (sizeof(struct fc_els_cmd_s));
+       return sizeof(struct fc_els_cmd_s);
 }
 
 int
@@ -696,7 +696,7 @@ fc_tprlo_acc_build(struct fchs_s *fchs, struct fc_tprlo_acc_s *tprlo_acc,
                tprlo_acc->tprlo_acc_params[page].orig_process_assc = 0;
                tprlo_acc->tprlo_acc_params[page].resp_process_assc = 0;
        }
-       return (bfa_os_ntohs(tprlo_acc->payload_len));
+       return bfa_os_ntohs(tprlo_acc->payload_len);
 }
 
 u16
@@ -721,7 +721,7 @@ fc_prlo_acc_build(struct fchs_s *fchs, struct fc_prlo_acc_s *prlo_acc,
                prlo_acc->prlo_acc_params[page].resp_process_assc = 0;
        }
 
-       return (bfa_os_ntohs(prlo_acc->payload_len));
+       return bfa_os_ntohs(prlo_acc->payload_len);
 }
 
 u16
@@ -735,7 +735,7 @@ fc_rnid_build(struct fchs_s *fchs, struct fc_rnid_cmd_s *rnid, u32 d_id,
        rnid->els_cmd.els_code = FC_ELS_RNID;
        rnid->node_id_data_format = data_format;
 
-       return (sizeof(struct fc_rnid_cmd_s));
+       return sizeof(struct fc_rnid_cmd_s);
 }
 
 u16
@@ -759,10 +759,10 @@ fc_rnid_acc_build(struct fchs_s *fchs, struct fc_rnid_acc_s *rnid_acc,
                rnid_acc->specific_id_data_length =
                        sizeof(struct fc_rnid_general_topology_data_s);
                bfa_os_assign(rnid_acc->gen_topology_data, *gen_topo_data);
-               return (sizeof(struct fc_rnid_acc_s));
+               return sizeof(struct fc_rnid_acc_s);
        } else {
-               return (sizeof(struct fc_rnid_acc_s) -
-                       sizeof(struct fc_rnid_general_topology_data_s));
+               return sizeof(struct fc_rnid_acc_s) -
+                       sizeof(struct fc_rnid_general_topology_data_s);
        }
 
 }
@@ -776,7 +776,7 @@ fc_rpsc_build(struct fchs_s *fchs, struct fc_rpsc_cmd_s *rpsc, u32 d_id,
        memset(rpsc, 0, sizeof(struct fc_rpsc_cmd_s));
 
        rpsc->els_cmd.els_code = FC_ELS_RPSC;
-       return (sizeof(struct fc_rpsc_cmd_s));
+       return sizeof(struct fc_rpsc_cmd_s);
 }
 
 u16
@@ -797,8 +797,8 @@ fc_rpsc2_build(struct fchs_s *fchs, struct fc_rpsc2_cmd_s *rpsc2,
        for (i = 0; i < npids; i++)
                rpsc2->pid_list[i].pid = pid_list[i];
 
-       return (sizeof(struct fc_rpsc2_cmd_s) + ((npids - 1) *
-                       (sizeof(u32))));
+       return sizeof(struct fc_rpsc2_cmd_s) + ((npids - 1) *
+                       (sizeof(u32)));
 }
 
 u16
@@ -819,7 +819,7 @@ fc_rpsc_acc_build(struct fchs_s *fchs, struct fc_rpsc_acc_s *rpsc_acc,
        rpsc_acc->speed_info[0].port_op_speed =
                bfa_os_htons(oper_speed->port_op_speed);
 
-       return (sizeof(struct fc_rpsc_acc_s));
+       return sizeof(struct fc_rpsc_acc_s);
 
 }
 
@@ -856,7 +856,7 @@ fc_pdisc_build(struct fchs_s *fchs, u32 d_id, u32 s_id,
        pdisc->port_name = port_name;
        pdisc->node_name = node_name;
 
-       return (sizeof(struct fc_logi_s));
+       return sizeof(struct fc_logi_s);
 }
 
 u16
@@ -865,21 +865,21 @@ fc_pdisc_rsp_parse(struct fchs_s *fchs, int len, wwn_t port_name)
        struct fc_logi_s     *pdisc = (struct fc_logi_s *) (fchs + 1);
 
        if (len < sizeof(struct fc_logi_s))
-               return (FC_PARSE_LEN_INVAL);
+               return FC_PARSE_LEN_INVAL;
 
        if (pdisc->els_cmd.els_code != FC_ELS_ACC)
-               return (FC_PARSE_ACC_INVAL);
+               return FC_PARSE_ACC_INVAL;
 
        if (!wwn_is_equal(pdisc->port_name, port_name))
-               return (FC_PARSE_PWWN_NOT_EQUAL);
+               return FC_PARSE_PWWN_NOT_EQUAL;
 
        if (!pdisc->class3.class_valid)
-               return (FC_PARSE_NWWN_NOT_EQUAL);
+               return FC_PARSE_NWWN_NOT_EQUAL;
 
        if (bfa_os_ntohs(pdisc->class3.rxsz) < (FC_MIN_PDUSZ))
-               return (FC_PARSE_RXSZ_INVAL);
+               return FC_PARSE_RXSZ_INVAL;
 
-       return (FC_PARSE_OK);
+       return FC_PARSE_OK;
 }
 
 u16
@@ -903,7 +903,7 @@ fc_prlo_build(struct fchs_s *fchs, u32 d_id, u32 s_id, u16 ox_id,
                prlo->prlo_params[page].resp_process_assc = 0;
        }
 
-       return (bfa_os_ntohs(prlo->payload_len));
+       return bfa_os_ntohs(prlo->payload_len);
 }
 
 u16
@@ -916,7 +916,7 @@ fc_prlo_rsp_parse(struct fchs_s *fchs, int len)
        len = len;
 
        if (prlo->command != FC_ELS_ACC)
-               return (FC_PARSE_FAILURE);
+               return FC_PARSE_FAILURE;
 
        num_pages = ((bfa_os_ntohs(prlo->payload_len)) - 4) / 16;
 
@@ -936,7 +936,7 @@ fc_prlo_rsp_parse(struct fchs_s *fchs, int len)
                if (prlo->prlo_acc_params[page].resp_process_assc != 0)
                        return FC_PARSE_FAILURE;
        }
-       return (FC_PARSE_OK);
+       return FC_PARSE_OK;
 
 }
 
@@ -968,7 +968,7 @@ fc_tprlo_build(struct fchs_s *fchs, u32 d_id, u32 s_id,
                }
        }
 
-       return (bfa_os_ntohs(tprlo->payload_len));
+       return bfa_os_ntohs(tprlo->payload_len);
 }
 
 u16
@@ -981,23 +981,23 @@ fc_tprlo_rsp_parse(struct fchs_s *fchs, int len)
        len = len;
 
        if (tprlo->command != FC_ELS_ACC)
-               return (FC_PARSE_ACC_INVAL);
+               return FC_PARSE_ACC_INVAL;
 
        num_pages = (bfa_os_ntohs(tprlo->payload_len) - 4) / 16;
 
        for (page = 0; page < num_pages; page++) {
                if (tprlo->tprlo_acc_params[page].type != FC_TYPE_FCP)
-                       return (FC_PARSE_NOT_FCP);
+                       return FC_PARSE_NOT_FCP;
                if (tprlo->tprlo_acc_params[page].opa_valid != 0)
-                       return (FC_PARSE_OPAFLAG_INVAL);
+                       return FC_PARSE_OPAFLAG_INVAL;
                if (tprlo->tprlo_acc_params[page].rpa_valid != 0)
-                       return (FC_PARSE_RPAFLAG_INVAL);
+                       return FC_PARSE_RPAFLAG_INVAL;
                if (tprlo->tprlo_acc_params[page].orig_process_assc != 0)
-                       return (FC_PARSE_OPA_INVAL);
+                       return FC_PARSE_OPA_INVAL;
                if (tprlo->tprlo_acc_params[page].resp_process_assc != 0)
-                       return (FC_PARSE_RPA_INVAL);
+                       return FC_PARSE_RPA_INVAL;
        }
-       return (FC_PARSE_OK);
+       return FC_PARSE_OK;
 }
 
 enum fc_parse_status
@@ -1024,7 +1024,7 @@ fc_ba_rjt_build(struct fchs_s *fchs, u32 d_id, u32 s_id,
        fchs->cat_info = FC_CAT_BA_RJT;
        ba_rjt->reason_code = reason_code;
        ba_rjt->reason_expl = reason_expl;
-       return (sizeof(struct fc_ba_rjt_s));
+       return sizeof(struct fc_ba_rjt_s);
 }
 
 static void
@@ -1073,7 +1073,7 @@ fc_gidpn_build(struct fchs_s *fchs, void *pyld, u32 s_id, u16 ox_id,
 
        bfa_os_memset(gidpn, 0, sizeof(struct fcgs_gidpn_req_s));
        gidpn->port_name = port_name;
-       return (sizeof(struct fcgs_gidpn_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_gidpn_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1090,7 +1090,7 @@ fc_gpnid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u16 ox_id,
 
        bfa_os_memset(gpnid, 0, sizeof(fcgs_gpnid_req_t));
        gpnid->dap = port_id;
-       return (sizeof(fcgs_gpnid_req_t) + sizeof(struct ct_hdr_s));
+       return sizeof(fcgs_gpnid_req_t) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1107,7 +1107,7 @@ fc_gnnid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u16 ox_id,
 
        bfa_os_memset(gnnid, 0, sizeof(fcgs_gnnid_req_t));
        gnnid->dap = port_id;
-       return (sizeof(fcgs_gnnid_req_t) + sizeof(struct ct_hdr_s));
+       return sizeof(fcgs_gnnid_req_t) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1137,7 +1137,7 @@ fc_scr_build(struct fchs_s *fchs, struct fc_scr_s *scr, u8 set_br_reg,
        if (set_br_reg)
                scr->vu_reg_func = FC_VU_SCR_REG_FUNC_FABRIC_NAME_CHANGE;
 
-       return (sizeof(struct fc_scr_s));
+       return sizeof(struct fc_scr_s);
 }
 
 u16
@@ -1157,7 +1157,7 @@ fc_rscn_build(struct fchs_s *fchs, struct fc_rscn_pl_s *rscn, u32 s_id,
        rscn->event[0].format = FC_RSCN_FORMAT_PORTID;
        rscn->event[0].portid = s_id;
 
-       return (sizeof(struct fc_rscn_pl_s));
+       return sizeof(struct fc_rscn_pl_s);
 }
 
 u16
@@ -1188,7 +1188,7 @@ fc_rftid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u16 ox_id,
                rftid->fc4_type[index] |= bfa_os_htonl(type_value);
        }
 
-       return (sizeof(struct fcgs_rftid_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_rftid_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1210,7 +1210,7 @@ fc_rftid_build_sol(struct fchs_s *fchs, void *pyld, u32 s_id,
        bfa_os_memcpy((void *)rftid->fc4_type, (void *)fc4_bitmap,
                        (bitmap_size < 32 ? bitmap_size : 32));
 
-       return (sizeof(struct fcgs_rftid_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_rftid_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1231,7 +1231,7 @@ fc_rffid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u16 ox_id,
        rffid->fc4ftr_bits  = fc4_ftrs;
        rffid->fc4_type         = fc4_type;
 
-       return (sizeof(struct fcgs_rffid_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_rffid_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1253,7 +1253,7 @@ fc_rspnid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u16 ox_id,
        rspnid->spn_len = (u8) strlen((char *)name);
        strncpy((char *)rspnid->spn, (char *)name, rspnid->spn_len);
 
-       return (sizeof(struct fcgs_rspnid_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_rspnid_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1275,7 +1275,7 @@ fc_gid_ft_build(struct fchs_s *fchs, void *pyld, u32 s_id,
        gidft->domain_id = 0;
        gidft->area_id = 0;
 
-       return (sizeof(struct fcgs_gidft_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_gidft_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1294,7 +1294,7 @@ fc_rpnid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u32 port_id,
        rpnid->port_id = port_id;
        rpnid->port_name = port_name;
 
-       return (sizeof(struct fcgs_rpnid_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_rpnid_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1313,7 +1313,7 @@ fc_rnnid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u32 port_id,
        rnnid->port_id = port_id;
        rnnid->node_name = node_name;
 
-       return (sizeof(struct fcgs_rnnid_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_rnnid_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1332,7 +1332,7 @@ fc_rcsid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u32 port_id,
        rcsid->port_id = port_id;
        rcsid->cos = cos;
 
-       return (sizeof(struct fcgs_rcsid_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_rcsid_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1351,7 +1351,7 @@ fc_rptid_build(struct fchs_s *fchs, void *pyld, u32 s_id, u32 port_id,
        rptid->port_id = port_id;
        rptid->port_type = port_type;
 
-       return (sizeof(struct fcgs_rptid_req_s) + sizeof(struct ct_hdr_s));
+       return sizeof(struct fcgs_rptid_req_s) + sizeof(struct ct_hdr_s);
 }
 
 u16
@@ -1368,7 +1368,7 @@ fc_ganxt_build(struct fchs_s *fchs, void *pyld, u32 s_id, u32 port_id)
        bfa_os_memset(ganxt, 0, sizeof(struct fcgs_ganxt_req_s));
        ganxt->port_id = port_id;
 
-       return (sizeof(struct ct_hdr_s) + sizeof(struct fcgs_ganxt_req_s));
+       return sizeof(struct ct_hdr_s) + sizeof(struct fcgs_ganxt_req_s);
 }
 
 /*
@@ -1385,7 +1385,7 @@ fc_fdmi_reqhdr_build(struct fchs_s *fchs, void *pyld, u32 s_id,
        fc_gs_fchdr_build(fchs, d_id, s_id, 0);
        fc_gs_fdmi_cthdr_build(cthdr, s_id, cmd_code);
 
-       return (sizeof(struct ct_hdr_s));
+       return sizeof(struct ct_hdr_s);
 }
 
 /*
@@ -1425,7 +1425,7 @@ fc_gmal_req_build(struct fchs_s *fchs, void *pyld, u32 s_id, wwn_t wwn)
        bfa_os_memset(gmal, 0, sizeof(fcgs_gmal_req_t));
        gmal->wwn = wwn;
 
-       return (sizeof(struct ct_hdr_s) + sizeof(fcgs_gmal_req_t));
+       return sizeof(struct ct_hdr_s) + sizeof(fcgs_gmal_req_t);
 }
 
 /*
@@ -1445,5 +1445,5 @@ fc_gfn_req_build(struct fchs_s *fchs, void *pyld, u32 s_id, wwn_t wwn)
        bfa_os_memset(gfn, 0, sizeof(fcgs_gfn_req_t));
        gfn->wwn = wwn;
 
-       return (sizeof(struct ct_hdr_s) + sizeof(fcgs_gfn_req_t));
+       return sizeof(struct ct_hdr_s) + sizeof(fcgs_gfn_req_t);
 }