From 5c1b85e209af41c7e99a2bfd21dc8fe840b342d8 Mon Sep 17 00:00:00 2001 From: Alexey Dobriyan Date: Wed, 8 Mar 2006 00:14:37 -0800 Subject: [SCSI] drivers/scsi/FlashPoint.c: don't use parenthesis with "return" Signed-off-by: Alexey Dobriyan Signed-off-by: Andrew Morton Signed-off-by: James Bottomley --- drivers/scsi/FlashPoint.c | 130 +++++++++++++++++++++++----------------------- 1 file changed, 65 insertions(+), 65 deletions(-) (limited to 'drivers/scsi/FlashPoint.c') diff --git a/drivers/scsi/FlashPoint.c b/drivers/scsi/FlashPoint.c index ce49fbcafbb..8e3d949b711 100644 --- a/drivers/scsi/FlashPoint.c +++ b/drivers/scsi/FlashPoint.c @@ -945,16 +945,16 @@ static int FlashPoint_ProbeHostAdapter(struct sccb_mgr_info *pCardInfo) ioport = pCardInfo->si_baseaddr; if (RD_HARPOON(ioport + hp_vendor_id_0) != ORION_VEND_0) - return ((int)FAILURE); + return (int)FAILURE; if ((RD_HARPOON(ioport + hp_vendor_id_1) != ORION_VEND_1)) - return ((int)FAILURE); + return (int)FAILURE; if ((RD_HARPOON(ioport + hp_device_id_0) != ORION_DEV_0)) - return ((int)FAILURE); + return (int)FAILURE; if ((RD_HARPOON(ioport + hp_device_id_1) != ORION_DEV_1)) - return ((int)FAILURE); + return (int)FAILURE; if (RD_HARPOON(ioport + hp_rev_num) != 0x0f) { @@ -964,7 +964,7 @@ static int FlashPoint_ProbeHostAdapter(struct sccb_mgr_info *pCardInfo) device. */ if (RD_HARPOON(ioport + hp_sub_device_id_0) & 0x0f) - return ((int)FAILURE); + return (int)FAILURE; } if (first_time) { @@ -986,7 +986,7 @@ static int FlashPoint_ProbeHostAdapter(struct sccb_mgr_info *pCardInfo) pCurrNvRam->niBaseAddr = ioport; FPT_RNVRamData(pCurrNvRam); } else - return ((int)FAILURE); + return (int)FAILURE; } } else pCurrNvRam = NULL; @@ -1194,7 +1194,7 @@ static int FlashPoint_ProbeHostAdapter(struct sccb_mgr_info *pCardInfo) pCardInfo->si_present = 0x01; - return (0); + return 0; } /*--------------------------------------------------------------------- @@ -1220,7 +1220,7 @@ static unsigned long FlashPoint_HardwareResetHostAdapter(struct sccb_mgr_info if (thisCard == MAX_CARDS) { - return (FAILURE); + return FAILURE; } if (FPT_BL_Card[thisCard].ioPort == ioport) { @@ -1388,7 +1388,7 @@ static unsigned long FlashPoint_HardwareResetHostAdapter(struct sccb_mgr_info (unsigned char)(RD_HARPOON((ioport + hp_semaphore)) | SCCB_MGR_PRESENT)); - return ((unsigned long)CurrCard); + return (unsigned long)CurrCard; } static void FlashPoint_ReleaseHostAdapter(unsigned long pCurrCard) @@ -1460,7 +1460,7 @@ static void FPT_RNVRamData(struct nvram_info *pNvRamInfo) static unsigned char FPT_RdStack(unsigned long portBase, unsigned char index) { WR_HARPOON(portBase + hp_stack_addr, index); - return (RD_HARPOON(portBase + hp_stack_data)); + return RD_HARPOON(portBase + hp_stack_data); } static void FPT_WrStack(unsigned long portBase, unsigned char index, @@ -1473,14 +1473,14 @@ static void FPT_WrStack(unsigned long portBase, unsigned char index, static unsigned char FPT_ChkIfChipInitialized(unsigned long ioPort) { if ((RD_HARPOON(ioPort + hp_arb_id) & 0x0f) != FPT_RdStack(ioPort, 4)) - return (0); + return 0; if ((RD_HARPOON(ioPort + hp_clkctrl_0) & CLKCTRL_DEFAULT) != CLKCTRL_DEFAULT) - return (0); + return 0; if ((RD_HARPOON(ioPort + hp_seltimeout) == TO_250ms) || (RD_HARPOON(ioPort + hp_seltimeout) == TO_290ms)) - return (1); - return (0); + return 1; + return 0; } @@ -1642,14 +1642,14 @@ static int FlashPoint_AbortCCB(unsigned long pCurrCard, struct sccb *p_Sccb) callback = p_Sccb->SccbCallback; callback(p_Sccb); - return (0); + return 0; } else { if (((struct sccb_card *)pCurrCard)->currentSCCB == p_Sccb) { p_Sccb->SccbStatus = SCCB_ABORT; - return (0); + return 0; } @@ -1691,7 +1691,7 @@ static int FlashPoint_AbortCCB(unsigned long pCurrCard, struct sccb *p_Sccb) } } MENABLE_INT(ioport); - return (0); + return 0; } else { currTar_Info = &FPT_sccbMgrTbl[thisCard][p_Sccb-> @@ -1702,13 +1702,13 @@ static int FlashPoint_AbortCCB(unsigned long pCurrCard, struct sccb *p_Sccb) LunDiscQ_Idx[p_Sccb->Lun]] == p_Sccb) { p_Sccb->SccbStatus = SCCB_ABORT; - return (0); + return 0; } } } } } - return (-1); + return -1; } /*--------------------------------------------------------------------- @@ -1726,12 +1726,12 @@ static unsigned char FlashPoint_InterruptPending(unsigned long pCurrCard) ioport = ((struct sccb_card *)pCurrCard)->ioPort; if (RD_HARPOON(ioport + hp_int_status) & INT_ASSERTED) { - return (1); + return 1; } else - return (0); + return 0; } /*--------------------------------------------------------------------- @@ -1783,7 +1783,7 @@ static int FlashPoint_HandleInterrupt(unsigned long pCurrCard) if (result) { MENABLE_INT(ioport); - return (result); + return result; } } @@ -1997,7 +1997,7 @@ static int FlashPoint_HandleInterrupt(unsigned long pCurrCard) MENABLE_INT(ioport); - return (0); + return 0; } /*--------------------------------------------------------------------- @@ -2092,7 +2092,7 @@ static unsigned char FPT_SccbMgr_bad_isr(unsigned long p_port, FPT_scini(p_card, pCurrCard->ourId, 0); - return (0xFF); + return 0xFF; } else if (p_int & FIFO) { @@ -2150,7 +2150,7 @@ static unsigned char FPT_SccbMgr_bad_isr(unsigned long p_port, WRW_HARPOON((p_port + hp_intstat), SCAM_SEL); } - return (0x00); + return 0x00; } /*--------------------------------------------------------------------- @@ -2295,12 +2295,12 @@ static unsigned char FPT_sfm(unsigned long port, struct sccb *pCurrSCCB) } if (TimeOutLoop > 20000) { WRW_HARPOON((port + hp_intstat), PARITY); - return (message); + return message; } if ((RD_HARPOON(port + hp_scsisig) & S_SCSI_PHZ) != S_MSGI_PH) { WRW_HARPOON((port + hp_intstat), PARITY); - return (message); + return message; } WR_HARPOON(port + hp_portctrl_0, SCSI_PORT); @@ -2315,7 +2315,7 @@ static unsigned char FPT_sfm(unsigned long port, struct sccb *pCurrSCCB) WR_HARPOON(port + hp_xferstat, 0); WR_HARPOON(port + hp_fiforead, 0); WR_HARPOON(port + hp_fifowrite, 0); - return (message); + return message; } /*--------------------------------------------------------------------- @@ -3219,14 +3219,14 @@ static unsigned char FPT_sisyncn(unsigned long port, unsigned char p_card, (AUTO_IMMED + CMD_ONLY_STRT)); } - return (1); + return 1; } else { currTar_Info->TarStatus |= (unsigned char)SYNC_SUPPORTED; currTar_Info->TarEEValue &= ~EE_SYNC_MASK; - return (0); + return 0; } } @@ -3431,7 +3431,7 @@ static unsigned char FPT_siwidn(unsigned long port, unsigned char p_card) ~(unsigned char)TAR_WIDE_MASK) | (unsigned char)WIDE_ENABLED); - return (1); + return 1; } else { @@ -3441,7 +3441,7 @@ static unsigned char FPT_siwidn(unsigned long port, unsigned char p_card) WIDE_NEGOCIATED); currTar_Info->TarEEValue &= ~EE_WIDE_SCSI; - return (0); + return 0; } } @@ -5155,11 +5155,11 @@ static unsigned char FPT_busMstrTimeOut(unsigned long p_port) RD_HARPOON(p_port + hp_int_status); /*Clear command complete */ if (RD_HARPOON(p_port + hp_ext_status) & BM_CMD_BUSY) { - return (1); + return 1; } else { - return (0); + return 0; } } @@ -5721,10 +5721,10 @@ static int FPT_scarb(unsigned long p_port, unsigned char p_sel_type) } if (RD_HARPOON(p_port + hp_scsisig) & SCSI_SEL) - return (0); + return 0; if (RD_HARPOON(p_port + hp_scsidata_0) != 00) - return (0); + return 0; WR_HARPOON(p_port + hp_scsisig, (RD_HARPOON(p_port + hp_scsisig) | SCSI_BSY)); @@ -5734,7 +5734,7 @@ static int FPT_scarb(unsigned long p_port, unsigned char p_sel_type) WR_HARPOON(p_port + hp_scsisig, (RD_HARPOON(p_port + hp_scsisig) & ~SCSI_BSY)); - return (0); + return 0; } WR_HARPOON(p_port + hp_scsisig, @@ -5745,7 +5745,7 @@ static int FPT_scarb(unsigned long p_port, unsigned char p_sel_type) WR_HARPOON(p_port + hp_scsisig, (RD_HARPOON(p_port + hp_scsisig) & ~(SCSI_BSY | SCSI_SEL))); - return (0); + return 0; } } @@ -5764,7 +5764,7 @@ static int FPT_scarb(unsigned long p_port, unsigned char p_sel_type) FPT_Wait(p_port, TO_250ms); - return (1); + return 1; } /*--------------------------------------------------------------------- @@ -5956,7 +5956,7 @@ static unsigned char FPT_scxferc(unsigned long p_port, unsigned char p_data) FPT_scwirod(p_port, BIT(6)); /*Wait for DB6 to be released. */ - return (ret_data); + return ret_data; } /*--------------------------------------------------------------------- @@ -5994,22 +5994,22 @@ static unsigned char FPT_scsendi(unsigned long p_port, } if ((ret_data & 0x1C) == 0x10) - return (0x00); /*End of isolation stage, we won! */ + return 0x00; /*End of isolation stage, we won! */ if (ret_data & 0x1C) - return (0xFF); + return 0xFF; if ((defer) && (!(ret_data & 0x1F))) - return (0x01); /*End of isolation stage, we lost. */ + return 0x01; /*End of isolation stage, we lost. */ } /*bit loop */ } /*byte loop */ if (defer) - return (0x01); /*We lost */ + return 0x01; /*We lost */ else - return (0); /*We WON! Yeeessss! */ + return 0; /*We WON! Yeeessss! */ } /*--------------------------------------------------------------------- @@ -6034,7 +6034,7 @@ static unsigned char FPT_sciso(unsigned long p_port, ret_data = FPT_scxferc(p_port, 0); if (ret_data & 0xFC) - return (0xFF); + return 0xFF; else { @@ -6056,9 +6056,9 @@ static unsigned char FPT_sciso(unsigned long p_port, } */ if (byte_cnt) - return (0x00); + return 0x00; else - return (0xFF); + return 0xFF; } } /*bit loop */ @@ -6067,7 +6067,7 @@ static unsigned char FPT_sciso(unsigned long p_port, } /*byte loop */ - return (0); + return 0; } /*--------------------------------------------------------------------- @@ -6142,10 +6142,10 @@ static unsigned char FPT_scvalq(unsigned char p_quintet) } if (p_quintet & 0x18) - return (0); + return 0; else - return (1); + return 1; } /*--------------------------------------------------------------------- @@ -6209,7 +6209,7 @@ static unsigned char FPT_scsell(unsigned long p_port, unsigned char targ_id) (RD_HARPOON(p_port + hp_page_ctrl) & ~G_INT_DISABLE)); - return (0); /*No legacy device */ + return 0; /*No legacy device */ } else { @@ -6228,7 +6228,7 @@ static unsigned char FPT_scsell(unsigned long p_port, unsigned char targ_id) (RD_HARPOON(p_port + hp_page_ctrl) & ~G_INT_DISABLE)); - return (1); /*Found one of them oldies! */ + return 1; /*Found one of them oldies! */ } } @@ -6342,7 +6342,7 @@ static unsigned char FPT_scmachid(unsigned char p_card, if (match) { FPT_scamInfo[i].state = ID_ASSIGNED; - return (i); + return i; } } @@ -6372,7 +6372,7 @@ static unsigned char FPT_scmachid(unsigned char p_card, if (FPT_BL_Card[p_card].pNvRamInfo == NULL) FPT_BL_Card[p_card].globalFlags |= F_UPDATE_EEPROM; - return (match); + return match; } @@ -6387,7 +6387,7 @@ static unsigned char FPT_scmachid(unsigned char p_card, } if (p_id_string[0] & BIT(7)) { - return (CLR_PRIORITY); + return CLR_PRIORITY; } if (p_id_string[0] & BIT(5)) @@ -6416,7 +6416,7 @@ static unsigned char FPT_scmachid(unsigned char p_card, if (FPT_BL_Card[p_card].pNvRamInfo == NULL) FPT_BL_Card[p_card].globalFlags |= F_UPDATE_EEPROM; - return (match); + return match; } @@ -6430,7 +6430,7 @@ static unsigned char FPT_scmachid(unsigned char p_card, } } - return (NO_ID_AVAIL); + return NO_ID_AVAIL; } /*--------------------------------------------------------------------- @@ -7181,7 +7181,7 @@ static unsigned char FPT_queueFindSccb(struct sccb *p_SCCB, currTar_Info->TarSelQ_Cnt--; - return (1); + return 1; } else { @@ -7189,7 +7189,7 @@ static unsigned char FPT_queueFindSccb(struct sccb *p_SCCB, } } - return (0); + return 0; } @@ -7416,14 +7416,14 @@ static unsigned short FPT_utilEERead(unsigned long p_port, ee_data2 = FPT_utilEEReadOrg(p_port, ee_addr); if (ee_data1 == ee_data2) - return (ee_data1); + return ee_data1; ee_data1 = ee_data2; i++; } while (i < 4); - return (ee_data1); + return ee_data1; } /*--------------------------------------------------------------------- @@ -7471,7 +7471,7 @@ static unsigned short FPT_utilEEReadOrg(unsigned long p_port, WR_HARPOON(p_port + hp_ee_ctrl, (ee_value | SEE_MS)); /*Turn off CS */ WR_HARPOON(p_port + hp_ee_ctrl, ee_value); /*Turn off Master Select */ - return (ee_data); + return ee_data; } /*--------------------------------------------------------------------- @@ -7559,7 +7559,7 @@ static unsigned short FPT_CalcCrc16(unsigned char buffer[]) ch >>= 1; } } - return (crc); + return crc; } static unsigned char FPT_CalcLrc(unsigned char buffer[]) @@ -7569,7 +7569,7 @@ static unsigned char FPT_CalcLrc(unsigned char buffer[]) lrc = 0; for (i = 0; i < ID_STRING_LENGTH; i++) lrc ^= buffer[i]; - return (lrc); + return lrc; } /* -- cgit v1.2.3-70-g09d2