megasas_enable_intr_xscale(struct megasas_instance *instance)
 {
        struct megasas_register_set __iomem *regs;
+
        regs = instance->reg_set;
        writel(0, &(regs)->outbound_intr_mask);
 
 {
        struct megasas_register_set __iomem *regs;
        u32 mask = 0x1f;
+
        regs = instance->reg_set;
        writel(mask, ®s->outbound_intr_mask);
        /* Dummy readl to force pci flush */
 {
        u32 status;
        u32 mfiStatus = 0;
+
        /*
         * Check if it is our interrupt
         */
                struct megasas_register_set __iomem *regs)
 {
        unsigned long flags;
+
        spin_lock_irqsave(&instance->hba_lock, flags);
        writel((frame_phys_addr >> 3)|(frame_count),
               &(regs)->inbound_queue_port);
 {
        u32 i;
        u32 pcidata;
+
        writel(MFI_ADP_RESET, ®s->inbound_doorbell);
 
        for (i = 0; i < 3; i++)
 megasas_check_reset_xscale(struct megasas_instance *instance,
                struct megasas_register_set __iomem *regs)
 {
-
        if ((instance->adprecovery != MEGASAS_HBA_OPERATIONAL) &&
            (le32_to_cpu(*instance->consumer) ==
                MEGASAS_ADPRESET_INPROG_SIGN))
 
 /**
 *      The following functions are defined for ppc (deviceid : 0x60)
-*      controllers
+*      controllers
 */
 
 /**
 megasas_enable_intr_ppc(struct megasas_instance *instance)
 {
        struct megasas_register_set __iomem *regs;
+
        regs = instance->reg_set;
        writel(0xFFFFFFFF, &(regs)->outbound_doorbell_clear);
 
 {
        struct megasas_register_set __iomem *regs;
        u32 mask = 0xFFFFFFFF;
+
        regs = instance->reg_set;
        writel(mask, ®s->outbound_intr_mask);
        /* Dummy readl to force pci flush */
                struct megasas_register_set __iomem *regs)
 {
        unsigned long flags;
+
        spin_lock_irqsave(&instance->hba_lock, flags);
        writel((frame_phys_addr | (frame_count<<1))|1,
                        &(regs)->inbound_queue_port);
 megasas_enable_intr_skinny(struct megasas_instance *instance)
 {
        struct megasas_register_set __iomem *regs;
+
        regs = instance->reg_set;
        writel(0xFFFFFFFF, &(regs)->outbound_intr_mask);
 
 {
        struct megasas_register_set __iomem *regs;
        u32 mask = 0xFFFFFFFF;
+
        regs = instance->reg_set;
        writel(mask, ®s->outbound_intr_mask);
        /* Dummy readl to force pci flush */
        writel(status, ®s->outbound_intr_status);
 
        /*
-       * dummy read to flush PCI
-       */
+        * dummy read to flush PCI
+        */
        readl(®s->outbound_intr_status);
 
        return mfiStatus;
                        struct megasas_register_set __iomem *regs)
 {
        unsigned long flags;
+
        spin_lock_irqsave(&instance->hba_lock, flags);
        writel(upper_32_bits(frame_phys_addr),
               &(regs)->inbound_high_queue_port);
 megasas_enable_intr_gen2(struct megasas_instance *instance)
 {
        struct megasas_register_set __iomem *regs;
+
        regs = instance->reg_set;
        writel(0xFFFFFFFF, &(regs)->outbound_doorbell_clear);
 
 {
        struct megasas_register_set __iomem *regs;
        u32 mask = 0xFFFFFFFF;
+
        regs = instance->reg_set;
        writel(mask, ®s->outbound_intr_mask);
        /* Dummy readl to force pci flush */
 {
        u32 status;
        u32 mfiStatus = 0;
+
        /*
         * Check if it is our interrupt
         */
                        struct megasas_register_set __iomem *regs)
 {
        unsigned long flags;
+
        spin_lock_irqsave(&instance->hba_lock, flags);
        writel((frame_phys_addr | (frame_count<<1))|1,
                        &(regs)->inbound_queue_port);
 megasas_adp_reset_gen2(struct megasas_instance *instance,
                        struct megasas_register_set __iomem *reg_set)
 {
-       u32                     retry = 0 ;
-       u32                     HostDiag;
-       u32 __iomem             *seq_offset = ®_set->seq_offset;
-       u32 __iomem             *hostdiag_offset = ®_set->host_diag;
+       u32 retry = 0 ;
+       u32 HostDiag;
+       u32 __iomem *seq_offset = ®_set->seq_offset;
+       u32 __iomem *hostdiag_offset = ®_set->host_diag;
 
        if (instance->instancet == &megasas_instance_template_skinny) {
                seq_offset = ®_set->fusion_seq_offset;
 
        HostDiag = (u32)readl(hostdiag_offset);
 
-       while ( !( HostDiag & DIAG_WRITE_ENABLE) ) {
+       while (!(HostDiag & DIAG_WRITE_ENABLE)) {
                msleep(100);
                HostDiag = (u32)readl(hostdiag_offset);
                dev_notice(&instance->pdev->dev, "RESETGEN2: retry=%x, hostdiag=%x\n",
        ssleep(10);
 
        HostDiag = (u32)readl(hostdiag_offset);
-       while ( ( HostDiag & DIAG_RESET_ADAPTER) ) {
+       while (HostDiag & DIAG_RESET_ADAPTER) {
                msleep(100);
                HostDiag = (u32)readl(hostdiag_offset);
                dev_notice(&instance->pdev->dev, "RESET_GEN2: retry=%x, hostdiag=%x\n",
 megasas_issue_polled(struct megasas_instance *instance, struct megasas_cmd *cmd)
 {
        int seconds;
-
        struct megasas_header *frame_hdr = &cmd->frame->hdr;
 
        frame_hdr->cmd_status = MFI_CMD_STATUS_POLL_MODE;
                          struct megasas_cmd *cmd, int timeout)
 {
        int ret = 0;
+
        cmd->cmd_status_drv = MFI_STAT_INVALID_STATUS;
 
        instance->instancet->issue_dcmd(instance, cmd);
        int num_cnt;
        int sge_bytes;
        u32 sge_sz;
-       u32 frame_count=0;
+       u32 frame_count = 0;
 
        sge_sz = (IS_DMA64) ? sizeof(struct megasas_sge64) :
            sizeof(struct megasas_sge32);
                        num_cnt = sge_count - 3;
        }
 
-       if(num_cnt>0){
+       if (num_cnt > 0) {
                sge_bytes = sge_sz * num_cnt;
 
                frame_count = (sge_bytes / MEGAMFI_FRAME_SIZE) +
                    ((sge_bytes % MEGAMFI_FRAME_SIZE) ? 1 : 0) ;
        }
        /* Main frame */
-       frame_count +=1;
+       frame_count += 1;
 
        if (frame_count > 7)
                frame_count = 8;
        memcpy(pthru->cdb, scp->cmnd, scp->cmd_len);
 
        /*
-       * If the command is for the tape device, set the
-       * pthru timeout to the os layer timeout value.
-       */
+        * If the command is for the tape device, set the
+        * pthru timeout to the os layer timeout value.
+        */
        if (scp->device->type == TYPE_TAPE) {
                if ((scp->request->timeout / HZ) > 0xFFFF)
                        pthru->timeout = cpu_to_le16(0xFFFF);
 
  /**
  * megasas_dump_pending_frames -       Dumps the frame address of all pending cmds
- *                                     in FW
+ *                                     in FW
  * @instance:                          Adapter soft state
  */
 static inline void
        dev_err(&instance->pdev->dev, "[%d]: Pending OS cmds in FW : \n",instance->host->host_no);
        for (i = 0; i < max_cmd; i++) {
                cmd = instance->cmd_list[i];
-               if(!cmd->scmd)
+               if (!cmd->scmd)
                        continue;
                dev_err(&instance->pdev->dev, "[%d]: Frame addr :0x%08lx : ",instance->host->host_no,(unsigned long)cmd->frame_phys_addr);
                if (megasas_cmd_type(cmd->scmd) == READ_WRITE_LDIO) {
                        instance->host->host_no, cmd->frame_count, ldio->cmd, ldio->target_id,
                        le32_to_cpu(ldio->start_lba_lo), le32_to_cpu(ldio->start_lba_hi),
                        le32_to_cpu(ldio->sense_buf_phys_addr_lo), sgcount);
-               }
-               else {
+               } else {
                        pthru = (struct megasas_pthru_frame *) cmd->frame;
                        mfi_sgl = &pthru->sgl;
                        sgcount = pthru->sge_count;
                        pthru->lun, pthru->cdb_len, le32_to_cpu(pthru->data_xfer_len),
                        le32_to_cpu(pthru->sense_buf_phys_addr_lo), sgcount);
                }
-       if(megasas_dbg_lvl & MEGASAS_DBG_LVL){
-               for (n = 0; n < sgcount; n++){
-                       if (IS_DMA64)
-                               dev_err(&instance->pdev->dev, "sgl len : 0x%x, sgl addr : 0x%llx\n",
-                                       le32_to_cpu(mfi_sgl->sge64[n].length),
-                                       le64_to_cpu(mfi_sgl->sge64[n].phys_addr));
-                       else
-                               dev_err(&instance->pdev->dev, "sgl len : 0x%x, sgl addr : 0x%x\n",
-                                       le32_to_cpu(mfi_sgl->sge32[n].length),
-                                       le32_to_cpu(mfi_sgl->sge32[n].phys_addr));
+               if (megasas_dbg_lvl & MEGASAS_DBG_LVL) {
+                       for (n = 0; n < sgcount; n++) {
+                               if (IS_DMA64)
+                                       dev_err(&instance->pdev->dev, "sgl len : 0x%x, sgl addr : 0x%llx\n",
+                                               le32_to_cpu(mfi_sgl->sge64[n].length),
+                                               le64_to_cpu(mfi_sgl->sge64[n].phys_addr));
+                               else
+                                       dev_err(&instance->pdev->dev, "sgl len : 0x%x, sgl addr : 0x%x\n",
+                                               le32_to_cpu(mfi_sgl->sge32[n].length),
+                                               le32_to_cpu(mfi_sgl->sge32[n].phys_addr));
                        }
                }
        } /*for max_cmd*/
 
                cmd = instance->cmd_list[i];
 
-               if(cmd->sync_cmd == 1){
+               if (cmd->sync_cmd == 1)
                        dev_err(&instance->pdev->dev, "0x%08lx : ", (unsigned long)cmd->frame_phys_addr);
-               }
        }
        dev_err(&instance->pdev->dev, "[%d]: Dumping Done\n\n",instance->host->host_no);
 }
 static int megasas_slave_configure(struct scsi_device *sdev)
 {
        /*
-       * The RAID firmware may require extended timeouts.
-       */
+        * The RAID firmware may require extended timeouts.
+        */
        blk_queue_rq_timeout(sdev->request_queue,
                MEGASAS_DEFAULT_CMD_TIMEOUT * HZ);
 
 
 static int megasas_slave_alloc(struct scsi_device *sdev)
 {
-       u16             pd_index = 0;
+       u16 pd_index = 0;
        struct megasas_instance *instance ;
+
        instance = megasas_lookup_instance(sdev->host->host_no);
        if (sdev->channel < MEGASAS_MAX_PD_CHANNELS) {
                /*
                (instance->pdev->device == PCI_DEVICE_ID_LSI_PLASMA) ||
                (instance->pdev->device == PCI_DEVICE_ID_LSI_INVADER) ||
                (instance->pdev->device == PCI_DEVICE_ID_LSI_FURY)) {
-               writel(MFI_STOP_ADP,
-                       &instance->reg_set->doorbell);
+               writel(MFI_STOP_ADP, &instance->reg_set->doorbell);
                /* Flush */
                readl(&instance->reg_set->doorbell);
                if (instance->mpio && instance->requestorId)
        unsigned long flags;
 
        /* If we have already declared adapter dead, donot complete cmds */
-       if (instance->adprecovery == MEGASAS_HW_CRITICAL_ERROR )
+       if (instance->adprecovery == MEGASAS_HW_CRITICAL_ERROR)
                return;
 
        spin_lock_irqsave(&instance->completion_lock, flags);
                if (adprecovery != MEGASAS_HBA_OPERATIONAL) {
                        dev_notice(&instance->pdev->dev, "reset: Stopping HBA.\n");
                        spin_lock_irqsave(&instance->hba_lock, flags);
-                       instance->adprecovery   = MEGASAS_HW_CRITICAL_ERROR;
+                       instance->adprecovery = MEGASAS_HW_CRITICAL_ERROR;
                        spin_unlock_irqrestore(&instance->hba_lock, flags);
                        return FAILED;
                }
 
-               reset_index     = 0;
+               reset_index = 0;
                while (!list_empty(&clist_local)) {
-                       reset_cmd       = list_entry((&clist_local)->next,
+                       reset_cmd = list_entry((&clist_local)->next,
                                                struct megasas_cmd, list);
                        list_del_init(&reset_cmd->list);
                        if (reset_cmd->scmd) {
        }
 
        for (i = 0; i < resetwaittime; i++) {
-
                int outstanding = atomic_read(&instance->fw_outstanding);
 
                if (!outstanding)
                i++;
        } while (i <= 3);
 
-       if (atomic_read(&instance->fw_outstanding) &&
-                                       !kill_adapter_flag) {
+       if (atomic_read(&instance->fw_outstanding) && !kill_adapter_flag) {
                if (instance->disableOnlineCtrlReset == 0) {
-
                        megasas_do_ocr(instance);
 
                        /* wait for 5 secs to let FW finish the pending cmds */
                                        (kill_adapter_flag == 2)) {
                dev_notice(&instance->pdev->dev, "pending cmds after reset\n");
                /*
-               * Send signal to FW to stop processing any pending cmds.
-               * The controller will be taken offline by the OS now.
-               */
+                * Send signal to FW to stop processing any pending cmds.
+                * The controller will be taken offline by the OS now.
+                */
                if ((instance->pdev->device ==
                        PCI_DEVICE_ID_LSI_SAS0073SKINNY) ||
                        (instance->pdev->device ==
                }
                megasas_dump_pending_frames(instance);
                spin_lock_irqsave(&instance->hba_lock, flags);
-               instance->adprecovery   = MEGASAS_HW_CRITICAL_ERROR;
+               instance->adprecovery = MEGASAS_HW_CRITICAL_ERROR;
                spin_unlock_irqrestore(&instance->hba_lock, flags);
                return FAILED;
        }
  */
 static int megasas_reset_device(struct scsi_cmnd *scmd)
 {
-       int ret;
-
        /*
         * First wait for all commands to complete
         */
-       ret = megasas_generic_reset(scmd);
-
-       return ret;
+       return megasas_generic_reset(scmd);
 }
 
 /**
 {
        int ret;
        struct megasas_instance *instance;
+
        instance = (struct megasas_instance *)scmd->device->host->hostdata;
 
        /*
 
 /**
  * megasas_bios_param - Returns disk geometry for a disk
- * @sdev:              device handle
+ * @sdev:              device handle
  * @bdev:              block device
  * @capacity:          drive capacity
  * @geom:              geometry parameters
        int sectors;
        sector_t cylinders;
        unsigned long tmp;
+
        /* Default heads (64) & sectors (32) */
        heads = 64;
        sectors = 32;
 megasas_service_aen(struct megasas_instance *instance, struct megasas_cmd *cmd)
 {
        unsigned long flags;
+
        /*
         * Don't signal app if it is just an aborted previously registered aen
         */
        if ((instance->unload == 0) &&
                ((instance->issuepend_done == 1))) {
                struct megasas_aen_event *ev;
+
                ev = kzalloc(sizeof(*ev), GFP_ATOMIC);
                if (!ev) {
                        dev_err(&instance->pdev->dev, "megasas_service_aen: out of memory\n");
 
        buff_addr = (unsigned long) buf;
 
-       if (buff_offset >
-               (instance->fw_crash_buffer_size * dmachunk)) {
+       if (buff_offset > (instance->fw_crash_buffer_size * dmachunk)) {
                dev_err(&instance->pdev->dev,
                        "Firmware crash dump offset is out of range\n");
                spin_unlock_irqrestore(&instance->crashdump_lock, flags);
 
        src_addr = (unsigned long)instance->crash_buf[buff_offset / dmachunk] +
                (buff_offset % dmachunk);
-       memcpy(buf, (void *)src_addr,  size);
+       memcpy(buf, (void *)src_addr, size);
        spin_unlock_irqrestore(&instance->crashdump_lock, flags);
 
        return size;
        struct Scsi_Host *shost = class_to_shost(cdev);
        struct megasas_instance *instance =
                (struct megasas_instance *) shost->hostdata;
+
        return snprintf(buf, PAGE_SIZE, "%d\n", instance->fw_crash_state);
 }
 
                cmd->cmd_status_drv = 0;
                wake_up(&instance->abort_cmd_wait_q);
        }
-
-       return;
 }
 
 /**
  * @instance:                  Adapter soft state
  * @cmd:                       Command to be completed
  * @alt_status:                        If non-zero, use this value as status to
- *                             SCSI mid-layer instead of the value returned
- *                             by the FW. This should be used if caller wants
- *                             an alternate status (as in the case of aborted
- *                             commands)
+ *                             SCSI mid-layer instead of the value returned
+ *                             by the FW. This should be used if caller wants
+ *                             an alternate status (as in the case of aborted
+ *                             commands)
  */
 void
 megasas_complete_cmd(struct megasas_instance *instance, struct megasas_cmd *cmd,
 
 /**
  * megasas_issue_pending_cmds_again -  issue all pending cmds
- *                                     in FW again because of the fw reset
+ *                                     in FW again because of the fw reset
  * @instance:                          Adapter soft state
  */
 static inline void
        spin_unlock_irqrestore(&instance->hba_lock, flags);
 
        while (!list_empty(&clist_local)) {
-               cmd     = list_entry((&clist_local)->next,
+               cmd = list_entry((&clist_local)->next,
                                        struct megasas_cmd, list);
                list_del_init(&cmd->list);
 
                                                "issue it again.\n", cmd);
                        cmd->cmd_status_drv = MFI_STAT_INVALID_STATUS;
                        instance->instancet->fire_cmd(instance,
-                                                       cmd->frame_phys_addr ,
+                                                       cmd->frame_phys_addr,
                                                        0, instance->reg_set);
                } else if (cmd->scmd) {
                        dev_notice(&instance->pdev->dev, "%p scsi cmd [%02x]"
                dev_notice(&instance->pdev->dev, "aen_cmd in def process\n");
                megasas_return_cmd(instance, instance->aen_cmd);
 
-               instance->aen_cmd       = NULL;
+               instance->aen_cmd = NULL;
        }
 
        /*
-       * Initiate AEN (Asynchronous Event Notification)
-       */
+        * Initiate AEN (Asynchronous Event Notification)
+        */
        seq_num = instance->last_seq_num;
        class_locale.members.reserved = 0;
        class_locale.members.locale = MR_EVT_LOCALE_ALL;
        u32 defer_index;
        unsigned long flags;
 
-       defer_index     = 0;
+       defer_index = 0;
        spin_lock_irqsave(&instance->mfi_pool_lock, flags);
        for (i = 0; i < max_cmd; i++) {
                cmd = instance->cmd_list[i];
 
                atomic_set(&instance->fw_reset_no_pci_access, 1);
                instance->instancet->adp_reset(instance, instance->reg_set);
-               atomic_set(&instance->fw_reset_no_pci_access, 0 );
+               atomic_set(&instance->fw_reset_no_pci_access, 0);
 
                dev_notice(&instance->pdev->dev, "FW restarted successfully,"
                                        "initiating next stage...\n");
                dev_notice(&instance->pdev->dev, "HBA recovery state machine,"
                                        "state 2 starting...\n");
 
-               /*waitting for about 20 second before start the second init*/
+               /* waiting for about 20 second before start the second init */
                for (wait = 0; wait < 30; wait++) {
                        msleep(1000);
                }
                megasas_issue_pending_cmds_again(instance);
                instance->issuepend_done = 1;
        }
-       return ;
 }
 
 /**
  * megasas_deplete_reply_queue -       Processes all completed commands
  * @instance:                          Adapter soft state
  * @alt_status:                                Alternate status to be returned to
- *                                     SCSI mid-layer instead of the status
- *                                     returned by the FW
+ *                                     SCSI mid-layer instead of the status
+ *                                     returned by the FW
  * Note: this must be called with hba lock held
  */
 static int
        struct megasas_irq_context *irq_context = devp;
        struct megasas_instance *instance = irq_context->instance;
        unsigned long flags;
-       irqreturn_t     rc;
+       irqreturn_t rc;
 
        if (atomic_read(&instance->fw_reset_no_pci_access))
                return IRQ_HANDLED;
 
        spin_lock_irqsave(&instance->hba_lock, flags);
-       rc =  megasas_deplete_reply_queue(instance, DID_OK);
+       rc = megasas_deplete_reply_queue(instance, DID_OK);
        spin_unlock_irqrestore(&instance->hba_lock, flags);
 
        return rc;
        sge_sz = (IS_DMA64) ? sizeof(struct megasas_sge64) :
            sizeof(struct megasas_sge32);
 
-       if (instance->flag_ieee) {
+       if (instance->flag_ieee)
                sge_sz = sizeof(struct megasas_sge_skinny);
-       }
 
        /*
         * For MFI controllers.
 void megasas_free_cmds(struct megasas_instance *instance)
 {
        int i;
+
        /* First free the MFI frame pool */
        megasas_teardown_frame_pool(instance);
 
                ret = megasas_issue_polled(instance, cmd);
 
        /*
-       * the following function will get the instance PD LIST.
-       */
+        * the following function will get the instance PD LIST.
+        */
 
        pd_addr = ci->addr;
 
-       if ( ret == 0 &&
+       if (ret == 0 &&
             (le32_to_cpu(ci->count) <
                  (MEGASAS_MAX_PD_CHANNELS * MEGASAS_MAX_DEV_PER_CHANNEL))) {
 
                instance->supportmax256vd ? "Extended VD(240 VD)firmware" :
                "Legacy(64 VD) firmware");
 
-       old_map_sz =  sizeof(struct MR_FW_RAID_MAP) +
+       old_map_sz = sizeof(struct MR_FW_RAID_MAP) +
                                (sizeof(struct MR_LD_SPAN_MAP) *
                                (instance->fw_supported_vd_count - 1));
-       new_map_sz =  sizeof(struct MR_FW_RAID_MAP_EXT);
-       fusion->drv_map_sz =  sizeof(struct MR_DRV_RAID_MAP) +
+       new_map_sz = sizeof(struct MR_FW_RAID_MAP_EXT);
+       fusion->drv_map_sz = sizeof(struct MR_DRV_RAID_MAP) +
                                (sizeof(struct MR_LD_SPAN_MAP) *
                                (instance->drv_supported_vd_count - 1));
 
                fusion->current_map_sz = new_map_sz;
        else
                fusion->current_map_sz = old_map_sz;
-
 }
 
 /**
 megasas_issue_init_mfi(struct megasas_instance *instance)
 {
        __le32 context;
-
        struct megasas_cmd *cmd;
-
        struct megasas_init_frame *init_frame;
        struct megasas_init_queue_info *initq_info;
        dma_addr_t init_frame_h;
        dev_err(&instance->pdev->dev, "INIT adapter done\n");
 
        /** for passthrough
-       * the following function will get the PD LIST.
-       */
-
-       memset(instance->pd_list, 0 ,
+        * the following function will get the PD LIST.
+        */
+       memset(instance->pd_list, 0,
                (MEGASAS_MAX_PD * sizeof(struct megasas_pd_list)));
        if (megasas_get_pd_list(instance) < 0) {
                dev_err(&instance->pdev->dev, "failed to get PD list\n");
                le16_to_cpu(ctrl_info->max_strips_per_io);
        max_sectors_2 = le32_to_cpu(ctrl_info->max_request_size);
 
-       tmp_sectors = min_t(u32, max_sectors_1 , max_sectors_2);
+       tmp_sectors = min_t(u32, max_sectors_1, max_sectors_2);
 
        instance->disableOnlineCtrlReset =
        ctrl_info->properties.OnOffProperties.disableOnlineCtrlReset;
 static int megasas_io_attach(struct megasas_instance *instance)
 {
        struct Scsi_Host *host = instance->host;
-       u32             error;
+       u32 error;
 
        /*
         * Export parameters required by SCSI mid-layer
 megasas_set_dma_mask(struct pci_dev *pdev)
 {
        /*
-        * All our contollers are capable of performing 64-bit DMA
+        * All our controllers are capable of performing 64-bit DMA
         */
        if (IS_DMA64) {
                if (pci_set_dma_mask(pdev, DMA_BIT_MASK(64)) != 0) {
 
        instance = (struct megasas_instance *)host->hostdata;
        memset(instance, 0, sizeof(*instance));
-       atomic_set( &instance->fw_reset_no_pci_access, 0 );
+       atomic_set(&instance->fw_reset_no_pci_access, 0);
        instance->pdev = pdev;
 
        switch (instance->pdev->device) {
 
        return 0;
 
-      fail_start_aen:
-      fail_io_attach:
+fail_start_aen:
+fail_io_attach:
        megasas_mgmt_info.count--;
        megasas_mgmt_info.instance[megasas_mgmt_info.max_index] = NULL;
        megasas_mgmt_info.max_index--;
        if (instance->msix_vectors)
                pci_disable_msix(instance->pdev);
 fail_init_mfi:
-      fail_alloc_dma_buf:
+fail_alloc_dma_buf:
        if (instance->evt_detail)
                pci_free_consistent(pdev, sizeof(struct megasas_evt_detail),
                                    instance->evt_detail,
                                    instance->consumer_h);
        scsi_host_put(host);
 
-      fail_alloc_instance:
-      fail_set_dma_mask:
+fail_alloc_instance:
+fail_set_dma_mask:
        pci_disable_device(pdev);
 
        return -ENODEV;
                        " from %s\n", __func__);
 
        megasas_return_cmd(instance, cmd);
-
-       return;
 }
 
 /**
                        "from %s\n", __func__);
 
        megasas_return_cmd(instance, cmd);
-
-       return;
 }
 
 #ifdef CONFIG_PM
        scsi_host_put(host);
 
        pci_disable_device(pdev);
-
-       return;
 }
 
 /**
 {
        unsigned int mask;
        unsigned long flags;
+
        poll_wait(file, &megasas_poll_wait, wait);
        spin_lock_irqsave(&poll_aen_lock, flags);
        if (megasas_poll_wait_aen)
-               mask =   (POLLIN | POLLRDNORM);
-
+               mask = (POLLIN | POLLRDNORM);
        else
                mask = 0;
        megasas_poll_wait_aen = 0;
  * @cmd:       MFI command frame
  */
 
-static int megasas_set_crash_dump_params_ioctl(
-       struct megasas_cmd *cmd)
+static int megasas_set_crash_dump_params_ioctl(struct megasas_cmd *cmd)
 {
        struct megasas_instance *local_instance;
        int i, error = 0;
                error = -EFAULT;
        }
 
-      out:
+out:
        if (sense) {
                dma_free_coherent(&instance->pdev->dev, ioc->sense_len,
                                    sense, sense_handle);
        spin_unlock_irqrestore(&instance->hba_lock, flags);
 
        error = megasas_mgmt_fw_ioctl(instance, user_ioc, ioc);
-      out_up:
+out_up:
        up(&instance->ioctl_sem);
 
-      out_kfree_ioc:
+out_kfree_ioc:
        kfree(ioc);
        return error;
 }
 megasas_sysfs_set_dbg_lvl(struct device_driver *dd, const char *buf, size_t count)
 {
        int retval = count;
-       if(sscanf(buf,"%u",&megasas_dbg_lvl)<1){
+
+       if (sscanf(buf, "%u", &megasas_dbg_lvl) < 1) {
                printk(KERN_ERR "megasas: could not set dbg_lvl\n");
                retval = -EINVAL;
        }
                                pd_index =
                                (i * MEGASAS_MAX_DEV_PER_CHANNEL) + j;
 
-                               sdev1 =
-                               scsi_device_lookup(host, i, j, 0);
+                               sdev1 = scsi_device_lookup(host, i, j, 0);
 
                                if (instance->pd_list[pd_index].driveState
                                                == MR_PD_STATE_SYSTEM) {
-                                               if (!sdev1) {
+                                       if (!sdev1)
                                                scsi_add_device(host, i, j, 0);
-                                               }
 
                                        if (sdev1)
                                                scsi_device_put(sdev1);
                                pd_index =
                                (i * MEGASAS_MAX_DEV_PER_CHANNEL) + j;
 
-                               sdev1 =
-                               scsi_device_lookup(host, i, j, 0);
+                               sdev1 = scsi_device_lookup(host, i, j, 0);
 
                                if (instance->pd_list[pd_index].driveState
                                        == MR_PD_STATE_SYSTEM) {
-                                       if (sdev1) {
+                                       if (sdev1)
                                                scsi_device_put(sdev1);
-                                       }
                                } else {
                                        if (sdev1) {
                                                scsi_remove_device(sdev1);
                }
        }
 
-       if ( instance->aen_cmd != NULL ) {
+       if (instance->aen_cmd != NULL) {
                kfree(ev);
                return ;
        }