staging/rdma/hfi1: Fix logical continuations
authorJubin John <jubin.john@intel.com>
Mon, 15 Feb 2016 04:20:15 +0000 (20:20 -0800)
committerDoug Ledford <dledford@redhat.com>
Fri, 11 Mar 2016 01:45:37 +0000 (20:45 -0500)
Move logical continuations to previous line to fix checkpatch check:
CHECK: Logical continuations should be on the previous line

Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
Reviewed-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Jubin John <jubin.john@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/staging/rdma/hfi1/chip.c
drivers/staging/rdma/hfi1/driver.c
drivers/staging/rdma/hfi1/file_ops.c
drivers/staging/rdma/hfi1/firmware.c
drivers/staging/rdma/hfi1/intr.c
drivers/staging/rdma/hfi1/mad.c
drivers/staging/rdma/hfi1/sdma.c

index 51256ba9d293b9133bf11d260fb794f1701e0cdf..b4c017abab9f98d6c0ea4fa760599a2f982c11e2 100644 (file)
@@ -6552,8 +6552,8 @@ void handle_sma_message(struct work_struct *work)
                 *
                 * Can activate the node.  Discard otherwise.
                 */
-               if (ppd->host_link_state == HLS_UP_ARMED
-                                       && ppd->is_active_optimize_enabled) {
+               if (ppd->host_link_state == HLS_UP_ARMED &&
+                   ppd->is_active_optimize_enabled) {
                        ppd->neighbor_normal = 1;
                        ret = set_link_state(ppd, HLS_UP_ACTIVE);
                        if (ret)
@@ -7032,8 +7032,8 @@ static void get_link_widths(struct hfi1_devdata *dd, u16 *tx_width,
         * handle_verify_cap().  The ASIC 8051 firmware does not correctly
         * set the max_rate field in handle_verify_cap until v0.19.
         */
-       if ((dd->icode == ICODE_RTL_SILICON)
-                               && (dd->dc8051_ver < dc8051_ver(0, 19))) {
+       if ((dd->icode == ICODE_RTL_SILICON) &&
+           (dd->dc8051_ver < dc8051_ver(0, 19))) {
                /* max_rate: 0 = 12.5G, 1 = 25G */
                switch (max_rate) {
                case 0:
@@ -7358,10 +7358,8 @@ retry:
                /* downgrade is disabled */
 
                /* bounce if not at starting active width */
-               if ((ppd->link_width_active !=
-                                       ppd->link_width_downgrade_tx_active)
-                               || (ppd->link_width_active !=
-                                       ppd->link_width_downgrade_rx_active)) {
+               if ((ppd->link_width_active != ppd->link_width_downgrade_tx_active) ||
+                   (ppd->link_width_active != ppd->link_width_downgrade_rx_active)) {
                        dd_dev_err(ppd->dd,
                                "Link downgrade is disabled and link has downgraded, downing link\n");
                        dd_dev_err(ppd->dd,
@@ -7371,8 +7369,8 @@ retry:
                                ppd->link_width_downgrade_rx_active);
                        do_bounce = 1;
                }
-       } else if ((lwde & ppd->link_width_downgrade_tx_active) == 0
-               || (lwde & ppd->link_width_downgrade_rx_active) == 0) {
+       } else if ((lwde & ppd->link_width_downgrade_tx_active) == 0 ||
+                  (lwde & ppd->link_width_downgrade_rx_active) == 0) {
                /* Tx or Rx is outside the enabled policy */
                dd_dev_err(ppd->dd,
                        "Link is outside of downgrade allowed, downing link\n");
@@ -7567,9 +7565,9 @@ static void handle_8051_interrupt(struct hfi1_devdata *dd, u32 unused, u64 reg)
        if (queue_link_down) {
                /* if the link is already going down or disabled, do not
                 * queue another */
-               if ((ppd->host_link_state
-                                   & (HLS_GOING_OFFLINE | HLS_LINK_COOLDOWN))
-                               || ppd->link_enabled == 0) {
+               if ((ppd->host_link_state &
+                   (HLS_GOING_OFFLINE | HLS_LINK_COOLDOWN)) ||
+                   ppd->link_enabled == 0) {
                        dd_dev_info(dd, "%s: not queuing link down\n",
                                __func__);
                } else {
@@ -8888,10 +8886,9 @@ static int init_loopback(struct hfi1_devdata *dd)
         *
         * Accept all valid loopback values.
         */
-       if ((dd->icode == ICODE_FUNCTIONAL_SIMULATOR)
-               && (loopback == LOOPBACK_SERDES
-                       || loopback == LOOPBACK_LCB
-                       || loopback == LOOPBACK_CABLE)) {
+       if ((dd->icode == ICODE_FUNCTIONAL_SIMULATOR) &&
+           (loopback == LOOPBACK_SERDES || loopback == LOOPBACK_LCB ||
+            loopback == LOOPBACK_CABLE)) {
                loopback = LOOPBACK_LCB;
                quick_linkup = 1;
                return 0;
@@ -10020,8 +10017,8 @@ int set_link_state(struct hfi1_pportdata *ppd, u32 state)
                state = dd->link_default;
 
        /* interpret poll -> poll as a link bounce */
-       poll_bounce = ppd->host_link_state == HLS_DN_POLL
-                               && state == HLS_DN_POLL;
+       poll_bounce = ppd->host_link_state == HLS_DN_POLL &&
+                     state == HLS_DN_POLL;
 
        dd_dev_info(dd, "%s: current %s, new %s %s%s\n", __func__,
                link_state_name(ppd->host_link_state),
@@ -10048,8 +10045,8 @@ int set_link_state(struct hfi1_pportdata *ppd, u32 state)
 
        switch (state) {
        case HLS_UP_INIT:
-               if (ppd->host_link_state == HLS_DN_POLL && (quick_linkup
-                           || dd->icode == ICODE_FUNCTIONAL_SIMULATOR)) {
+               if (ppd->host_link_state == HLS_DN_POLL &&
+                   (quick_linkup || dd->icode == ICODE_FUNCTIONAL_SIMULATOR)) {
                        /*
                         * Quick link up jumps from polling to here.
                         *
@@ -10779,8 +10776,8 @@ int set_buffer_control(struct hfi1_pportdata *ppd,
                                                != cur_bc.vl[i].shared;
                if (this_shared_changing)
                        any_shared_limit_changing = 1;
-               if (new_bc->vl[i].dedicated != cur_bc.vl[i].dedicated
-                               || this_shared_changing) {
+               if (new_bc->vl[i].dedicated != cur_bc.vl[i].dedicated ||
+                   this_shared_changing) {
                        changing[i] = 1;
                        changing_mask |= stat_mask;
                        change_count++;
@@ -11227,8 +11224,8 @@ void hfi1_rcvctrl(struct hfi1_devdata *dd, unsigned int op, int ctxt)
 
        rcvctrl = read_kctxt_csr(dd, ctxt, RCV_CTXT_CTRL);
        /* if the context already enabled, don't do the extra steps */
-       if ((op & HFI1_RCVCTRL_CTXT_ENB)
-                       && !(rcvctrl & RCV_CTXT_CTRL_ENABLE_SMASK)) {
+       if ((op & HFI1_RCVCTRL_CTXT_ENB) &&
+           !(rcvctrl & RCV_CTXT_CTRL_ENABLE_SMASK)) {
                /* reset the tail and hdr addresses, and sequence count */
                write_kctxt_csr(dd, ctxt, RCV_HDR_ADDR,
                                rcd->rcvhdrq_phys);
@@ -11344,8 +11341,8 @@ void hfi1_rcvctrl(struct hfi1_devdata *dd, unsigned int op, int ctxt)
        write_kctxt_csr(dd, ctxt, RCV_CTXT_CTRL, rcd->rcvctrl);
 
        /* work around sticky RcvCtxtStatus.BlockedRHQFull */
-       if (did_enable
-           && (rcvctrl & RCV_CTXT_CTRL_DONT_DROP_RHQ_FULL_SMASK)) {
+       if (did_enable &&
+           (rcvctrl & RCV_CTXT_CTRL_DONT_DROP_RHQ_FULL_SMASK)) {
                reg = read_kctxt_csr(dd, ctxt, RCV_CTXT_STATUS);
                if (reg != 0) {
                        dd_dev_info(dd, "ctxt %d status %lld (blocked)\n",
@@ -13989,8 +13986,8 @@ struct hfi1_devdata *hfi1_init_dd(struct pci_dev *pdev,
                /* link width active is 0 when link is down */
                /* link width downgrade active is 0 when link is down */
 
-               if (num_vls < HFI1_MIN_VLS_SUPPORTED
-                       || num_vls > HFI1_MAX_VLS_SUPPORTED) {
+               if (num_vls < HFI1_MIN_VLS_SUPPORTED ||
+                   num_vls > HFI1_MAX_VLS_SUPPORTED) {
                        hfi1_early_err(&pdev->dev,
                                       "Invalid num_vls %u, using %u VLs\n",
                                    num_vls, HFI1_MAX_VLS_SUPPORTED);
index fee5e395608a9525700d5c3d5c697b127f599a04..3ef297ecdd60e1b10c82849c73d7cf64d5380ab3 100644 (file)
@@ -1123,9 +1123,9 @@ int set_mtu(struct hfi1_pportdata *ppd)
        ppd->ibmaxlen = ppd->ibmtu + lrh_max_header_bytes(ppd->dd);
 
        mutex_lock(&ppd->hls_lock);
-       if (ppd->host_link_state == HLS_UP_INIT
-                       || ppd->host_link_state == HLS_UP_ARMED
-                       || ppd->host_link_state == HLS_UP_ACTIVE)
+       if (ppd->host_link_state == HLS_UP_INIT ||
+           ppd->host_link_state == HLS_UP_ARMED ||
+           ppd->host_link_state == HLS_UP_ACTIVE)
                is_up = 1;
 
        drain = !is_ax(dd) && is_up;
index b52cb78c1f450be173007f339c182398a1290e89..5077ee0691548c18267e78fd2310083de4dc476e 100644 (file)
@@ -1626,12 +1626,12 @@ static ssize_t ui_read(struct file *filp, char __user *buf, size_t count,
                 * them.  These registers are defined as having a read value
                 * of 0.
                 */
-               else if (csr_off == ASIC_GPIO_CLEAR
-                               || csr_off == ASIC_GPIO_FORCE
-                               || csr_off == ASIC_QSFP1_CLEAR
-                               || csr_off == ASIC_QSFP1_FORCE
-                               || csr_off == ASIC_QSFP2_CLEAR
-                               || csr_off == ASIC_QSFP2_FORCE)
+               else if (csr_off == ASIC_GPIO_CLEAR ||
+                        csr_off == ASIC_GPIO_FORCE ||
+                        csr_off == ASIC_QSFP1_CLEAR ||
+                        csr_off == ASIC_QSFP1_FORCE ||
+                        csr_off == ASIC_QSFP2_CLEAR ||
+                        csr_off == ASIC_QSFP2_FORCE)
                        data = 0;
                else if (csr_off >= barlen) {
                        /*
index 31550a377f177317fbfeae8d9c663f115e29e850..1af5e3406f04a30b851deba0665c0dfc582579b3 100644 (file)
@@ -391,19 +391,13 @@ static int invalid_header(struct hfi1_devdata *dd, const char *what,
 static int verify_css_header(struct hfi1_devdata *dd, struct css_header *css)
 {
        /* verify CSS header fields (most sizes are in DW, so add /4) */
-       if (invalid_header(dd, "module_type", css->module_type, CSS_MODULE_TYPE)
-                       || invalid_header(dd, "header_len", css->header_len,
-                                       (sizeof(struct firmware_file) / 4))
-                       || invalid_header(dd, "header_version",
-                                       css->header_version, CSS_HEADER_VERSION)
-                       || invalid_header(dd, "module_vendor",
-                                       css->module_vendor, CSS_MODULE_VENDOR)
-                       || invalid_header(dd, "key_size",
-                                       css->key_size, KEY_SIZE / 4)
-                       || invalid_header(dd, "modulus_size",
-                                       css->modulus_size, KEY_SIZE / 4)
-                       || invalid_header(dd, "exponent_size",
-                                       css->exponent_size, EXPONENT_SIZE / 4)) {
+       if (invalid_header(dd, "module_type", css->module_type, CSS_MODULE_TYPE) ||
+           invalid_header(dd, "header_len", css->header_len, (sizeof(struct firmware_file) / 4)) ||
+           invalid_header(dd, "header_version", css->header_version, CSS_HEADER_VERSION) ||
+           invalid_header(dd, "module_vendor", css->module_vendor, CSS_MODULE_VENDOR) ||
+           invalid_header(dd, "key_size", css->key_size, KEY_SIZE / 4) ||
+           invalid_header(dd, "modulus_size", css->modulus_size, KEY_SIZE / 4) ||
+           invalid_header(dd, "exponent_size", css->exponent_size, EXPONENT_SIZE / 4)) {
                return -EINVAL;
        }
        return 0;
index 685fb4d9c924281d2d90e102352a1c0438ad3dc1..03cebae672a3be00047cf9b02874056b14869f3e 100644 (file)
@@ -131,8 +131,7 @@ void handle_linkup_change(struct hfi1_devdata *dd, u32 linkup)
                 * NOTE: This uses this device's vAU, vCU, and vl15_init for
                 * the remote values.  Both sides must be using the values.
                 */
-               if (quick_linkup
-                           || dd->icode == ICODE_FUNCTIONAL_SIMULATOR) {
+               if (quick_linkup || dd->icode == ICODE_FUNCTIONAL_SIMULATOR) {
                        set_up_vl15(dd, dd->vau, dd->vl15_init);
                        assign_remote_cm_au_table(dd, dd->vcu);
                        ppd->neighbor_guid =
index adfd0a9cead7257edd82f3d58e3e388469ef1af1..ae594f49c78d26f4647cdcc895f7c73a5fcaf015 100644 (file)
@@ -1170,8 +1170,8 @@ static int __subn_set_opa_portinfo(struct opa_smp *smp, u32 am, u8 *data,
        ppd->port_error_action = be32_to_cpu(pi->port_error_action);
        lwe = be16_to_cpu(pi->link_width.enabled);
        if (lwe) {
-               if (lwe == OPA_LINK_WIDTH_RESET
-                               || lwe == OPA_LINK_WIDTH_RESET_OLD)
+               if (lwe == OPA_LINK_WIDTH_RESET ||
+                   lwe == OPA_LINK_WIDTH_RESET_OLD)
                        set_link_width_enabled(ppd, ppd->link_width_supported);
                else if ((lwe & ~ppd->link_width_supported) == 0)
                        set_link_width_enabled(ppd, lwe);
@@ -1180,8 +1180,8 @@ static int __subn_set_opa_portinfo(struct opa_smp *smp, u32 am, u8 *data,
        }
        lwe = be16_to_cpu(pi->link_width_downgrade.enabled);
        /* LWD.E is always applied - 0 means "disabled" */
-       if (lwe == OPA_LINK_WIDTH_RESET
-                       || lwe == OPA_LINK_WIDTH_RESET_OLD) {
+       if (lwe == OPA_LINK_WIDTH_RESET ||
+           lwe == OPA_LINK_WIDTH_RESET_OLD) {
                set_link_width_downgrade_enabled(ppd,
                                ppd->link_width_downgrade_supported);
        } else if ((lwe & ~ppd->link_width_downgrade_supported) == 0) {
@@ -2335,8 +2335,8 @@ static int pma_get_opa_portstatus(struct opa_pma_mad *pmp,
                return reply((struct ib_mad_hdr *)pmp);
        }
 
-       if (nports != 1 || (port_num && port_num != port)
-           || num_vls > OPA_MAX_VLS || (vl_select_mask & ~VL_MASK_ALL)) {
+       if (nports != 1 || (port_num && port_num != port) ||
+           num_vls > OPA_MAX_VLS || (vl_select_mask & ~VL_MASK_ALL)) {
                pmp->mad_hdr.status |= IB_SMP_INVALID_FIELD;
                return reply((struct ib_mad_hdr *)pmp);
        }
index c0ff07943936b51d0f4a0d6fb87238fbd63ff6d9..c8c0aace70ffa2306db94545136de340ecc3c39a 100644 (file)
@@ -672,8 +672,8 @@ static void sdma_set_state(struct sdma_engine *sde,
        ss->previous_op = ss->current_op;
        ss->current_state = next_state;
 
-       if (ss->previous_state != sdma_state_s99_running
-               && next_state == sdma_state_s99_running)
+       if (ss->previous_state != sdma_state_s99_running &&
+           next_state == sdma_state_s99_running)
                sdma_flush(sde);
 
        if (action[next_state].op_enable)