Funny, there are plenty of the braces in case block where one does not need them while in here they seem to be forgotten. Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxx> --- drivers/scsi/arcmsr/arcmsr_hba.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c index f91f79c..b57834b 100644 --- a/drivers/scsi/arcmsr/arcmsr_hba.c +++ b/drivers/scsi/arcmsr/arcmsr_hba.c @@ -673,10 +673,11 @@ static void arcmsr_report_ccb_state(struct AdapterControlBlock *acb, \ id = ccb->pcmd->device->id; lun = ccb->pcmd->device->lun; if (!(flag_ccb & ARCMSR_CCBREPLY_FLAG_ERROR)) { - if (acb->devstate[id][lun] == ARECA_RAID_GONE) + if (acb->devstate[id][lun] == ARECA_RAID_GONE) { acb->devstate[id][lun] = ARECA_RAID_GOOD; ccb->pcmd->result = DID_OK << 16; arcmsr_ccb_complete(ccb, 1); + } } else { switch (ccb->arcmsr_cdb.DeviceStatus) { case ARCMSR_DEV_SELECT_TIMEOUT: { -- 1.5.2.2